mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-29 03:09:36 +00:00
Matysh
601e68e52d
Счётчик исключений внутри карточки живёт в 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
61 lines
3.1 KiB
JavaScript
61 lines
3.1 KiB
JavaScript
// #353 AC3b: a proxy-cached entry `houseplan-card.js` that survived an update
|
|
// points at a main chunk the manifest-gated server no longer serves. Before
|
|
// the fix the static re-export aborted the whole module and the card died
|
|
// silently; now the entry defines a fallback element with a human message,
|
|
// and `await import(entry)` + `createElement(...).setConfig(...)` stays valid.
|
|
import { readFileSync } from 'node:fs';
|
|
import { chromium } from 'playwright';
|
|
import { check, checkAll, finish } from './serve.mjs';
|
|
|
|
const entry = readFileSync('dist/houseplan-card.js');
|
|
const page404 = `<!doctype html><meta charset="utf-8"><body><script type="module">
|
|
await import('/assets/houseplan-card.js');
|
|
const card = document.createElement('houseplan-card');
|
|
card.setConfig({ type: 'custom:houseplan-card' });
|
|
document.body.appendChild(card);
|
|
window.__done = true;
|
|
<\/script></body>`;
|
|
|
|
const run = async (locale, expected) => {
|
|
const browser = await chromium.launch({ args: ['--no-sandbox'] });
|
|
const page = await (await browser.newContext({ locale })).newPage();
|
|
let pageErrors = 0;
|
|
page.on('pageerror', (error) => { pageErrors++; console.log('EXC', error.message); });
|
|
await page.route('**/*', (route) => {
|
|
const path = new URL(route.request().url()).pathname;
|
|
if (path === '/stale.html') {
|
|
return route.fulfill({ status: 200, contentType: 'text/html', body: page404 });
|
|
}
|
|
if (path === '/assets/houseplan-card.js') {
|
|
return route.fulfill({ status: 200, contentType: 'text/javascript', body: entry });
|
|
}
|
|
// Every hashed chunk of the cached build is gone after the update.
|
|
return route.fulfill({ status: 404, body: 'nf' });
|
|
});
|
|
await page.goto('http://demo.local/stale.html', { waitUntil: 'domcontentloaded' });
|
|
await page.waitForFunction(() => window.__done === true, { timeout: 9000 });
|
|
const text = await page.locator('houseplan-card').textContent();
|
|
await browser.close();
|
|
return { text, pageErrors };
|
|
};
|
|
|
|
const en = await run('en-US');
|
|
const ru = await run('ru-RU');
|
|
const fr = await run('fr-FR');
|
|
const out = {
|
|
entrySurvivesMissingChunk: en.pageErrors === 0 && ru.pageErrors === 0 && fr.pageErrors === 0,
|
|
englishMessageVisible: en.text.includes('House Plan was updated')
|
|
&& en.text.includes('reload the page'),
|
|
russianMessageVisible: ru.text.includes('House Plan обновился')
|
|
&& ru.text.includes('перезагрузите страницу'),
|
|
frenchMessageVisible: fr.text.includes('House Plan a été mis à jour')
|
|
&& fr.text.includes('recharger la page'),
|
|
};
|
|
checkAll(out);
|
|
// #407: до этого смок не мог провалиться в принципе. `check`/`checkAll`
|
|
// складывали неудачи в `_failures`, но их никто не печатал и код возврата не
|
|
// выставлял — ровно тот паттерн «печатали булевы значения и всегда выходили
|
|
// нулём», который описан в шапке serve.mjs как исправленный в 2026-07-27.
|
|
// Браузеры закрыты внутри run(), поэтому finish() получает undefined.
|
|
await finish(undefined, out);
|