Skip to content

test(autofix): single-source the infra-signature list from the workflow #7565

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wenshao merged 7 commits into from
Jul 23, 2026
Merged

test(autofix): single-source the infra-signature list from the workflow #7565

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
213f3e9
feat(autofix): auto-rerun a check that died on infrastructure, once
Jul 23, 2026
3ffac12
fix(autofix): use POSIX ERE groups in infra-failure regex, cover all …
Jul 23, 2026
3cd2195
fix(autofix): also treat a git fetch/clone transport death as infra
Jul 23, 2026
6fd7557
test(autofix): single-source the infra-signature list from the workflow
Jul 23, 2026
59a079f
fix(autofix): paginate annotations and filter Autofix runs in infra-r…
qwen-code-ci-bot Jul 23, 2026
e9fe3b5
Merge branch 'ci/autofix-rerun-infra-flake' into ci/autofix-infra-sig…
Jul 23, 2026
6aea34c
Merge branch 'main' into ci/autofix-infra-signature-single-source
Jul 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 10 additions & 2 deletions scripts/tests/qwen-autofix-workflow.test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -413,6 +413,15 @@ describe('qwen-autofix workflow', () => {
expect(block).toBeTruthy();
const script = block.replace(/^ {12}/gm, '');

// Single-source the signature list from the workflow rather than re-typing
// it here, so the production value and this test can never drift out of
// sync (same extract-from-source idiom as NON_BLOCKING_CHECKS above). The
// toContain guard fails loudly if the env is renamed or the regex breaks —
// otherwise an empty pattern would match every line and silently pass.
const INFRA_SIGNATURES =
workflow.match(/INFRA_FAILURE_SIGNATURES: '([^']*)'/)?.[1] ?? '';
expect(INFRA_SIGNATURES).toContain('lost communication with the server');

const run = ({
checks,
annotations,
Expand Down Expand Up @@ -466,8 +475,7 @@ describe('qwen-autofix workflow', () => {
PR: '1',
PR_META: JSON.stringify({ headRefOid: 'headSHA' }),
CHECKS_JSON: JSON.stringify(checks),
INFRA_FAILURE_SIGNATURES:
'lost communication with the server|No space left on device|ENOSPC|received a shutdown signal|The runner has received|Failed to initialize container|runner (was|has been) (lost|terminated)|invalid index-pack output|RPC failed',
INFRA_FAILURE_SIGNATURES: INFRA_SIGNATURES,
PATH: `${bin}:${process.env.PATH}`,
},
encoding: 'utf8',
Expand Down