test(web-test): обвязка check.mjs для фикстуры _hang + не ограничивать prepare

check.mjs спавнит раннер дочерним процессом и превращает «читать глазами два условия» в 0/1. Проверяет шесть вещей: раннер завершился за 90с (а не завис — это и есть суть), вердикт hang, контекст прерван с успешным logout, следующий тест зелёный (лицензия вернулась), результат зависшего теста попал в отчёт (инкрементальная запись), код выхода 1. Отдельный код 2 — стенд не поднят: у фикстуры нет своих хуков, и «нет стенда» не должно выглядеть как поломка механики. Заодно фикс собственной регрессии: hooks.prepare был обёрнут в bounded(), который ГЛОТАЕТ ошибку — упавшая пересборка стенда молча пропускалась бы, и вместо одной внятной ошибки прогон вываливал бы экран непонятных падений. Плюс бюджет 120с обрезал бы легитимно долгую пересборку большой базы. Возвращено к голому await: prepare честно долгий, а его падение обязано быть фатальным. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
2026-07-18 00:29:42 +03:00 · 2026-07-15 17:50:08 +03:00
parent dc8afb402b
commit 9352d28008
2 changed files with 135 additions and 2 deletions
@@ -33,7 +33,7 @@ export async function cmdTest(rawArgs) {
    stopRecording: 40000,   // ffmpeg has its own 30s inside
    closeContext: 20000,
    disconnect: 30000,
-    hooks: 120000,          // prepare/cleanup/beforeAll/afterAll do real work (db rebuilds)
+    hooks: 120000,          // afterAll/cleanup only — prepare/beforeAll stay unbounded (see below)
    abortAll: 30000,        // whole abort+cleanup sequence for one hung test
    probe: 2000,
  };
@@ -294,7 +294,11 @@ export async function cmdTest(rawArgs) {

  const hookLog = (...a) => W.write(`[hooks] ${a.map(String).join(' ')}\n`);
  const hookEnv = { hookArgs, log: hookLog, config };
-  if (hooks.prepare) await bounded(hooks.prepare(hookEnv), D.hooks, 'hooks.prepare');
+  // Deliberately unbounded and allowed to throw: prepare() rebuilds the stand (db-create +
+  // load + update), whose honest duration depends on the application's size — a deadline here
+  // would cut a legitimate rebuild. And its failure must stay fatal: proceeding into a run
+  // without a stand turns one clear error into a screenful of confusing ones.
+  if (hooks.prepare) await hooks.prepare(hookEnv);

  /** Force-release every open context (frees 1C licenses), then drop the browser. */
  async function shutdownAll() {
@@ -0,0 +1,129 @@
+// web-test _hang/check v1.0 — automated verdict for the hang fixture
+// Source: https://github.com/Nikolay-Shirokov/cc-1c-skills
+//
+// The fixture itself cannot assert on the runner that is running it, and its expected result
+// (`1 passed, 1 failed`, exit code 1 — the red test IS the success) is too easy to misread.
+// This harness spawns the runner as a child process and turns that into a plain 0/1.
+//
+//   node tests/web-test/_hang/check.mjs
+//
+// Exit codes: 0 — the abort machinery works; 1 — it regressed; 2 — inconclusive (stand down).
+import { spawn } from 'child_process';
+import { existsSync, readdirSync, readFileSync, rmSync, mkdirSync } from 'fs';
+import { resolve, dirname } from 'path';
+import { fileURLToPath } from 'url';
+import { tmpdir } from 'os';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const REPO = resolve(__dirname, '../../..');
+const RUNNER = resolve(REPO, '.claude/skills/web-test/scripts/run.mjs');
+const URL = 'http://localhost:9191/webtest-runner/ru_RU';
+
+// Generous: the fixture takes ~20s (10s test timeout + probe + abort + browser relaunch).
+// This bound is the point of the whole check — a regressed runner hangs forever instead.
+const DEADLINE_MS = 90000;
+
+const reportDir = resolve(tmpdir(), 'webtest-hang-check-' + process.pid);
+
+async function standIsUp() {
+  try {
+    const res = await fetch(URL, { signal: AbortSignal.timeout(5000) });
+    return res.ok;
+  } catch { return false; }
+}
+
+function runFixture() {
+  return new Promise((done) => {
+    const child = spawn(process.execPath, [RUNNER, 'test', resolve(__dirname), '--format=allure', `--report-dir=${reportDir}`], {
+      cwd: REPO,
+      stdio: ['ignore', 'pipe', 'pipe'],
+    });
+    let out = '';
+    child.stdout.on('data', d => { out += d; });
+    child.stderr.on('data', d => { out += d; });
+
+    const killer = setTimeout(() => {
+      child.kill('SIGKILL');
+      done({ out, code: null, timedOut: true });
+    }, DEADLINE_MS);
+
+    child.on('close', (code) => {
+      clearTimeout(killer);
+      done({ out, code, timedOut: false });
+    });
+  });
+}
+
+function allureResults() {
+  if (!existsSync(reportDir)) return [];
+  return readdirSync(reportDir)
+    .filter(f => f.endsWith('-result.json'))
+    .map(f => { try { return JSON.parse(readFileSync(resolve(reportDir, f), 'utf8')); } catch { return null; } })
+    .filter(Boolean);
+}
+
+// ── main ──────────────────────────────────────────────────────────────────
+if (!await standIsUp()) {
+  console.error(`INCONCLUSIVE: стенд не отвечает на ${URL}`);
+  console.error('  Фикстура проверяет раннер, а не стенд, и своих _hooks.mjs не имеет.');
+  console.error('  Подними публикацию: /web-publish webtest  (или прогони обычный набор — его хуки поднимут стенд)');
+  process.exit(2);
+}
+
+mkdirSync(reportDir, { recursive: true });
+console.log(`running fixture (deadline ${DEADLINE_MS / 1000}s)…`);
+const { out, code, timedOut } = await runFixture();
+const results = allureResults();
+
+const hung = results.find(r => /заблокированный JS-поток/.test(r.name));
+const survivor = results.find(r => /следующий тест работает/.test(r.name));
+
+const checks = [
+  {
+    name: 'раннер завершился (не завис)',
+    ok: !timedOut,
+    detail: timedOut
+      ? `процесс не завершился за ${DEADLINE_MS / 1000}s и был убит — прерывание зависшего теста НЕ работает`
+      : `exit code ${code}`,
+  },
+  {
+    name: 'зависший тест распознан как hang',
+    ok: /verdict: hang/.test(out),
+    detail: /verdict: (\S+)/.exec(out)?.[0] || 'строки verdict в выводе нет',
+  },
+  {
+    name: 'контекст прерван, сеанс 1С освобождён',
+    ok: /recovery: context aborted \(logout: (node|page|sibling)/.test(out),
+    detail: /recovery: [^\n]*/.exec(out)?.[0] || 'строки recovery в выводе нет',
+  },
+  {
+    name: 'следующий тест прошёл (прогон поехал дальше, лицензия вернулась)',
+    ok: survivor?.status === 'passed',
+    detail: survivor ? `status=${survivor.status}` : 'результата 02-survivor нет вовсе',
+  },
+  {
+    name: 'результат зависшего теста записан в отчёт',
+    ok: hung?.status === 'failed',
+    detail: hung ? `status=${hung.status}` : 'результата 01 нет — инкрементальная запись сломана',
+  },
+  {
+    name: 'код выхода 1 (падение зависшего теста — штатное)',
+    ok: code === 1,
+    detail: `exit code ${code}`,
+  },
+];
+
+console.log();
+for (const c of checks) console.log(`  ${c.ok ? '✓' : '✗'} ${c.name}\n      ${c.detail}`);
+
+const failed = checks.filter(c => !c.ok);
+console.log();
+if (failed.length) {
+  console.log(`FAIL: ${failed.length}/${checks.length} проверок не прошло — механика прерывания сломана`);
+  console.log('\n─── вывод раннера ───');
+  console.log(out.trim());
+  process.exit(1);
+}
+try { rmSync(reportDir, { recursive: true, force: true }); } catch {}
+console.log(`OK: ${checks.length}/${checks.length} — таймаут прерывает зависший тест, прогон продолжается, отчёт пишется`);
+process.exit(0);