Skip to content

test: bound exhaustive credential invariant timeout - #12

Merged
andrei-hasna merged 1 commit into
mainfrom
fix/shi-00003-credential-invariant-timeout
Aug 7, 2026
Merged

test: bound exhaustive credential invariant timeout#12
andrei-hasna merged 1 commit into
mainfrom
fix/shi-00003-credential-invariant-timeout

Conversation

@andrei-hasna

@andrei-hasna andrei-hasna commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Summary

  • give the exhaustive database credential invariant an explicit 30-second per-test timeout
  • keep every other Shield test, including the safe-noncredential control, on Bun's default timeout

Evidence

  • RED before the change: 9 pass, 1 fail, 6000 expect() calls; exhaustive invariant timed out after 5000ms
  • focused file: 10 pass, 0 fail, 6000 expect() calls
  • exhaustive invariant only: 1 pass, 0 fail, 864 expect() calls
  • safe noncredential control only: 1 pass, 0 fail, 28 expect() calls
  • full bun test --parallel=1: 439 pass, 0 fail, 7038 expect() calls
  • bun run typecheck: pass
  • strict OSS secrets policy: Violations: 0
  • staged and commit-range gitleaks: no leaks found
  • independent adversarial review: [REVIEW] GO, no in-scope P0/P1 findings

Todos: SHI-00003


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@andrei-hasna
andrei-hasna marked this pull request as ready for review August 7, 2026 21:05
@andrei-hasna
andrei-hasna merged commit 3724586 into main Aug 7, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant