Skip to content

test: add comprehensive Vitest unit tests for allowlist module (#71) - #303

Closed
stand-out-of-my-sun wants to merge 1 commit into
outerbase:mainfrom
stand-out-of-my-sun:test-allowlist-71-1792228533
Closed

stand-out-of-my-sun wants to merge 1 commit into
outerbase:mainfrom
stand-out-of-my-sun:test-allowlist-71-1792228533

Conversation

@stand-out-of-my-sun

@stand-out-of-my-sun stand-out-of-my-sun commented Oct 17, 2026 •

Copy link
Copy Markdown

Resolves #71

/claim #71

Summary

  • Adds exhaustive unit test coverage for src/allowlist/index.ts using Vitest.
  • Tests isEnabled: false default passthrough.
  • Tests role: 'admin' administrative bypass.
  • Tests empty/null SQL validation.
  • Tests normalized SQL statement and AST comparisons.
  • Tests unlisted query rejection and tmp_allowlist_rejections audit logging.
  • Tests database RPC error resiliency.

@stand-out-of-my-sun stand-out-of-my-sun closed this by deleting the head repository Oct 18, 2026
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.

Improve test coverage with Vitest

1 participant