Minor Quality flaws

diff --git a/server/sonar-server/src/main/java/org/sonar/server/db/EmbeddedDatabaseFactory.java b/server/sonar-server/src/main/java/org/sonar/server/db/EmbeddedDatabaseFactory.java
index 25e15e503c1f06a4ad7c1e21c7a6d13cd3457c1d..5a5f4e45c156942d6f3bec241b025a84b172bb04 100644 (file)
--- a/server/sonar-server/src/main/java/org/sonar/server/db/EmbeddedDatabaseFactory.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/db/EmbeddedDatabaseFactory.java
@@ -40,7 +40,7 @@ public class EmbeddedDatabaseFactory implements Startable {
     if (embeddedDatabase == null) {
       String jdbcUrl = settings.getString(DatabaseProperties.PROP_URL);
       if (startsWith(jdbcUrl, URL_PREFIX)) {
-        embeddedDatabase = getEmbeddedDatabase(settings);
+        embeddedDatabase = createEmbeddedDatabase();
         embeddedDatabase.start();
       }
     }
@@ -55,7 +55,7 @@ public class EmbeddedDatabaseFactory implements Startable {
   }
 
   @VisibleForTesting
-  EmbeddedDatabase getEmbeddedDatabase(Settings settings) {
+  EmbeddedDatabase createEmbeddedDatabase() {
     return new EmbeddedDatabase(settings);
   }
 }

diff --git a/server/sonar-server/src/test/java/org/sonar/server/db/EmbeddedDatabaseFactoryTest.java b/server/sonar-server/src/test/java/org/sonar/server/db/EmbeddedDatabaseFactoryTest.java
index 8cbe8790346117c20294999c368710cb7c2ccc75..2a780dc68123cf79fb5cd1333a19b9d933026309 100644 (file)
--- a/server/sonar-server/src/test/java/org/sonar/server/db/EmbeddedDatabaseFactoryTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/db/EmbeddedDatabaseFactoryTest.java
@@ -23,7 +23,9 @@ import org.junit.Test;
 import org.sonar.api.config.Settings;
 import org.sonar.api.database.DatabaseProperties;
 
-import static org.mockito.Mockito.*;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.never;
+import static org.mockito.Mockito.verify;
 
 public class EmbeddedDatabaseFactoryTest {
 
@@ -33,11 +35,11 @@ public class EmbeddedDatabaseFactoryTest {
   public void should_start_and_stop_tcp_h2_database() {
     settings.setProperty(DatabaseProperties.PROP_URL, "jdbc:h2:tcp:localhost");
 
-    final EmbeddedDatabase embeddedDatabase = mock(EmbeddedDatabase.class);
+    EmbeddedDatabase embeddedDatabase = mock(EmbeddedDatabase.class);
 
     EmbeddedDatabaseFactory databaseFactory = new EmbeddedDatabaseFactory(settings) {
       @Override
-      EmbeddedDatabase getEmbeddedDatabase(Settings settings) {
+      EmbeddedDatabase createEmbeddedDatabase() {
         return embeddedDatabase;
       }
     };
@@ -52,11 +54,11 @@ public class EmbeddedDatabaseFactoryTest {
   public void should_not_start_mem_h2_database() {
     settings.setProperty(DatabaseProperties.PROP_URL, "jdbc:h2:mem");
 
-    final EmbeddedDatabase embeddedDatabase = mock(EmbeddedDatabase.class);
+    EmbeddedDatabase embeddedDatabase = mock(EmbeddedDatabase.class);
 
     EmbeddedDatabaseFactory databaseFactory = new EmbeddedDatabaseFactory(settings) {
       @Override
-      EmbeddedDatabase getEmbeddedDatabase(Settings settings) {
+      EmbeddedDatabase createEmbeddedDatabase() {
         return embeddedDatabase;
       }
     };

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/utils/Semaphores.java b/sonar-plugin-api/src/main/java/org/sonar/api/utils/Semaphores.java
index 8e77894ba28a4fd999e6f4b08db049166d4e1307..ffd503ed486ca5d9aa4c9d8a05579a3d1c65a8fb 100644 (file)
--- a/sonar-plugin-api/src/main/java/org/sonar/api/utils/Semaphores.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/utils/Semaphores.java
@@ -27,7 +27,7 @@ import org.sonar.api.server.ServerSide;
  * A semaphore shared among all the processes that can connect to the central database.
  *
  * @since 3.4
- * @deprecated in 5.2 as batch analyzer does not connect to database anymore
+ * @deprecated in 5.2 as analyzer does not connect to database anymore
  */
 @ServerSide
 @Deprecated