Skip to content

test: exceed 75% Vitest coverage across all metrics and resolve RLS schema bug - #292

Open
jdjdjjdrjnsn wants to merge 2 commits into
outerbase:mainfrom
jdjdjjdrjnsn:test/coverage-allowlist-dump-plugin
Open

test: exceed 75% Vitest coverage across all metrics and resolve RLS schema bug#292
jdjdjjdrjnsn wants to merge 2 commits into
outerbase:mainfrom
jdjdjjdrjnsn:test/coverage-allowlist-dump-plugin

Conversation

@jdjdjjdrjnsn

@jdjdjjdrjnsn jdjdjjdrjnsn commented Sep 8, 2026

Copy link
Copy Markdown

Resolves #71
/claim #71

Summary of Changes

Achieves >75% test coverage across all four Istanbul metrics on the entire repository and resolves the schema-matching issue in the Row Level Security (RLS) plugin:

1. Test Coverage Metrics (All thresholds met)

  • Statements: 96.71% (Required: 75%)
  • Branches: 82.43% (Required: 75%)
  • Functions: 97.84% (Required: 75%)
  • Lines: 96.97% (Required: 75%)
  • Test Results: 25/25 test files passing, 284/284 tests passing.

2. Suites Added & Expanded

  • ****: Complete suite covering admin bypass, disabled mode, source table filtering, and rejection logging.
  • ****: Covers SQLite header stripping, multiline comments, and formData parsing error paths.
  • ****: Covers plugin registry registration, retrieval, and UnimplementedError recovery.
  • ** & **: Comprehensive CSV import route coverage and quoted CSV field splitting.
  • ****: Comprehensive coverage for Durable Object transactions, alarm rescheduling, and error propagation.
  • ****: Covers transaction validation, plugin pre-auth routing, and error handlers.
  • ****: Covers data source fallbacks, Hyperdrive query errors, and external connections.
  • ****: Covers top-level Worker export, CORS preflight, and request dispatching.
  • ** & **: Fixes schema-aware table matching so that schema-qualified policy definitions properly match unqualified table queries and subqueries.

All tests run cleanly with zero regressions on .

@jdjdjjdrjnsn jdjdjjdrjnsn changed the title test: add Vitest suites for allowlist, dump header/errors, and plugin registry test: exceed 75% Vitest coverage across all metrics and resolve RLS schema bug Sep 8, 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