Issue #160: fix teamcity violations

checkstyle · Sep 26, 2018 · 1daf188 · 1daf188
1 parent 010d408
commit 1daf188
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/checkstyle-sonar-plugin/src/main/java/org/sonar/plugins/checkstyle/CheckstyleExecutor.java b/checkstyle-sonar-plugin/src/main/java/org/sonar/plugins/checkstyle/CheckstyleExecutor.java
@@ -143,7 +143,7 @@ private URLClassLoader createClassloader() {
         for (File file : classpathElements) {
             urls.add(getUrl(file.toURI()));
         }
-        return new URLClassLoader(urls.toArray(new URL[urls.size()]), null);
+        return new URLClassLoader(urls.toArray(new URL[0]), null);
     }
 
 }
diff --git a/...onar-plugin/src/test/java/org/sonar/plugins/checkstyle/CheckstyleProfileImporterTest.java b/...onar-plugin/src/test/java/org/sonar/plugins/checkstyle/CheckstyleProfileImporterTest.java
@@ -183,7 +183,7 @@ private static RuleFinder newRuleFinder() {
 
     private static class RuleAnswer implements Answer<Rule> {
         @Override
-        public Rule answer(InvocationOnMock iom) throws Throwable {
+        public Rule answer(InvocationOnMock iom) {
             final RuleQuery query = (RuleQuery) iom.getArguments()[0];
             Rule rule = null;
             if (StringUtils.equals(query.getConfigKey(), "Checker/JavadocPackage")) {

diff --git a/checkstyle-sonar-plugin/src/test/java/org/sonar/plugins/checkstyle/internal/ChecksTest.java b/checkstyle-sonar-plugin/src/test/java/org/sonar/plugins/checkstyle/internal/ChecksTest.java
@@ -394,11 +394,11 @@ else if (AbstractFileSetCheck.class.isAssignableFrom(clss)) {
         }
 
         // remove undocumented properties
-        new HashSet<>(properties).stream()
-            .filter(property -> {
-                return UNDOCUMENTED_PROPERTIES.contains(clss.getSimpleName() + "." + property);
-            })
-            .forEach(properties::remove);
+        new HashSet<>(properties)
+                .stream()
+                .filter(property -> UNDOCUMENTED_PROPERTIES.contains(clss
+                    .getSimpleName() + "." + property))
+                .forEach(properties::remove);
 
         if (AbstractCheck.class.isAssignableFrom(clss)) {
             final AbstractCheck check;