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.

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

Implementation of Partial Mutation Pitests in pom.xml #20

Merged
merged 4 commits into from
Jun 1, 2024

Conversation

ElijahCanderson
Copy link
Contributor

@ElijahCanderson ElijahCanderson commented May 25, 2024

Closes #19

Implemented the ability to do partial Pitests with the commands provided below. Only some simple edits to pom.xml and edits to readme for the main commands.

For tests on one class do:
mvn pitest:mutationCoverage -DtargetClasses=edu.ucsb.cs156.organic.controllers.CLASS_NAME

You must use the class name, not the file name.

For tests on a package:

mvn pitest:mutationCoverage -DtargetClasses=edu.ucsb.cs156.organic.controllers.*

Output after running --- : mvn pitest:mutationCoverage -DtargetClasses=edu.ucsb.cs156.organic.controllers.StudentsController

image

Output after running ---:mvn pitest:mutationCoverage -DtargetClasses=edu.ucsb.cs156.organic.controllers.*
image

Division7
Division7 previously approved these changes May 25, 2024
Copy link
Contributor

@Division7 Division7 left a comment

Choose a reason for hiding this comment

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

LGTM 😎 🤙

Copy link
Contributor

@pconrad pconrad left a comment

Choose a reason for hiding this comment

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

Nice... but please copy over the changes to the README.md as well from the source PR.

@pconrad pconrad added the FIXME-See CR Please review the comments in the code review and address them; then remove this label label May 25, 2024
@Division7 Division7 added FIXME-See CR Please review the comments in the code review and address them; then remove this label and removed FIXME-See CR Please review the comments in the code review and address them; then remove this label labels May 25, 2024
@ElijahCanderson ElijahCanderson requested a review from pconrad May 26, 2024 04:29
@ElijahCanderson ElijahCanderson removed the FIXME-See CR Please review the comments in the code review and address them; then remove this label label May 26, 2024
Division7
Division7 previously approved these changes May 27, 2024
Copy link
Contributor

@Division7 Division7 left a comment

Choose a reason for hiding this comment

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

LGTM! 😎🤙

README.md Outdated Show resolved Hide resolved
@pconrad pconrad added the FIXME-See CR Please review the comments in the code review and address them; then remove this label label May 30, 2024
Copy link
Contributor

@GuBinhti GuBinhti left a comment

Choose a reason for hiding this comment

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

LGTM

@ElijahCanderson ElijahCanderson removed the FIXME-See CR Please review the comments in the code review and address them; then remove this label label May 31, 2024
@pconrad pconrad merged commit d05aeb0 into main Jun 1, 2024
8 checks passed
@pconrad pconrad added the 5 5 points label Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Partial Pitests
4 participants