From e8903fa716ea082f11c75db2b96132232702828d Mon Sep 17 00:00:00 2001 From: Tibsfox Date: Mon, 6 Apr 2026 22:49:46 -0700 Subject: [PATCH] fix(test): remove extraneous test file from wrong branch The find-missing-summaries-closed.test.ts was accidentally committed to this branch but belongs to fix/find-missing-summaries-skip-closed. Remove it to fix CI failures. Co-Authored-By: Claude Opus 4.6 (1M context) --- .../find-missing-summaries-closed.test.ts | 48 ------------------- 1 file changed, 48 deletions(-) delete mode 100644 src/resources/extensions/gsd/tests/find-missing-summaries-closed.test.ts diff --git a/src/resources/extensions/gsd/tests/find-missing-summaries-closed.test.ts b/src/resources/extensions/gsd/tests/find-missing-summaries-closed.test.ts deleted file mode 100644 index a0d0d70b0..000000000 --- a/src/resources/extensions/gsd/tests/find-missing-summaries-closed.test.ts +++ /dev/null @@ -1,48 +0,0 @@ -/** - * Regression test for #3669 — findMissingSummaries skips closed slices - * - * When a slice has status "skipped", "complete", or "done", it should be - * excluded from the missing-summary check because closed slices intentionally - * lack SUMMARY files (or their DB status is authoritative). - * - * This is a structural verification test — it reads the source to confirm the - * CLOSED_STATUSES guard exists at the filter site. - */ - -import { describe, test } from 'node:test'; -import assert from 'node:assert/strict'; -import { readFileSync } from 'node:fs'; -import { fileURLToPath } from 'node:url'; -import { dirname, join } from 'node:path'; - -const __filename = fileURLToPath(import.meta.url); -const __dirname = dirname(__filename); - -const source = readFileSync(join(__dirname, '..', 'auto-dispatch.ts'), 'utf-8'); - -describe('findMissingSummaries closed-status exclusion (#3669)', () => { - test('CLOSED_STATUSES set includes skipped, complete, and done', () => { - // The source must define a CLOSED_STATUSES set with all three statuses - assert.match(source, /CLOSED_STATUSES.*=.*new Set\(/, - 'CLOSED_STATUSES set should be defined'); - assert.match(source, /"skipped"/, 'CLOSED_STATUSES should include "skipped"'); - assert.match(source, /"complete"/, 'CLOSED_STATUSES should include "complete"'); - assert.match(source, /"done"/, 'CLOSED_STATUSES should include "done"'); - }); - - test('filter uses CLOSED_STATUSES.has() to exclude closed slices', () => { - assert.match(source, /CLOSED_STATUSES\.has\(s\.status\)/, - 'filter should call CLOSED_STATUSES.has(s.status)'); - }); - - test('findMissingSummaries function exists', () => { - assert.match(source, /function findMissingSummaries\(/, - 'findMissingSummaries function should be defined'); - }); - - test('filter is negated (excludes closed, keeps open)', () => { - // The filter should use !CLOSED_STATUSES.has() to exclude closed slices - assert.match(source, /!CLOSED_STATUSES\.has\(s\.status\)/, - 'filter should negate CLOSED_STATUSES.has() to exclude closed slices'); - }); -});