Skip to content

Issue #13321: Kill mutation for SummaryJavaDocCheck-12#13330

Merged
romani merged 1 commit intocheckstyle:masterfrom
Kevin222004:s12
Jun 29, 2023
Merged

Issue #13321: Kill mutation for SummaryJavaDocCheck-12#13330
romani merged 1 commit intocheckstyle:masterfrom
Kevin222004:s12

Conversation

@Kevin222004
Copy link
Copy Markdown
Contributor

@Kevin222004 Kevin222004 commented Jun 29, 2023

Issue #13321: Kill mutation for SummaryJavaDocCheck-12

Check :-

https://checkstyle.org/checks/javadoc/summaryjavadoc.html#SummaryJavadoc

Mutation

checkstyle/config/pitest-suppressions/pitest-javadoc-suppressions.xml

Lines 970 to 977 in 82589c1

<mutation unstable="false">
<sourceFile>SummaryJavadocCheck.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.checks.javadoc.SummaryJavadocCheck</mutatedClass>
<mutatedMethod>validateInlineReturnTag</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.ArgumentPropagationMutator</mutator>
<description>replaced call to com/puppycrawl/tools/checkstyle/checks/javadoc/SummaryJavadocCheck::getVisibleContent with argument</description>
<lineContent>final String returnVisible = getVisibleContent(inlineReturn);</lineContent>
</mutation>

Explaination

Test added

@Kevin222004 Kevin222004 mentioned this pull request Jun 29, 2023
Closed
41 tasks
romani
romani approved these changes Jun 29, 2023
View reviewed changes
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 9e7a909 into checkstyle:master Jun 29, 2023
@Kevin222004 Kevin222004 mentioned this pull request Aug 2, 2023
Closed
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