Restore mutation testing#4276
Open
scordio wants to merge 1 commit into
Open
Conversation
9b23161 to
23b4b91
Compare
This comment was marked as outdated.
This comment was marked as outdated.
0a00688 to
b957187
Compare
Closed
b957187 to
5def91b
Compare
5ad4a5a to
78ec6f4
Compare
scordio
commented
Jun 5, 2026
78c05d9 to
51fba2f
Compare
Member
Author
|
The previous comment proves that the changes in this PR, combined with |
There was a problem hiding this comment.
Pull request overview
This PR restores mutation testing support in the AssertJ build by introducing Arcmutate’s PIT/GitHub integration and adding a dedicated GitHub Actions workflow to run PIT on selected integration-test modules.
Changes:
- Add Arcmutate PIT plugin versions and plugin management to the root Maven build.
- Configure PIT cross-module mode in the integration-test modules used by the workflow.
- Add a new
pitest-run.ymlworkflow to run mutation testing and upload aggregated reports as an artifact.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
pom.xml |
Adds Arcmutate/PIT plugin management and PIT configuration used by modules/workflow. |
assertj-tests/assertj-integration-tests/assertj-guava-tests/pom.xml |
Enables PIT cross-module mode for guava integration tests. |
assertj-tests/assertj-integration-tests/assertj-core-tests/pom.xml |
Enables PIT cross-module mode for core integration tests. |
assertj-guava/pom.xml |
Reorders enforcer/jacoco plugin declarations (no functional PIT change). |
assertj-core/pom.xml |
Removes the old module-local PIT plugin configuration block. |
.github/workflows/pitest-run.yml |
Adds a PR workflow to run PIT + aggregate results + upload artifact. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
7cdf11a to
5056f37
Compare
5056f37 to
3c5a34b
Compare
3c5a34b to
d928017
Compare
|
Looks good. No mutations were possible for these changes. |
✅ All tests passed ✅🏷️ Commit: d928017 Learn more about TestLens at testlens.app. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This is conceptually equivalent to #4275, but we want an incremental set of changes to properly verify the GitHub integration, and that must be done from a branch within the repo to avoid adopting the two-way steps from the beginning.