mirror of
https://github.com/Matysh/houseplan-card
synced 2026-10-04 05:41:34 +00:00
Счётчик исключений внутри карточки живёт в demo/serve.mjs, и читала его одна функция — finish(). Шесть смоков её не вызывали вовсе: у трёх своя развязка (`if (!ok) process.exit(1)`), у двух throw из try/finally, у smoke_entry_stale ни того ни другого. Необработанное исключение во время этих шести проходило незамеченным всегда — в лог печаталось EXC, а прогон оставался зелёным. smoke_entry_stale был хуже остальных: он складывал неудачи в _failures через check/checkAll, но их никто не печатал и код возврата не выставлял. То есть смок не мог провалиться в принципе — ровно паттерн «печатали булевы значения и всегда выходили нулём», который шапка serve.mjs описывает как исправленный в 2026-07-27. Добавлен reportPageErrors(): тот же вердикт, что у finish(), для смоков со своей логикой выхода. Каждый из шести теперь вердикт запрашивает, а smoke_entry_stale получил finish() и вместе с ним настоящий код возврата. Вердикт обязан ОСТАНАВЛИВАТЬ, а не только помечать. Первый заход выставлял process.exitCode, и отрицательный прогон напечатал «FAILED: 1 uncaught exception(s)» и следом «OK deep-link: …»: код был верным, вывод противоречивым, а читают вывод. Доказано отрицательным прогоном, а не рассуждением: на ветке experiment/407-negative smoke_deeplink получил намеренное исключение внутри карточки, шард 2/3 упал с exit code 1, в логе FAIL и FAILED без строки успеха. Ветка удалена. Гейт против повторения — test/smoke-harness-contract.test.mjs: он падает, если смок не запрашивает вердикт или запрашивает, не останавливаясь. На origin/dev до починки он находил ровно шесть файлов, после — ноль. Issue: #407 User-Visible: no
130 lines
7.3 KiB
JavaScript
130 lines
7.3 KiB
JavaScript
// Shared launcher for demo captures: starts headless Chromium serving demo/srv/
|
|
// via request interception (no HTTP server needed). Usage: const {page,browser}=await launch();
|
|
import { chromium } from 'playwright';
|
|
import { assertFreshDemoBundleUnlessAllowed } from './bundle-freshness.mjs';
|
|
import { ensureHarnessEditorRuntime } from './editor-runtime-compat.mjs';
|
|
import { readFileSync, existsSync } from 'node:fs';
|
|
import { fileURLToPath } from 'node:url';
|
|
import { dirname } from 'node:path';
|
|
const ROOT = dirname(fileURLToPath(import.meta.url)) + '/srv';
|
|
// Корень репозитория, а не каталог раздачи: фингерпринт считается по
|
|
// src/** и demo/fixtures, которых внутри demo/srv нет (#236).
|
|
const REPO_ROOT = dirname(dirname(fileURLToPath(import.meta.url)));
|
|
const CT = { '.html': 'text/html', '.js': 'text/javascript', '.svg': 'image/svg+xml' };
|
|
// ---- assertion harness (audit T1) --------------------------------------
|
|
// Until 2026-07-27 the smokes printed booleans and always exited 0: a broken
|
|
// build reported success. `check()` accumulates named failures, `finish()`
|
|
// prints them and sets the exit code.
|
|
const _failures = [];
|
|
let _pageErrors = 0;
|
|
|
|
/** Assert one named fact. `expected` defaults to true. */
|
|
export function check(name, actual, expected = true) {
|
|
const ok = JSON.stringify(actual) === JSON.stringify(expected);
|
|
if (!ok) _failures.push(`${name}: expected ${JSON.stringify(expected)}, got ${JSON.stringify(actual)}`);
|
|
return ok;
|
|
}
|
|
|
|
/** Assert a whole result object: every key must equal true unless listed. */
|
|
export function checkAll(out, expected = {}) {
|
|
for (const [k, v] of Object.entries(out)) check(k, v, k in expected ? expected[k] : true);
|
|
return out;
|
|
}
|
|
|
|
/**
|
|
* Тот же вердикт по исключениям в карточке, что у `finish()`, для смоков со
|
|
* своей логикой выхода (#407).
|
|
*
|
|
* Счётчик `_pageErrors` живёт здесь, а читал его только `finish()`. Шесть
|
|
* смоков `finish()` не вызывали вовсе — у каждого была своя ручная развязка
|
|
* (`if (!ok) process.exit(1)`) либо `throw` из try/finally, и ни одна не
|
|
* спрашивала про исключения. То есть необработанное исключение внутри карточки
|
|
* во время этих шести проходило незамеченным всегда: в лог печаталось `EXC`, а
|
|
* прогон оставался зелёным.
|
|
*
|
|
* Функция ничего не бросает намеренно. Смок с ручной развязкой продолжает
|
|
* решать сам, что печатать и когда выходить; здесь только выставляется код
|
|
* возврата, который дальше уже не отнять — `process.exitCode` переживёт любой
|
|
* последующий `console.log`.
|
|
*
|
|
* @returns true, если карточка бросала — чтобы вызывающий мог добавить своё
|
|
* сообщение, не считая исключения заново.
|
|
*/
|
|
export function reportPageErrors() {
|
|
if (!_pageErrors) return false;
|
|
console.error(`FAILED: ${_pageErrors} uncaught exception(s) inside the card`);
|
|
process.exitCode = 1;
|
|
return true;
|
|
}
|
|
|
|
/** Print the result, report failures, close the browser, set the exit code. */
|
|
export async function finish(browser, out) {
|
|
if (out !== undefined) console.log(JSON.stringify(out, null, 1));
|
|
if (_pageErrors) _failures.push(`${_pageErrors} uncaught exception(s) inside the card`);
|
|
await browser?.close?.();
|
|
if (_failures.length) {
|
|
console.error('\nFAILED (' + _failures.length + '):');
|
|
for (const f of _failures) console.error(' - ' + f);
|
|
process.exitCode = 1;
|
|
} else {
|
|
console.log('OK');
|
|
}
|
|
}
|
|
|
|
async function launchInternal(
|
|
preloadEditorRuntime,
|
|
viewport = { width: 820, height: 760 },
|
|
scale = 1,
|
|
browserArgs = [],
|
|
contextOptions = {},
|
|
serveRoot = ROOT,
|
|
repoRoot = REPO_ROOT,
|
|
) {
|
|
const browser = await chromium.launch({ args: ['--no-sandbox', ...browserArgs] });
|
|
const page = await (await browser.newContext({
|
|
viewport, deviceScaleFactor: scale, ...contextOptions,
|
|
})).newPage();
|
|
// audit T1: an exception inside the card used to be logged and ignored
|
|
page.on('pageerror', (e) => { _pageErrors++; console.log('EXC', e.stack || e.message); });
|
|
await page.route('**/*', (r) => {
|
|
const u = new URL(r.request().url());
|
|
let p = decodeURIComponent(u.pathname);
|
|
if (p === '/') p = '/demo.html';
|
|
const f = serveRoot + p;
|
|
existsSync(f)
|
|
? r.fulfill({ status: 200, headers: { 'content-type': CT[p.slice(p.lastIndexOf('.'))] || 'application/octet-stream' }, body: readFileSync(f) })
|
|
: r.fulfill({ status: 404, body: 'nf' });
|
|
});
|
|
await page.goto('http://demo.local/demo.html', { waitUntil: 'domcontentloaded' });
|
|
await page.waitForFunction(() => window.__card?._model?.length > 0, { timeout: 9000 });
|
|
// Свежесть бандла проверяется здесь, а не в каждом смоке (#236). Смок читает
|
|
// demo/srv/assets/houseplan-card.js; если туда не скопирован свежий dist,
|
|
// проверяется прежняя версия карточки — и результат выглядит осмысленным,
|
|
// потому что часть проверок краснеет, а часть зеленеет. На #234 это стоило
|
|
// круга разбора: три проверки упали, четвёртая ложно прошла, поскольку старый
|
|
// код одинаково врал в двух местах, которые сверялись друг с другом.
|
|
// golden и бенчмарки эту защиту имели с самого начала, смоки — нет.
|
|
await assertFreshDemoBundleUnlessAllowed(page, repoRoot);
|
|
// HP-1552: the first-open boot veil hides the plan (visibility:hidden) until
|
|
// the stage height settles — real pointer interaction cannot hit a hidden
|
|
// plan, so every smoke starts where the user does: with the plan revealed.
|
|
await page.waitForFunction(() => window.__card._booting === false, { timeout: 9000 });
|
|
// hass flows continuously in production; the stub sets it once — nudge a rebuild
|
|
await page.evaluate(() => { const c = window.__card; c.hass = { ...c.hass }; });
|
|
await page.waitForFunction(() => window.__card._devices.length > 0, { timeout: 9000 });
|
|
// Existing product smokes exercise editor internals directly. They preload
|
|
// the new #337 runtime without changing mode; the dedicated network smoke
|
|
// uses launchColdView() and proves the real cold-View boundary separately.
|
|
// Comparative performance runs also launch the previous stable through this
|
|
// file. A pre-#337 monolithic card has no preload method and is already
|
|
// ready; a current card still has to return truthy (#380).
|
|
if (preloadEditorRuntime) {
|
|
const ready = await page.evaluate(ensureHarnessEditorRuntime);
|
|
if (!ready) throw new Error('editor runtime did not preload for browser smoke');
|
|
}
|
|
return { page, browser };
|
|
}
|
|
|
|
export const launch = (...args) => launchInternal(true, ...args);
|
|
export const launchColdView = (...args) => launchInternal(false, ...args);
|