Issue checkstyle#12282: Resolve Pitest suppression for ParenPad

Kevin222004 · Oct 21, 2022 · ae5b248 · ae5b248
1 parent ad27aa0
commit ae5b248
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 10 deletions.
diff --git a/.ci/pitest-suppressions/pitest-whitespace-suppressions.xml b/.ci/pitest-suppressions/pitest-whitespace-suppressions.xml
@@ -81,15 +81,6 @@
     <lineContent>processLeft(ast.findFirstToken(TokenTypes.LPAREN));</lineContent>
   </mutation>
 
-  <mutation unstable="false">
-    <sourceFile>ParenPadCheck.java</sourceFile>
-    <mutatedClass>com.puppycrawl.tools.checkstyle.checks.whitespace.ParenPadCheck</mutatedClass>
-    <mutatedMethod>visitToken</mutatedMethod>
-    <mutator>org.pitest.mutationtest.engine.gregor.mutators.experimental.RemoveSwitchMutator_3</mutator>
-    <description>RemoveSwitch 3 mutation</description>
-    <lineContent>switch (ast.getType()) {</lineContent>
-  </mutation>
-
   <mutation unstable="false">
     <sourceFile>ParenPadCheck.java</sourceFile>
     <mutatedClass>com.puppycrawl.tools.checkstyle.checks.whitespace.ParenPadCheck</mutatedClass>

diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadCheck.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/whitespace/ParenPadCheck.java
@@ -254,7 +254,6 @@ public void visitToken(DetailAST ast) {
                 break;
             case TokenTypes.DOT:
             case TokenTypes.EXPR:
-            case TokenTypes.QUESTION:
                 processExpression(ast);
                 break;
             case TokenTypes.LITERAL_FOR:
@@ -270,6 +269,7 @@ public void visitToken(DetailAST ast) {
             case TokenTypes.RESOURCE_SPECIFICATION:
                 visitResourceSpecification(ast);
                 break;
+            case TokenTypes.QUESTION:
             default:
                 processLeft(ast.findFirstToken(TokenTypes.LPAREN));
                 processRight(ast.findFirstToken(TokenTypes.RPAREN));