Issue #12326: Resolve Pitest suppression for AccessModifierOption

checkstyle · Nov 15, 2022 · 973fc77 · 973fc77
1 parent d429f55
commit 973fc77
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 17 deletions.
diff --git a/src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/AccessModifierOption.java b/src/main/java/com/puppycrawl/tools/checkstyle/checks/naming/AccessModifierOption.java
@@ -63,7 +63,7 @@ private String getName() {
      * @return the AccessModifier associated with given access modifier name.
      */
     public static AccessModifierOption getInstance(String modifierName) {
-        return valueOf(AccessModifierOption.class, modifierName.trim().toUpperCase(Locale.ENGLISH));
+        return valueOf(AccessModifierOption.class, modifierName.toUpperCase(Locale.ENGLISH));
     }
 
 }
diff --git a/src/test/java/com/puppycrawl/tools/checkstyle/checks/naming/AccessModifierOptionTest.java b/src/test/java/com/puppycrawl/tools/checkstyle/checks/naming/AccessModifierOptionTest.java
@@ -57,20 +57,4 @@ public void testCase() {
                 .isEqualTo("private");
     }
 
-    @Test
-    public void testGetInstance() throws Exception {
-        assertWithMessage("Case mismatch.")
-                .that(AccessModifierOption.getInstance(" " + "public"))
-                .isEqualTo(AccessModifierOption.PUBLIC);
-        assertWithMessage("Case mismatch.")
-                .that(AccessModifierOption.getInstance(" " + "protected"))
-                .isEqualTo(AccessModifierOption.PROTECTED);
-        assertWithMessage("Case mismatch.")
-                .that(AccessModifierOption.getInstance(" " + "package"))
-                .isEqualTo(AccessModifierOption.PACKAGE);
-        assertWithMessage("Case mismatch.")
-                .that(AccessModifierOption.getInstance(" " + "private"))
-                .isEqualTo(AccessModifierOption.PRIVATE);
-    }
-
 }