adutra opened a new pull request, #3780:
URL: https://github.com/apache/polaris/pull/3780
This PR refactors the authorization tests to use JUnit 5's dynamic test
feature (`@TestFactory`).
**There is no functional change, just a refactoring.**
* Better test organization: tests are now organized into hierarchies,
grouping related privilege tests together.
* Clearer test naming: Each dynamic test has a descriptive name that
includes:
* The operation being tested (e.g., `listNamespaces`, `createCatalog`)
* The privilege(s) being tested
* Whether the test expects success or failure
* Improved test output: the dynamic test structure provides better
visibility into which specific privilege combinations are being tested and
which ones pass or fail. This greatly facilitates debugging failing tests.
* Simplified test methods: the base class `PolarisAuthzTestBase` now
provides many overloaded methods with sensible defaults. Child classes don't
need to declare such methods anymore.
<!--
๐ Describe what changes you're proposing, especially breaking or user-facing
changes.
๐ See https://github.com/apache/polaris/blob/main/CONTRIBUTING.md for more.
-->
## Checklist
- [ ] ๐ก๏ธ Don't disclose security issues! (contact [email protected])
- [ ] ๐ Clearly explained why the changes are needed, or linked related
issues: Fixes #
- [ ] ๐งช Added/updated tests with good coverage, or manually tested (and
explained how)
- [ ] ๐ก Added comments for complex logic
- [ ] ๐งพ Updated `CHANGELOG.md` (if needed)
- [ ] ๐ Updated documentation in `site/content/in-dev/unreleased` (if needed)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
