Skip to content

Commit 198cf22

Browse files
authored
ci: isolate dropped-language-gap.test.ts as a regression canary (#1147) (#1169)
Add a dedicated step in the engine-parity job that runs only `tests/integration/dropped-language-gap.test.ts`. The broader `test` job already runs this file as part of `npm test`, but a failure there gets buried in 500+ test names. The standalone step surfaces a clear "Dropped-language gap regression guard" status on every PR and rules out test pollution from neighbouring files when the canary fires. The gap-repair path (#1083) is load-bearing for engine parity but silently breaks if `backfillNativeDroppedFiles` ever stops writing the file node row — exactly the shape #1147 reported. Refs #1147
1 parent 383cbbe commit 198cf22

1 file changed

Lines changed: 12 additions & 0 deletions

File tree

.github/workflows/ci.yml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -284,6 +284,18 @@ jobs:
284284
CODEGRAPH_PARITY: '1'
285285
run: npx vitest run tests/engines/ tests/integration/build-parity.test.ts --reporter=verbose
286286

287+
# Isolated regression canary for #1147 — runs only the dropped-language-gap
288+
# suite so a failure surfaces under a clear step name and can't be masked
289+
# by pollution from neighbouring tests in the larger `test` job. The gap
290+
# repair path (#1083) is load-bearing for engine parity but easy to break
291+
# silently — the bug shape is "node row never re-inserted", which only
292+
# this test exercises directly.
293+
- name: Dropped-language gap regression guard (#1147)
294+
shell: bash
295+
env:
296+
CODEGRAPH_PARITY: '1'
297+
run: npx vitest run tests/integration/dropped-language-gap.test.ts --reporter=verbose
298+
287299
rust-check:
288300
runs-on: ubuntu-latest
289301
name: Rust compile check

0 commit comments

Comments
 (0)