Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve mutation testing setup #383

Merged
merged 2 commits into from
Dec 5, 2022
Merged

Improve mutation testing setup #383

merged 2 commits into from
Dec 5, 2022

Conversation

Stephan202
Copy link
Member

@Stephan202 Stephan202 commented Dec 3, 2022 •
edited

Suggested commit message:

Improve mutation testing setup (#383)

Summary of changes:
- Enable Pitest's built-in `STRONGER` mutator group.
- Enable Arcmutate's `EXTENDED` mutator group.
- Enable Arcmutate's JUnit 5 Accelerator Plugin.
- Modify `MoreTypesTest` such that it is impacted by mutations of the
  `MoreTypes` class.

See:
- https://pitest.org/quickstart/mutators/
- https://docs.arcmutate.com/docs/extended-operators.html
- https://docs.arcmutate.com/docs/accelerator.html

@Stephan202 Stephan202 added this to the 0.6.0 milestone Dec 3, 2022
@Stephan202 Stephan202 mentioned this pull request Dec 3, 2022
Merged
@github-actions
Copy link

github-actions bot commented Dec 3, 2022

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

1 similar comment
@github-actions
Copy link

github-actions bot commented Dec 3, 2022

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@Stephan202 Stephan202 marked this pull request as ready for review December 3, 2022 13:00
@Stephan202 Stephan202 mentioned this pull request Dec 3, 2022
Merged
1 task
werli
werli approved these changes Dec 4, 2022
View reviewed changes
Copy link
Member

@werli werli left a comment

Choose a reason for hiding this comment

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

Interesting change. LGTM 👍

@rickie rickie added the chore A task not related to code (build, formatting, process, ...) label Dec 5, 2022
rickie
rickie approved these changes Dec 5, 2022
View reviewed changes
Copy link
Member

@rickie rickie left a comment

Choose a reason for hiding this comment

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

Nice improvements! Love how we integrated this tool in our development process 😄.

@rickie rickie force-pushed the sschroevers/improve-pitest-setup branch from 54b5998 to b2c234d Compare December 5, 2022 07:59
@rickie
Copy link
Member

rickie commented Dec 5, 2022

Rebased, will merge once 🟢 .

@github-actions
Copy link

github-actions bot commented Dec 5, 2022

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@Stephan202 @rickie
Improve mutation testing setup
b73a029 
Summary of changes:
- Enable Pitest's built-in `STRONGER` mutator group.
- Enable Arcmutate's `EXTENDED` mutator group.
- Enable Arcmutate's JUnit 5 Accelerator Plugin.
- Modify `MoreTypesTest` such that it is impacted by mutations to the
  `MoreTypes` class.

See:
- https://pitest.org/quickstart/mutators/
- https://docs.arcmutate.com/docs/extended-operators.html
- https://docs.arcmutate.com/docs/accelerator.html
@Stephan202 @rickie
Doh
29b4f3b
@rickie rickie force-pushed the sschroevers/improve-pitest-setup branch from b2c234d to 29b4f3b Compare December 5, 2022 08:01
@github-actions
Copy link

github-actions bot commented Dec 5, 2022

Looks good. No mutations were possible for these changes.
Mutation testing report by Pitest. Review any surviving mutants by inspecting the line comments under Files changed.

@rickie rickie merged commit 066591c into master Dec 5, 2022
@rickie rickie deleted the sschroevers/improve-pitest-setup branch December 5, 2022 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@werli werli werli approved these changes

@rickie rickie rickie approved these changes

Assignees
No one assigned
Labels
chore A task not related to code (build, formatting, process, ...)
Milestone
0.6.0
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Stephan202 @rickie @werli