For #884: Review corrections.

Author: paulodamaso

qulice-spotbugs/pom.xml

@@ -48,15 +48,6 @@
             <artifactId>qulice-spi</artifactId>
             <version>${project.version}</version>
         </dependency>
-        <dependency>
-            <groupId>org.cactoos</groupId>
-            <artifactId>cactoos</artifactId>
-            <version>0.38</version>
-        </dependency>
-        <dependency>
-            <groupId>com.jcabi</groupId>
-            <artifactId>jcabi-log</artifactId>
-        </dependency>
         <dependency>
             <groupId>junit</groupId>
             <artifactId>junit</artifactId>

qulice-spotbugs/src/main/java/com/qulice/spotbugs/SpotBugsValidator.java

@@ -29,78 +29,25 @@
  */
 package com.qulice.spotbugs;
 
-import com.jcabi.log.Logger;
 import com.qulice.spi.Environment;
 import com.qulice.spi.ValidationException;
 import com.qulice.spi.Validator;
-import java.util.Collection;
-import java.util.LinkedList;
 
 /**
  * Validates source code and compiled binaries with SpotBugs.
  *
  * @since 0.19
  */
-@SuppressWarnings({"PMD.ExcessiveImports", "PMD.AvoidDuplicateLiterals"})
 public final class SpotBugsValidator implements Validator {
 
     @Override
     public void validate(final Environment env) throws ValidationException {
-        if (env.outdir().exists()) {
-            if (!env.exclude("spotbugs", "")) {
-                this.check(SpotBugsValidator.spotbugs(env));
-            }
-        } else {
-            Logger.info(
-                this,
-                "No classes at %s, no SpotBugs validation",
-                env.outdir()
-            );
-        }
+        throw new UnsupportedOperationException("validate not implemented");
     }
 
     @Override
     public String name() {
         return "SpotBugs";
     }
 
-    /**
-     * Start spotbugs and return its output.
-     * @param env Environment
-     * @return Output of spotbugs
-     */
-    @SuppressWarnings("PMD.AvoidDuplicateLiterals")
-    private static String spotbugs(final Environment env) {
-        throw new UnsupportedOperationException("spotbugs not implemented");
-    }
-
-    /**
-     * Java options.
-     * @return Options
-     */
-    private static Collection<String> options() {
-        return new LinkedList<>();
-    }
-
-    /**
-     * Check report for errors.
-     * @param report The report
-     * @throws ValidationException If it contains errors
-     */
-    private void check(final String report) throws ValidationException {
-        int total = 0;
-        for (final String line
-            : report.split(System.getProperty("line.separator"))) {
-            if (line.matches("[a-zA-Z ]+: .*")) {
-                Logger.warn(this, "SpotBugs: %s", line);
-                ++total;
-            }
-        }
-        if (total > 0) {
-            throw new ValidationException(
-                "%d SpotBugs violations (see log above)",
-                total
-            );
-        }
-    }
 }