Skip to content

Issue #13501: Kill mutation for RegexpCheck-3#13383

Merged
romani merged 1 commit intocheckstyle:masterfrom
Kevin222004:regexp3
Aug 2, 2023
Merged

Issue #13501: Kill mutation for RegexpCheck-3#13383
romani merged 1 commit intocheckstyle:masterfrom
Kevin222004:regexp3

Conversation

@Kevin222004
Copy link
Copy Markdown
Contributor

@Kevin222004 Kevin222004 commented Jul 9, 2023

Issue #13501: Kill mutation for RegexpCheck-3


Check

https://checkstyle.org/checks/regexp/regexp.html#Regexp


Mutation

<mutation unstable="false">
<sourceFile>RegexpCheck.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck</mutatedClass>
<mutatedMethod>beginTree</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.MemberVariableMutator</mutator>
<description>Removed assignment to member variable errorCount</description>
<lineContent>errorCount = 0;</lineContent>
</mutation>
<mutation unstable="false">
<sourceFile>RegexpCheck.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck</mutatedClass>
<mutatedMethod>beginTree</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.MemberVariableMutator</mutator>
<description>Removed assignment to member variable matchCount</description>
<lineContent>matchCount = 0;</lineContent>
</mutation>
<mutation unstable="false">
<sourceFile>RegexpCheck.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck</mutatedClass>
<mutatedMethod>isIgnore</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.NonVoidMethodCallMutator</mutator>
<description>removed call to com/puppycrawl/tools/checkstyle/api/FileText::lineColumn</description>
<lineContent>end = text.lineColumn(0);</lineContent>
</mutation>
<mutation unstable="false">
<sourceFile>RegexpCheck.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.checks.regexp.RegexpCheck</mutatedClass>
<mutatedMethod>setDuplicateLimit</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.MemberVariableMutator</mutator>
<description>Removed assignment to member variable duplicateLimit</description>
<lineContent>this.duplicateLimit = duplicateLimit;</lineContent>
</mutation>


Explaination

Test added

Copy link
Copy Markdown
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

Items

@Kevin222004
Copy link
Copy Markdown
Contributor Author

Please avoid ci I will resolve them

please confirm the method looks good or not

Copy link
Copy Markdown
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

Apply for all cases like this:

@romani
Copy link
Copy Markdown
Member

romani commented Aug 1, 2023

Waiting for green CI

@Kevin222004 Kevin222004 changed the title Issue #13321: Kill mutation for RegexpCheck-3 Issue #13501: Kill mutation for RegexpCheck-3 Aug 2, 2023
@Kevin222004
Copy link
Copy Markdown
Contributor Author

#13383 (comment)

@romani done failure not related to ci

Copy link
Copy Markdown
Member

@romani romani left a comment

Choose a reason for hiding this comment

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

Ok to merge

@romani romani merged commit 19b2594 into checkstyle:master Aug 2, 2023
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.

2 participants