Skip to content

test: cover sql macros plugin#281

Open
wangxiangzhu88-droid wants to merge 1 commit into
outerbase:mainfrom
wangxiangzhu88-droid:codex/outerbase-sql-macros-tests
Open

test: cover sql macros plugin#281
wangxiangzhu88-droid wants to merge 1 commit into
outerbase:mainfrom
wangxiangzhu88-droid:codex/outerbase-sql-macros-tests

Conversation

@wangxiangzhu88-droid

Copy link
Copy Markdown

/claim #71

Summary

  • Add focused coverage for the SQL macros plugin's no-data-source, internal $_exclude(...), external data source, non-admin SELECT *, and admin SELECT * paths.
  • Fix checkSelectStar to handle parser results returned as either a single AST object or an array, so preventSelectStar rejects single-statement SELECT * queries as intended.

Tests

  • vitest run plugins/sql-macros/index.test.ts
  • prettier --check plugins/sql-macros/index.ts plugins/sql-macros/index.test.ts
  • vitest run currently reports existing failures in src/rls/index.test.ts unrelated to this change; the new SQL macros tests pass.

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.

1 participant