starred/openclaude
1
0
Fork 0
mirror of https://github.com/Gitlawb/openclaude.git synced 2026-11-06 20:58:14 -04:00
Code Issues Packages Projects Releases Wiki Activity

test(provider-flags): exclude all test files from the consumed-flag scan

Browse Source 
CodeRabbit follow-up: the "registered flags are actually consumed" check only
skipped the registry and its own test, so a flag referenced solely by another
test fixture would still look consumed. Skip every *.test.* file so the
assertion proves real runtime consumption.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
gitdevine
2026-06-26 11:37:39 +02:00
co-authored by Claude Fable 5
parent 0ec44b310f
commit ef5ae3a359
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/utils/providerSelectionFlags.test.ts
+4 -4
View File
@@ -49,12 +49,12 @@ describe('PROVIDER_SELECTION_FLAGS parity', () => {
const glob = new Bun.Glob('**/*.{ts,tsx}')
let allContent = ''
for (const relPath of glob.scanSync({ cwd: srcRoot })) {
// Exclude the registry itself (and its test) so the assertion proves the
// flag is referenced by real runtime code, not just by its own
// definition — otherwise the check is tautological.
// Exclude the registry itself and every test file so the assertion proves
// the flag is referenced by real runtime code, not just by its own
// definition or by a fixture — otherwise the check is tautological.
if (
relPath.endsWith('providerSelectionFlags.ts') ||
relPath.endsWith('providerSelectionFlags.test.ts')
/\.test\.[tj]sx?$/.test(relPath)
) {
continue
}