Repository navigation
Conversation
…table matching - Fix schema-qualified policy table matching and alias support in src/rls/index.ts - Repair existing RLS test suite and add edge cases in src/rls/index.test.ts - Add unit test suite for allowlist module in src/allowlist/index.test.ts - Add unit test suite for CSV import route in src/import/csv.test.ts - Add unit test suite for worker fetch handler and middleware in src/index.test.ts - Expand test coverage in src/operation.test.ts, src/handler.test.ts, and src/do.test.ts - Add pnpm-workspace.yaml to permit required build scripts under pnpm 10/12 - Exceed 75% Vitest threshold across all metrics (lines 94.49%, stmts 94.13%, funcs 93.98%, branches 79.60%) Fixes outerbase#71
Saik2789
force-pushed
the
feat/test-coverage-vitest-71
branch
from
October 25, 2026 05:38
c24d62d to
8dfd5a3
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
Resolves #71: resolves the 4 pre-existing failing RLS tests and brings Vitest test coverage across all metrics (lines, statements, branches, and functions) well above the 75% threshold with meaningful unit and integration tests.
Changes
schema.tablevstable), table aliasing in queries withJOIN ... AS, and traversal of subqueries insrc/rls/index.ts.src/rls/index.test.ts.src/allowlist/index.test.ts.src/import/csv.tscovering unsupported content types, empty payloads, raw CSV, JSON-wrapped CSV column mapping, multipart parsing, and partial row failure reporting.src/index.tscovering CORS preflight, JWT verification, Durable Object selection, client/admin route authorization, interface route handling, and top-level errors.src/operation.test.ts,src/handler.test.ts, andsrc/do.test.tsto exercise raw query returns, Hyperdrive fallbacks, transaction failures, and feature toggles.pnpm-workspace.yamlallowing builds foresbuildandworkerdunder modern pnpm.Verification
Ran
pnpm test -- --runandpnpm exec vitest run --coverage:Prettier check and
git diff --checkboth pass cleanly.Bounty Claim
@algora-pbc /claim #71
Payout Information
0x20A0525b47f087f1Cc498cda257A0feB78D2320B