supplemental: resolve checkstyle violations from expanded search #444

Author: rnveach

config/sevntu_suppressions.xml

@@ -21,6 +21,8 @@
 
 
     <!-- Tone down the checking for test code -->
+    <suppress checks="MultipleStringLiteralsExtended" files=".*[\\/]src[\\/]it[\\/]"/>
     <suppress checks="MultipleStringLiteralsExtended" files=".*[\\/]src[\\/]test[\\/]"/>
     <suppress checks="ForbidCCommentsInMethods" files=".*[\\/]src[\\/]test[\\/]"/>
+    <suppress checks="AvoidModifiersForTypes" files=".*[\\/]src[\\/]it[\\/]"/>
 </suppressions>

src/it/java/org/checkstyle/plugins/sonar/RunPluginTest.java

@@ -103,15 +103,9 @@ public static void afterAll() {
     }
 
     @Test
-    public void testSonarExecution() {
-        try {
-            final MavenBuild build = testProjectBuild();
-            executeBuildWithCommonProperties(build, true);
-        }
-        catch (IOException exception) {
-            LOG.error("Build execution error.", exception);
-            fail("Failed to execute build.");
-        }
+    public void testSonarExecution() throws IOException {
+        final MavenBuild build = testProjectBuild();
+        executeBuildWithCommonProperties(build, true);
     }
 
     private static void executeBuildWithCommonProperties(Build<?> build, boolean buildQuietly)

src/test/java/org/sonar/plugins/checkstyle/CheckstyleRulesDefinitionTest.java

@@ -24,6 +24,7 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.Map.Entry;
 import java.util.stream.Collectors;
 
 import org.junit.Rule;
@@ -71,11 +72,11 @@ public void test() {
         });
         final List<String> duplicatedRuleWithTemplate = ruleCounts.entrySet().stream()
                 .filter(entry -> entry.getValue() > 1)
-                .map(entry -> entry.getKey())
+                .map(Entry::getKey)
                 .collect(Collectors.toList());
         final List<String> rulesWithDuplicateTemplate = ruleCounts.entrySet().stream()
                 .filter(entry -> entry.getValue() == 1)
-                .map(entry -> entry.getKey())
+                .map(Entry::getKey)
                 .collect(Collectors.toList());
         // such number should not change during checkstyle version upgrade
         assertThat(duplicatedRuleWithTemplate).hasSize(0);