mirror of
https://github.com/garrytan/gstack.git
synced 2026-10-31 14:06:50 +01:00
test(host-conformance): delete a named section in the C3 broken-link case
The case removed the first .md readdirSync returned under gstack-qa/sections. qa-patterns.md and system-functional.md are referenced only in table form, which setup --status does not count as section links, so on an ext4 directory order that listed one of them first the deleted file broke nothing and --status exited 0: deterministic red on remote-gate VMs, green on Actions. Delete scope.md, which every host's rendered SKILL.md reads. Closes #3479 Co-authored-by: capy-ai[bot] <230910855+capy-ai[bot]@users.noreply.github.com>
This commit is contained in:
1 parent
cacc6003ea
commit
fd468ebd09
2 files changed
+2
-2
No files matched your search
@@ -11,6 +11,7 @@
|
||||
|
||||
### Tests
|
||||
- `test/factory-scorecard.test.ts`: the recorded window (`test/fixtures/factory-scorecard/recorded.json`) yields the expected merges, lead times, gate kinds and outcomes, artifact and comment cost sources, flake counts, free retries, issues and releases; a crew login list overrides the association rule; runs older than the window, non-dispatch events and draft releases are left out; the artifact cap leaves later runs without cost or flake data; a missing input marks only its lines unavailable; the markdown and CSV carry every row; gate classification, READY-marker matching, verdict-cost and flake-record parsing and nearest-rank percentiles; `--gate-event any` keeps non-dispatch runs; an empty window prints n/a instead of NaN; a failed merged-PR read leaves the gate rows standing; the bin end to end with `--fixtures`, `--crew`, `--out-dir`, `--csv`, `--ready-marker`, bad usage (including `--now`) and `--help`; the retro template pins the fence.
|
||||
- `test/host-conformance.test.ts` C3 ("--status names a symlinked router and a broken section link per host") deletes a named section (`scope.md`) instead of the first `.md` that `readdirSync` returns: two gstack-qa sections are referenced only in table form, which `setup --status` does not count as links, so on an ext4 directory order that listed one of them first the case failed deterministically on remote-gate VMs while passing on Actions (#3479).
|
||||
|
||||
## [1.91.246.0] - 2026-10-26
|
||||
|
||||
|
||||
@@ -202,8 +202,7 @@ describe.skipIf(process.platform === 'win32')('host conformance kit', () => {
|
||||
const router = join(f.home, '.codex/skills/gstack/SKILL.md');
|
||||
rmSync(router);
|
||||
symlinkSync(join(src, '.agents/skills/gstack/SKILL.md'), router);
|
||||
const qa = join(src, '.agents/skills/gstack-qa/sections');
|
||||
rmSync(join(qa, readdirSync(qa).find(n => n.endsWith('.md'))!));
|
||||
rmSync(join(src, '.agents/skills/gstack-qa/sections/scope.md'));
|
||||
const status = runSetup(f, join(src, 'setup'), ['--status']);
|
||||
// A1: broken section links make --status exit 1 instead of always 0.
|
||||
expect(status.status).toBe(1);
|
||||
|
||||
Reference in new issue
Block a user