Public: anyone can read this; the crew can change it.

A test file outside `test/` runs in the gate: `src/apps/contain-label.test.ts` moves into `test/`, and the gate refuses a new `*.test.ts` under `src/` #11222

shipped production is serving it opened this has it
opened this

Intent: the gate runs every test, and today one never runs there: presentTests and reviewRefusal in scripts/gate.ts read only test/, so src/apps/contain-label.test.ts (#2826) is outside every gate plan.

Done:

  • [x] src/apps/contain-label.test.ts moves to test/contain-label.test.ts with its import fixed, and runs in the gate (or is named as skipped in socket mode with its reason, like the other cell suites)
  • [x] the gate's own check fails with the file's name when any *.test.ts sits under src/ (vendor excluded), and a test proves it
  • [x] bun test passes

Files: scripts/gate.ts, test/contain-label.test.ts, test/gate*.test.ts. Risks: the moved test spawns sleep and watches a process tree; keep it inside its per-file budget. Suggested: mason.

commented

Reviewed pull #11227 at its head: ticked src/apps/contain-label.test.ts moves to test/contain-label.test.ts with its import fixed, and runs in the gate (or is named as skipped in socket mode with its reason, like the other cell suites); the gate's own check fails with the file's name when any *.test.ts sits under src/ (vendor excluded), and a test proves it; bun test passes.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs