From 63b6b6862f9e9f28a2206ade5e1d36f06a7dd011 Mon Sep 17 00:00:00 2001
From: Shubh Srivastava <shubh22.srivastava@gmail.com>
Date: Wed, 12 Apr 2023 10:35:57 +0530
Subject: [PATCH] Issue #12876: Resolve pitest suppression for
 AbstractViolationReporter

---
 .../pitest-api-suppressions.xml               |  9 --------
 .../coding/UnusedLocalVariableCheckTest.java  |  9 ++++++++
 ...nusedLocalVariableTestWarningSeverity.java | 21 +++++++++++++++++++
 3 files changed, 30 insertions(+), 9 deletions(-)
 create mode 100644 src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/unusedlocalvariable/InputUnusedLocalVariableTestWarningSeverity.java

diff --git a/config/pitest-suppressions/pitest-api-suppressions.xml b/config/pitest-suppressions/pitest-api-suppressions.xml
index 52f7123f7a2..25a7a03d546 100644
--- 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/src/test/java/com/puppycrawl/tools/checkstyle/checks/coding/UnusedLocalVariableCheckTest.java b/src/test/java/com/puppycrawl/tools/checkstyle/checks/coding/UnusedLocalVariableCheckTest.java
index 5b321276d58..97cf08fd26c 100644
--- a/src/test/java/com/puppycrawl/tools/checkstyle/checks/coding/UnusedLocalVariableCheckTest.java
+++ b/src/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/src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/unusedlocalvariable/InputUnusedLocalVariableTestWarningSeverity.java b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/unusedlocalvariable/InputUnusedLocalVariableTestWarningSeverity.java
new file mode 100644
index 00000000000..1cf15996f82
--- /dev/null
+++ b/src/test/resources/com/puppycrawl/tools/checkstyle/checks/coding/unusedlocalvariable/InputUnusedLocalVariableTestWarningSeverity.java
@@ -0,0 +1,21 @@
+/*
+UnusedLocalVariable
+severity = warning
+
+
+*/
+
+package com.puppycrawl.tools.checkstyle.checks.coding.unusedlocalvariable;
+
+public class InputUnusedLocalVariableTestWarningSeverity {
+
+    void m() {
+     @Test.A Outer p1 = new @Test.A Outer(); // ok
+     @Test.A Outer.@Test.B Inner p2 = p1.new @Test.B Inner();
+     // violation above but severity is set to warning
+    }
+
+}
+
+
+