From d8bc1bc63627a870c95be7b3024641446cf1bc90 Mon Sep 17 00:00:00 2001 From: Tom Boucher Date: Mon, 29 Jun 2026 17:10:55 -0400 Subject: [PATCH] fix(#1831): add state-rebuild test files to lint-test-file-count allowlist MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PRs #1829 and #1830 added tests/state-rebuild.test.cjs and tests/state-rebuild-cli.test.cjs but did not add them to the lint-test-file-count allowlist for the 'state' module. The lint-test-file-count.test.cjs harness reported FAIL_NOVEL_FILES with the two files listed as novel. Verified via gsd-test (--base origin/main --head origin/next): without this fix, 2/22097 tests fail (both lint-test-file-count allowlist cases). The state module has 17 test files; allowlist entry now lists all 17 alphabetically. Process note: the original PRs should have updated this allowlist in the same commit that added the test files. Recapture of the golden-install-parity fixtures is NOT required — those fixtures on next are correct (verified: no diff from UPDATE_GOLDEN=1 locally). --- scripts/lint-test-file-count.allowlist.json | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/scripts/lint-test-file-count.allowlist.json b/scripts/lint-test-file-count.allowlist.json index 6758bb796..da4d884af 100644 --- a/scripts/lint-test-file-count.allowlist.json +++ b/scripts/lint-test-file-count.allowlist.json @@ -110,10 +110,12 @@ "bug-3454-state-dollar-backreference-growth.test.cjs", "bug-397-state-preserve-executor-authored.test.cjs", "bug-905-state-syncstatefrontmatter-preserve-scalars.test.cjs", - "bug-948-state-noop-write-guard.test.cjs", - "state-acquirestatelock-non-eexist.test.cjs", - "state-prune.test.cjs", - "state.test.cjs" + "bug-948-state-noop-write-guard.test.cjs", + "state-acquirestatelock-non-eexist.test.cjs", + "state-prune.test.cjs", + "state-rebuild-cli.test.cjs", + "state-rebuild.test.cjs", + "state.test.cjs" ], "issue": "180" },