From: Simon Brandhof <simon.brandhof@gmail.com>
Date: Mon, 21 Oct 2013 15:07:40 +0000 (+0200)
Subject: Fix some quality flaws
X-Git-Tag: 4.0~40
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=0a67df44a1f04ffd6a4d7d2845b6a1e136519f04;p=sonarqube.git

Fix some quality flaws
---

diff --git a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndex.java b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndex.java
index ce70ee6d636..e58052abd99 100644
--- a/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndex.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/scan/filesystem/FileIndex.java
@@ -150,18 +150,18 @@ public class FileIndex implements BatchComponent {
     if (path == null) {
       LoggerFactory.getLogger(getClass()).warn(String.format("File '%s' is not in basedir '%s'", file.getAbsolutePath(), fileSystem.baseDir()));
     } else {
-        InputFile input = newInputFile(fileSystem, sourceDir, type, file, path);
-        if (input != null && accept(input)) {
-          cache.put(fileSystem.moduleKey(), input);
-          status.markAsIndexed(path);
-        }
+      InputFile input = newInputFile(fileSystem, sourceDir, type, file, path);
+      if (input != null && accept(input)) {
+        cache.put(fileSystem.moduleKey(), input);
+        status.markAsIndexed(path);
       }
+    }
   }
 
   @CheckForNull
   private InputFile newInputFile(ModuleFileSystem fileSystem, File sourceDir, String type, File file, String path) {
     String lang = languageRecognizer.of(file);
-    if (lang==null) {
+    if (lang == null) {
       return null;
     }
 
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/batch/SquidUtils.java b/sonar-plugin-api/src/main/java/org/sonar/api/batch/SquidUtils.java
index b9136c0fb69..dec485a41a7 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/batch/SquidUtils.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/batch/SquidUtils.java
@@ -21,7 +21,6 @@ package org.sonar.api.batch;
 
 import org.apache.commons.io.FilenameUtils;
 import org.apache.commons.lang.StringUtils;
-import org.sonar.api.resources.Java;
 import org.sonar.api.resources.JavaFile;
 import org.sonar.api.resources.JavaPackage;
 
diff --git a/sonar-server/src/main/java/org/sonar/server/db/migrations/violation/ViolationConverters.java b/sonar-server/src/main/java/org/sonar/server/db/migrations/violation/ViolationConverters.java
index 3115472e1b5..622708d239f 100644
--- a/sonar-server/src/main/java/org/sonar/server/db/migrations/violation/ViolationConverters.java
+++ b/sonar-server/src/main/java/org/sonar/server/db/migrations/violation/ViolationConverters.java
@@ -26,7 +26,10 @@ import org.sonar.core.persistence.Database;
 import java.util.List;
 import java.util.Timer;
 import java.util.TimerTask;
-import java.util.concurrent.*;
+import java.util.concurrent.Callable;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
+import java.util.concurrent.Future;
 
 class ViolationConverters {
 
@@ -38,7 +41,7 @@ class ViolationConverters {
     this.settings = settings;
   }
 
-  void execute(Referentials referentials, Database db) throws ExecutionException, InterruptedException {
+  void execute(Referentials referentials, Database db) {
     Progress progress = new Progress(referentials.totalViolations());
 
     List<Callable<Object>> converters = Lists.newArrayList();
@@ -49,7 +52,7 @@ class ViolationConverters {
     doExecute(progress, converters);
   }
 
-  void doExecute(TimerTask progress, List<Callable<Object>> converters) throws InterruptedException, ExecutionException {
+  void doExecute(TimerTask progress, List<Callable<Object>> converters) {
     Timer timer = new Timer(Progress.THREAD_NAME);
     timer.schedule(progress, Progress.DELAY_MS, Progress.DELAY_MS);
     try {
@@ -59,6 +62,8 @@ class ViolationConverters {
       for (Future result : results) {
         result.get();
       }
+    } catch (Exception e) {
+      throw new IllegalStateException("Fail to start migration threads", e);
     } finally {
       progress.cancel();
       timer.cancel();
diff --git a/sonar-server/src/test/java/org/sonar/server/db/migrations/violation/ViolationConvertersTest.java b/sonar-server/src/test/java/org/sonar/server/db/migrations/violation/ViolationConvertersTest.java
index 9b144fe7846..6e37ed949e0 100644
--- a/sonar-server/src/test/java/org/sonar/server/db/migrations/violation/ViolationConvertersTest.java
+++ b/sonar-server/src/test/java/org/sonar/server/db/migrations/violation/ViolationConvertersTest.java
@@ -20,6 +20,7 @@
 package org.sonar.server.db.migrations.violation;
 
 import com.google.common.collect.Lists;
+import org.apache.commons.lang.exception.ExceptionUtils;
 import org.junit.Test;
 import org.sonar.api.config.Settings;
 
@@ -68,8 +69,8 @@ public class ViolationConvertersTest {
     try {
       new ViolationConverters(new Settings()).doExecute(new FakeTimerTask(), callables);
       fail();
-    } catch (ExecutionException e) {
-      assertThat(e.getCause().getMessage()).isEqualTo("Need to cry");
+    } catch (Exception e) {
+      assertThat(ExceptionUtils.getRootCause(e).getMessage()).isEqualTo("Need to cry");
     }
 
   }