Skip to content

test: expand operation coverage#153

Open
Iineman2 wants to merge 1 commit into
outerbase:mainfrom
Iineman2:codex/operation-coverage-71
Open

test: expand operation coverage#153
Iineman2 wants to merge 1 commit into
outerbase:mainfrom
Iineman2:codex/operation-coverage-71

Conversation

@Iineman2
Copy link
Copy Markdown

@Iineman2 Iineman2 commented May 12, 2026

/claim #71

Summary

  • Expand src/operation.test.ts with focused coverage for execution paths that were still mostly uncovered.
  • Cover raw result transformation through registry hooks, internal empty-result handling, Hyperdrive success/error handling, External API SDK fallback, and every supported SDK connection branch.
  • Keep this test-only and non-overlapping with the active CSV import, RLS, cache, and allowlist coverage PRs.

Validation

  • .\node_modules\.bin\prettier.cmd --check src/operation.test.ts - passed
  • .\node_modules\.bin\vitest.cmd run src/operation.test.ts --coverage.enabled true --coverage.include src/operation.ts --coverage.reporter text - passed; src/operation.ts reports 94.21% statements, 89.47% branches, 100% functions, 94.06% lines
  • .\node_modules\.bin\vitest.cmd run - operation tests pass; current main still has the existing 4 failing src/rls/index.test.ts expectations that are unrelated to this operation-only change
  • git diff --check - passed

Demo video

@Iineman2
Copy link
Copy Markdown
Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant