Skip to content

test: cover query log failure paths#187

Open
TheWaste11202 wants to merge 1 commit into
outerbase:mainfrom
TheWaste11202:codex/starbasedb-query-log-failure-tests
Open

test: cover query log failure paths#187
TheWaste11202 wants to merge 1 commit into
outerbase:mainfrom
TheWaste11202:codex/starbasedb-query-log-failure-tests

Conversation

@TheWaste11202
Copy link
Copy Markdown

@TheWaste11202 TheWaste11202 commented May 12, 2026

Summary

  • adds focused Vitest coverage for QueryLogPlugin failure paths
  • verifies addQuery swallows insert failures and returns []
  • verifies expireLog returns false and logs when deletion fails

Related issue

Part of #71

Verification

  • pnpm vitest run plugins/query-log/index.test.ts
  • pnpm exec prettier --check plugins/query-log/index.test.ts
  • git diff --check

Notes

  • pnpm test run currently has existing failures in src/rls/index.test.ts on main (4 RLS policy assertion failures). The query-log test file itself passes locally: 10/10.

/claim #71

Signed-off-by: z1551778201-ctrl <Nurtai1120@outlook.com>
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