mirror of
https://github.com/Nikolay-Shirokov/cc-1c-skills.git
synced 2026-08-29 06:29:41 +03:00
На Windows fs.rmSync и fs.cpSync молча не делают ничего, когда не-ASCII символы есть в самом аргументе пути (nodejs/node#61067). Для аудитории проекта это боевой сценарий: кириллическое имя пользователя даёт кириллический %TEMP%, где раннер создаёт воркспейсы, плюс кириллические имена объектов 1С в путях внутри кейсов. Замерено на восьми сборках. Затронуты 22.18.0–22.23.2 (последняя LTS Jod) и 24.12.0–24.14.0; исправны 22.15.1, 24.15.0+, 25.x, 26.x. Фикс приехал в fs-слой Node 24.15 и в ветку 22.x не бэкпортирован, поэтому «обновить Node» вопрос не закрывает. Зависимость не монотонна по версиям (24.14 чинит rmSync, но не cpSync) — гард по номеру версии невозможен, решает только сам путь. Симптомы: rmSync молча ничего не удаляет; cpSync с не-ASCII приёмником молча ничего не копирует; cpSync с не-ASCII источником валит процесс нативно (0xC0000409) мимо try/catch. Не затронуты mkdirSync, readdirSync, lstatSync, copyFileSync (включая перезапись), unlinkSync, rmdirSync — на них стоит обход. Что ломалось: под кириллическим %TEMP% фикстуры не доезжали до воркспейсов (meta-info — 15 ложных падений из 26, один кейс ложно-зелёный на пустом воркспейсе); cfe-validate/module-state-flag-without-file был красным даже при ASCII %TEMP% (кириллица в самом deletePath); --update-snapshots молча не сносил старый эталон, то есть портил коммитимые артефакты. Реализация — единый fsutil в двух побайтно одинаковых копиях (tests/common/ и внутри автономного навыка web-test), раскатанный на все 42 точки вызова в 11 файлах: tests/skills/*, tests/web-test/*, hooks/test/run.mjs, движок web-test. Предикат судит по resolve(p), а не по строке аргумента: относительный ASCII-аргумент при не-ASCII cwd платформа роняет так же молча. Ретраи доживают до ручного обхода, перезапись как у cpSync с force: true, настоящие ошибки не глотаются, выживший после удаления путь — громкая ошибка. Гард tests/skills/check-nonascii-fs.mjs в check-all.mjs проверяет хелпер, а не платформу (поэтому зелёный и на исправной Node), сверяет хеши обеих копий и печатает справкой состояние текущей сборки. Co-Authored-By: androman.pro <5669019+andromanpro@users.noreply.github.com> Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
133 lines
5.8 KiB
JavaScript
133 lines
5.8 KiB
JavaScript
// 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, mkdirSync } from 'fs';
|
|
import { resolve, dirname } from 'path';
|
|
import { fileURLToPath } from 'url';
|
|
import { tmpdir } from 'os';
|
|
// fs.rmSync/fs.cpSync are never called directly: on Windows they silently do nothing when
|
|
// the path argument contains non-ASCII characters. Build matrix and details live in the module.
|
|
import { removePathSync } from '../../common/fsutil.mjs';
|
|
|
|
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 { removePathSync(reportDir); } catch {}
|
|
console.log(`OK: ${checks.length}/${checks.length} — таймаут прерывает зависший тест, прогон продолжается, отчёт пишется`);
|
|
process.exit(0);
|