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

Resolve pitest suppression for AbstractViolationReporter #12876

Closed
ThatSneakyCoder opened this issue Mar 21, 2023 · 3 comments · Fixed by #12884
Closed

Resolve pitest suppression for AbstractViolationReporter #12876

ThatSneakyCoder opened this issue Mar 21, 2023 · 3 comments · Fixed by #12884
Labels
approved miscellaneous
Milestone
10.10.0

Comments

@ThatSneakyCoder
Copy link
Contributor

ThatSneakyCoder commented Mar 21, 2023 •
edited

Related to #12341

Kill the surviving mutation.

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

Lines 3 to 10 in d59b83b

<mutation unstable="false">
<sourceFile>AbstractViolationReporter.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.api.AbstractViolationReporter</mutatedClass>
<mutatedMethod>setSeverity</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.MemberVariableMutator</mutator>
<description>Removed assignment to member variable severityLevel</description>
<lineContent>severityLevel = SeverityLevel.getInstance(severity);</lineContent>
</mutation>

@ThatSneakyCoder
Copy link
Contributor Author

ThatSneakyCoder commented Mar 21, 2023 •
edited

@nrmancuso @romani please approve.

@ThatSneakyCoder ThatSneakyCoder mentioned this issue Mar 22, 2023
Merged
@ThatSneakyCoder
Copy link
Contributor Author

I'm on it.

ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Mar 22, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
13acc31 
…ionReporter
@ThatSneakyCoder ThatSneakyCoder mentioned this issue Mar 22, 2023
Merged
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Mar 23, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
5088964 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Mar 26, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
1489b0f 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 5, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
b1392ab 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 11, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
9b5f0fc 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 12, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
c634202 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 12, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
63b6b68 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 12, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
18ad6f2 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 12, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
3667cda 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 13, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
eea02ff 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 13, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
9cd8ab6 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 13, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
76e14c3 
…ionReporter
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Apr 14, 2023
@ThatSneakyCoder
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
edc5109 
…ionReporter
nrmancuso pushed a commit that referenced this issue Apr 14, 2023
@ThatSneakyCoder @nrmancuso
Issue #12876: Resolve pitest suppression for AbstractViolationReporter
82cdd1e
@github-actions github-actions bot added this to the 10.9.4 milestone Apr 14, 2023
@nrmancuso
Copy link
Member

Closed via #12884

Sloent pushed a commit to Sloent/checkstyle that referenced this issue Apr 18, 2023
@ThatSneakyCoder @Sloent
Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…
8f4d9b9 
…ionReporter
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
approved miscellaneous
Projects
None yet
Milestone
10.10.0
Development

Successfully merging a pull request may close this issue.

Issue #12876: Resolve pitest suppression for AbstractViolationReporter ThatSneakyCoder/checkstyle
3 participants
@romani @nrmancuso @ThatSneakyCoder