mirror of
https://github.com/Matysh/houseplan-card
synced 2026-09-29 03:09:36 +00:00
Диапазон классификации брался от `github.event.before` — головы предыдущего пуша. Это допущение «до этого уже проверено», и оно неверно ровно тогда, когда прогон предыдущего пуша не завершился. А не завершается он штатно: concurrency отменяет его следующим пушем. На #86 (r5) это дало ложный зелёный: push04da7eb1тронул dist/** и frontend/**, его прогон отменили через три минуты; следующий pushfa146fb1тронул только docs/images/**, классификация сравнила эти два коммита и выставила frontend=false. Job «Фронтенд», а за ней golden, smoke и backend оказались skipped — прогон при этом success. Маркеры переиспользования эти гейты тоже не подтверждали: `Cache not found` по всем четырём. Теперь база — самый новый предок HEAD, для которого Validate ДЕЙСТВИТЕЛЬНО завершился успешно; если такого нет, диапазон расширяется до merge-base с dev, то есть до всего вклада ветки. Работает по индукции: цепочка узких диффов покрывает всё изменённое с последней настоящей проверки, а одно незавершённое звено теперь расширяет диапазон, а не сужает. Недоступность API не роняет job: пустой ответ опускает базу до merge-base, то есть в сторону большего объёма проверок. Защита от force-push (#347) сохранена: механизм, из-за которого merge-base врал на переписанной истории, до конца не разобран, и снимать защиту, не объяснив её, — способ получить #347 второй раз. Issue: #387 User-Visible: no
117 lines
5.3 KiB
JavaScript
117 lines
5.3 KiB
JavaScript
import test from 'node:test';
|
|
import assert from 'node:assert/strict';
|
|
import { execFileSync, spawnSync } from 'node:child_process';
|
|
import { mkdtempSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
|
|
import { tmpdir } from 'node:os';
|
|
import { join } from 'node:path';
|
|
import { fileURLToPath } from 'node:url';
|
|
|
|
import { MAX_CANDIDATES, baseSummary, greenShas, pickBase } from '../scripts/classify-base.mjs';
|
|
|
|
const SCRIPT = fileURLToPath(new URL('../scripts/classify-base.mjs', import.meta.url));
|
|
|
|
test('зелёными считаются только успешно завершённые прогоны (#387)', () => {
|
|
const green = greenShas({
|
|
workflow_runs: [
|
|
{ head_sha: 'aaa', conclusion: 'success' },
|
|
// Ровно тот случай, из-за которого заведён #387: прогон предыдущего
|
|
// пуша отменён следующим пушем.
|
|
{ head_sha: 'bbb', conclusion: 'cancelled' },
|
|
{ head_sha: 'ccc', conclusion: 'failure' },
|
|
{ head_sha: 'ddd', conclusion: null, status: 'in_progress' },
|
|
{ head_sha: 'eee', conclusion: 'skipped' },
|
|
],
|
|
});
|
|
assert.deepEqual([...green], ['aaa']);
|
|
});
|
|
|
|
test('недоступный или битый ответ API даёт пустой список, а не падение (#387)', () => {
|
|
for (const payload of [null, undefined, {}, { workflow_runs: null }, 'мусор', 42]) {
|
|
assert.equal(greenShas(payload).size, 0, `${JSON.stringify(payload)}`);
|
|
}
|
|
});
|
|
|
|
test('база — самый новый зелёный предок (#387)', () => {
|
|
const choice = pickBase({
|
|
candidates: ['c3', 'c2', 'c1'],
|
|
green: new Set(['c2', 'c1']),
|
|
mergeBase: 'mb',
|
|
});
|
|
assert.equal(choice.base, 'c2', 'из двух зелёных берётся более новый');
|
|
assert.equal(choice.proven, true);
|
|
assert.equal(choice.skipped, 1);
|
|
});
|
|
|
|
test('цепочка отменённых прогонов расширяет диапазон до merge-base (#387)', () => {
|
|
// Сценарий #86 r5: у предыдущего пуша прогон cancelled, зелёных предков нет.
|
|
const choice = pickBase({ candidates: ['fa146fb1', '04da7eb1'], green: new Set(), mergeBase: 'mb' });
|
|
assert.equal(choice.base, 'mb');
|
|
assert.equal(choice.proven, false);
|
|
assert.match(baseSummary(choice, { head: 'head1234', mergeBase: 'mb345678' }).join('\n'),
|
|
/расширен до/);
|
|
});
|
|
|
|
test('ветка без собственных коммитов не ломает выбор (#387)', () => {
|
|
const choice = pickBase({ candidates: [], green: new Set(['x']), mergeBase: 'mb' });
|
|
assert.equal(choice.base, 'mb');
|
|
assert.equal(choice.skipped, 0);
|
|
});
|
|
|
|
test('обход кандидатов ограничен сверху (#387)', () => {
|
|
const many = Array.from({ length: MAX_CANDIDATES + 5 }, (_, i) => `c${i}`);
|
|
// Зелёный есть, но лежит за пределом обхода: ответ обязан быть безопасным,
|
|
// то есть более широким, а не «ничего не нашли — берём последний».
|
|
const choice = pickBase({ candidates: many, green: new Set([`c${MAX_CANDIDATES + 2}`]), mergeBase: 'mb' });
|
|
assert.equal(choice.base, 'mb');
|
|
});
|
|
|
|
test('CLI считает базу по настоящей истории git (#387)', () => {
|
|
const dir = mkdtempSync(join(tmpdir(), 'hp-classify-'));
|
|
const git = (...args) => execFileSync('git', args, { cwd: dir, encoding: 'utf8' }).trim();
|
|
try {
|
|
git('init', '-q', '-b', 'main');
|
|
git('config', 'user.email', 'test@example.com');
|
|
git('config', 'user.name', 'test');
|
|
const commit = (text) => {
|
|
writeFileSync(join(dir, 'file.txt'), text);
|
|
git('add', '-A');
|
|
git('commit', '-qm', text);
|
|
return git('rev-parse', 'HEAD');
|
|
};
|
|
const root = commit('root');
|
|
const green = commit('проверенный');
|
|
const cancelled = commit('прогон отменён');
|
|
const head = commit('текущий пуш');
|
|
|
|
const runsFile = join(dir, 'runs.json');
|
|
const out = join(dir, 'out.txt');
|
|
const run = (runs) => {
|
|
writeFileSync(runsFile, JSON.stringify(runs));
|
|
writeFileSync(out, '');
|
|
const result = spawnSync(process.execPath, [
|
|
SCRIPT, `--head=${head}`, `--merge-base=${root}`, `--runs=${runsFile}`,
|
|
], { cwd: dir, encoding: 'utf8', env: { ...process.env, GITHUB_OUTPUT: out } });
|
|
assert.equal(result.status, 0, result.stderr);
|
|
return { stdout: result.stdout, output: readFileSync(out, 'utf8') };
|
|
};
|
|
|
|
const withGreen = run({
|
|
workflow_runs: [
|
|
{ head_sha: green, conclusion: 'success' },
|
|
{ head_sha: cancelled, conclusion: 'cancelled' },
|
|
// HEAD в списке зелёных быть не может, но даже если API его вернёт,
|
|
// сам себя коммит проверенным не объявляет.
|
|
{ head_sha: head, conclusion: 'success' },
|
|
],
|
|
});
|
|
assert.match(withGreen.output, new RegExp(`base=${green}\\n`));
|
|
assert.match(withGreen.output, /proven=true/);
|
|
|
|
const noGreen = run({ workflow_runs: [{ head_sha: cancelled, conclusion: 'cancelled' }] });
|
|
assert.match(noGreen.output, new RegExp(`base=${root}\\n`));
|
|
assert.match(noGreen.output, /proven=false/);
|
|
} finally {
|
|
rmSync(dir, { recursive: true, force: true });
|
|
}
|
|
});
|