Skip to content

Issue #13501: Kill mutation for DetailAstImpl-6#13652

Merged
romani merged 1 commit intocheckstyle:masterfrom
Kevin222004:DetailAstImpl3
Aug 28, 2023
Merged

Issue #13501: Kill mutation for DetailAstImpl-6#13652
romani merged 1 commit intocheckstyle:masterfrom
Kevin222004:DetailAstImpl3

Conversation

@Kevin222004
Copy link
Copy Markdown
Contributor

@Kevin222004 Kevin222004 commented Aug 27, 2023 •
edited
Loading

Issue #13501: Kill mutation for DetailAstImpl-6

https://github.com/checkstyle/checkstyle/blob/master/src/main/java/com/puppycrawl/tools/checkstyle/DetailAstImpl.java

Mutations

checkstyle/config/pitest-suppressions/pitest-common-2-suppressions.xml

Lines 75 to 91 in 88486f7

<mutation unstable="false">
<sourceFile>DetailAstImpl.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.DetailAstImpl</mutatedClass>
<mutatedMethod>getHiddenBefore</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.ArgumentPropagationMutator</mutator>
<description>replaced call to java/util/Collections::unmodifiableList with argument</description>
<lineContent>returnList = Collections.unmodifiableList(hiddenBefore);</lineContent>
</mutation>
<mutation unstable="false">
<sourceFile>DetailAstImpl.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.DetailAstImpl</mutatedClass>
<mutatedMethod>removeChildren</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.MemberVariableMutator</mutator>
<description>Removed assignment to member variable firstChild</description>
<lineContent>firstChild = null;</lineContent>
</mutation>

Explaination

Test added

@Kevin222004 Kevin222004 marked this pull request as ready for review August 27, 2023 13:17
@romani romani merged commit 1bb1db6 into checkstyle:master Aug 28, 2023
@Kevin222004 Kevin222004 mentioned this pull request Aug 29, 2023
Closed
61 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@romani romani romani approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Kevin222004 @romani