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 AccessModifierOption #12326

Closed
ThatSneakyCoder opened this issue Oct 22, 2022 · 2 comments
Closed

Resolve Pitest suppression for AccessModifierOption #12326

ThatSneakyCoder opened this issue Oct 22, 2022 · 2 comments
Labels
approved miscellaneous
Milestone
10.5.0

Comments

@ThatSneakyCoder
Copy link
Contributor

Reference from #7797

Kill the mutation

checkstyle/.ci/pitest-suppressions/pitest-naming-suppressions.xml

Lines 12 to 19 in 4c7c3cc

<mutation unstable="false">
<sourceFile>AccessModifierOption.java</sourceFile>
<mutatedClass>com.puppycrawl.tools.checkstyle.checks.naming.AccessModifierOption</mutatedClass>
<mutatedMethod>getInstance</mutatedMethod>
<mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.NakedReceiverMutator</mutator>
<description>replaced call to java/lang/String::trim with receiver</description>
<lineContent>return valueOf(AccessModifierOption.class, modifierName.trim().toUpperCase(Locale.ENGLISH));</lineContent>
</mutation>

@ThatSneakyCoder
Copy link
Contributor Author

I am on it.

ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Oct 23, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
a054f56 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Oct 23, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
5d1fea9 
…Option
@ThatSneakyCoder ThatSneakyCoder mentioned this issue Oct 23, 2022
Merged
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 15, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
d429f55 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 15, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
973fc77 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 17, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
4b685bb 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 20, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
856dca1 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 20, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
c7b0c4c 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 20, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
9ebf025 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 23, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
af40a7a 
…Option
ThatSneakyCoder added a commit to ThatSneakyCoder/checkstyle that referenced this issue Nov 23, 2022
@ThatSneakyCoder
Issue checkstyle#12326: Resolve Pitest suppression for AccessModifier…
cb0778e 
…Option
romani pushed a commit that referenced this issue Nov 23, 2022
@ThatSneakyCoder @romani
Issue #12326: Resolve Pitest suppression for AccessModifierOption
edd2b09
@romani romani added this to the 10.5 milestone Nov 23, 2022
@romani
Copy link
Member

romani commented Nov 23, 2022

Fix is merged

@romani romani closed this as completed Nov 23, 2022
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.5.0
Development

No branches or pull requests
2 participants
@romani @ThatSneakyCoder