Skip to content

update POM with pitest - #26

Merged
VonAdamo merged 2 commits into
mainfrom
feature/pom-pitest
Feb 10, 2026
Merged

update POM with pitest#26
VonAdamo merged 2 commits into
mainfrom
feature/pom-pitest

Conversation

@kristinaxm

@kristinaxm kristinaxm commented Feb 10, 2026

Copy link
Copy Markdown

Summary by CodeRabbit

  • Chores
    • Added a mutation testing tool to the development build, including JUnit 5 support and configuration to target the project's test and class packages for quality validation. This change affects build-time test analysis only and does not alter runtime behavior.

@kristinaxm kristinaxm self-assigned this Feb 10, 2026
@coderabbitai

coderabbitai Bot commented Feb 10, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Adds the PITest Maven plugin (org.pitest:pitest-maven:1.22.0) to the project's POM with the pitest-junit5-plugin dependency and configuration targeting classes and tests in the org.juv25d.* package.

Changes

Cohort / File(s) Summary
Build configuration
pom.xml
Added org.pitest:pitest-maven:1.22.0 plugin and org.pitest:pitest-junit5-plugin:1.2.2 dependency; configured targetClasses and targetTests to org.juv25d.*.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related issues

Suggested reviewers

  • LinusWestling

Poem

🐰
I nibble code and count each test,
Pitest now prowls and does its best,
Mutations hop, then I inspect,
Org.juv25d—I'll keep you checked! ✨

🚥 Pre-merge checks | ✅ 3
✅ 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 summarizes the main change: adding pitest configuration to the project POM file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/pom-pitest

No actionable comments were generated in the recent review. 🎉


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

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@pom.xml`:
- Around line 138-150: The pitest-maven plugin configuration is missing the
pitest-junit5-plugin so PITest won't discover JUnit Jupiter tests; inside the
existing <plugin> for org.pitest:pitest-maven (the block containing
<targetClasses> and <targetTests>), add a <pluginDependencies> entry referencing
the pitest JUnit 5 adapter (groupId org.pitest, artifactId pitest-junit5-plugin)
with a version compatible with the pitest-maven version (ensure the
pluginDependency is nested under the pitest-maven <plugin> element) so mutation
testing can run JUnit 5 tests.

Comment thread pom.xml
@bamsemats
bamsemats self-requested a review February 10, 2026 11:58
@bamsemats

Copy link
Copy Markdown

Since it is such a small update, I would suggest just adding the dependency provided by CodeRabbitAI and then it should be ready for approval.

@VonAdamo VonAdamo left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's go ✔🚀🚀

@VonAdamo
VonAdamo merged commit 68b5341 into main Feb 10, 2026
2 checks passed
@kristinaxm
kristinaxm deleted the feature/pom-pitest branch February 10, 2026 12:56
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.

3 participants