Skip to content

Conversation

RakeshGuptaDev
Copy link
Contributor

@RakeshGuptaDev RakeshGuptaDev commented Sep 24, 2025

What changes were proposed in this pull request?

AI Disclosure

Added AI-generated Test methods/Test classes using Cursor AI for Common library for Plugins module and manually reviewed.

Added unit test cases for agents-common module

How was this patch tested?

Build completed successfully along with all test cases using the command: mvn clean compile package install.

As per Jacoco
Apache -> Common library for Plugins

Packages Covered

Improved Jacoco coverage across multiple packages:
org.apache.ranger.services.gds → 0% → 100%
org.apache.ranger.services.tag → 0% → 99%
org.apache.ranger.admin.client.datatype→ 0% → 84%
org.apache.hadoop.security → 0% → 96%
org.apache.ranger.plugin.client → 0% → 92%
org.apache.ranger.plugin.contextenricher.externalretrievers → 0% → 88%
org.apache.ranger.plugin.store → 1% → 85%
org.apache.ranger.admin.client → 7% → 84%
org.apache.ranger.plugin.service → 32% → 75%
org.apache.ranger.plugin.model → 36% → 77%
org.apache.ranger.plugin.util → 40% → 75%
org.apache.ranger.authorization.utils → 41% → 93%
org.apache.ranger.authorization.hadoop.config → 49% → 89%
org.apache.ranger.plugin.audit → 63% → 98%
org.apache.ranger.plugin.conditionevaluator → 59% → 90%
org.apache.ranger.plugin.contextenricher → 48% → 78%
org.apache.ranger.plugin.geo → 55% → 96%

@kumaab
Copy link
Contributor

kumaab commented Sep 29, 2025

Can the review be split across multiple PRs, this PR introduces a lot of new lines to be reviewed at once.

@RakeshGuptaDev
Copy link
Contributor Author

Created subtask JIRAs to split the review across multiple PRs.

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.

2 participants