proxysql

Commit Graph

Author	SHA1	Message	Date
René Cannaò	666eb4a75c	Address review feedback on rule matching unit tests (PR #5503 ) - Add test case for null session username: verifies that a rule with a username filter correctly rejects when session username is nullptr (+1 test, plan updated to 23)	3 months ago
René Cannaò	aae1ad4019	Fix rule matching extraction: proper unit tests with harness Fixes the Copilot-generated PR #5501 test file: - Remove misplaced test from test/tap/tests/ (wouldn't link against libproxysql.a with the default TAP build rules) - Add proper unit test at test/tap/tests/unit/rule_matching_unit-t.cpp using the test harness (test_globals.h, test_init.h) - 22 test cases covering all matching criteria: flagIN, username, schemaname, client_addr wildcard, proxy_addr/port, digest, match_digest (RE2 + PCRE), match_pattern, negate_match_pattern, CASELESS modifier, rewritten query, combined AND logic, null rule - Register in unit test Makefile Production code (rule_matches_query extraction) cherry-picked from Copilot's original commit — no changes to the extraction itself.	3 months ago

Author

SHA1

Message

Date

René Cannaò

666eb4a75c

Address review feedback on rule matching unit tests (PR #5503 )

- Add test case for null session username: verifies that a rule with
  a username filter correctly rejects when session username is nullptr
  (+1 test, plan updated to 23)

René Cannaò

aae1ad4019

Fix rule matching extraction: proper unit tests with harness

Fixes the Copilot-generated PR #5501 test file:

- Remove misplaced test from test/tap/tests/ (wouldn't link against
  libproxysql.a with the default TAP build rules)
- Add proper unit test at test/tap/tests/unit/rule_matching_unit-t.cpp
  using the test harness (test_globals.h, test_init.h)
- 22 test cases covering all matching criteria: flagIN, username,
  schemaname, client_addr wildcard, proxy_addr/port, digest,
  match_digest (RE2 + PCRE), match_pattern, negate_match_pattern,
  CASELESS modifier, rewritten query, combined AND logic, null rule
- Register in unit test Makefile

Production code (rule_matches_query extraction) cherry-picked from
Copilot's original commit — no changes to the extraction itself.

2 Commits (feature/pgsql-native-backend-protocol)