Issue checkstyle#12876: Resolve pitest suppression for AbstractViolat…

…ionReporter
ThatSneakyCoder · Apr 12, 2023 · c634202 · c634202
1 parent d2648e4
commit c634202
Show file tree

Hide file tree

Showing 3 changed files with 32 additions and 9 deletions.
diff --git a/config/pitest-suppressions/pitest-api-suppressions.xml b/config/pitest-suppressions/pitest-api-suppressions.xml
@@ -1,14 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <suppressedMutations>
-  <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>
-
   <mutation unstable="false">
     <sourceFile>FileContents.java</sourceFile>
     <mutatedClass>com.puppycrawl.tools.checkstyle.api.FileContents</mutatedClass>

diff --git a/...test/java/com/puppycrawl/tools/checkstyle/checks/coding/UnusedLocalVariableCheckTest.java b/...test/java/com/puppycrawl/tools/checkstyle/checks/coding/UnusedLocalVariableCheckTest.java
@@ -241,6 +241,15 @@ public void testUnusedLocalVarNestedClasses3() throws Exception {
                 expected);
     }
 
+    @Test
+    public void testUnusedLocalVarTestWarningSeverity() throws Exception {
+        final String[] expected = CommonUtil.EMPTY_STRING_ARRAY;
+
+        verifyWithInlineConfigParser(
+                getPath("InputUnusedLocalVariableTestWarningSeverity.java"),
+                expected);
+    }
+
     @Test
     public void testUnusedLocalVarEnum() throws Exception {
         final String[] expected = {

diff --git a/...kstyle/checks/coding/unusedlocalvariable/InputUnusedLocalVariableTestWarningSeverity.java b/...kstyle/checks/coding/unusedlocalvariable/InputUnusedLocalVariableTestWarningSeverity.java
@@ -0,0 +1,23 @@
+/*
+UnusedLocalVariable
+severity = warning
+
+
+*/
+
+package com.puppycrawl.tools.checkstyle.checks.coding.unusedlocalvariable;
+
+public class InputUnusedLocalVariableTestWarningSeverity {
+    /*
+       To kill the mutation in AbstractViolationReporter in the api package,
+       we only had to change the severity level in the input file to warning
+    */
+
+    /*
+       some code
+    */
+
+}
+
+
+