[MOSIP-44006] : Added surefire plugin and jacoco version

[chetankh239]

Summary by CodeRabbit

• Chores
  • Enhanced test execution infrastructure and added code coverage monitoring capabilities.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Added Maven Surefire plugin with test execution configuration and JaCoCo plugin for code coverage measurement. Surefire's argLine now includes JaCoCo integration, Java module opens (--add-opens), and --enable-preview flag. JaCoCo is configured with prepare-agent goal and report generation during verify phase.

Changes

Cohort / File(s)	Change Summary
Maven Test & Coverage Configuration kernel/kernel-ref-idobjectvalidator/pom.xml	Added maven-surefire-plugin with argLine configuration including jacocoArgLine, --add-opens module directives, and --enable-preview flag. Added jacoco-maven-plugin with prepare-agent and report goals for code coverage measurement during verify phase.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

• Review plugin configurations for correct syntax and appropriate JVM options
• Verify JaCoCo prepare-agent goal properly wires to jacocoArgLine property
• Confirm --add-opens module directives are necessary for the test suite

Possibly related PRs

• [MOSIP-44006] : Sonar coverage #1085: Expanded test suite implementation that prompted the addition of these Surefire and JaCoCo build configurations for test execution and coverage measurement.

Suggested reviewers

• JanardhanBS-SyncByte
• Prafulrakhade

Poem

🐰 A rabbit hops through Maven's domain,
Testing tools and coverage reign,
Surefire bounces, JaCoCo measures true,
Preview flags and module opens too!
Build it better, measure it all—
Quality rises, none shall fall! 📊✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the main change—adding surefire and jacoco plugin configurations to the pom.xml file for test execution and code coverage.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f2aade1 and c2e3c04.

📒 Files selected for processing (1)

• kernel/kernel-ref-idobjectvalidator/pom.xml (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)

• GitHub Check: build-maven-cache-provider-redis / maven-build
• GitHub Check: build-maven-pre-registration-booking-service / maven-build
• GitHub Check: build-maven-registration-processor-external-integration-service / maven-build
• GitHub Check: build-maven-registration-processor-external-stage / maven-build
• GitHub Check: build-maven-kernel / maven-build

🔇 Additional comments (2)

kernel/kernel-ref-idobjectvalidator/pom.xml (2)

109-115: Verify compatibility with preview features and module access.

The argLine configuration includes --enable-preview and multiple --add-opens flags for internal Java modules (java.xml, java.base). These flags can impact Java version compatibility and maintainability:

• --enable-preview requires the target Java version to explicitly support preview features
• --add-opens flags bypass Java module encapsulation, which may break across Java versions

Verify that:

1. The target Java version (defined in maven.compiler.source/target) is compatible with the preview features being used
2. The --add-opens flags are necessary for the actual application code (not just test dependencies)
3. These flags align with the project's Java version support policy

---

119-141: JaCoCo plugin version property is properly defined in parent pom.

The property ${maven.jacoco.plugin.version} is correctly inherited from the parent pom kernel-ref-parent (./kernel/pom.xml line 77), where it is set to 0.8.11. No action required.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}