]> source.dussan.org Git - sonarqube.git/commitdiff
Remove direct usage of System2.INSTANCE in constructors
authorSimon Brandhof <simon.brandhof@sonarsource.com>
Fri, 15 May 2015 11:59:51 +0000 (13:59 +0200)
committerSimon Brandhof <simon.brandhof@sonarsource.com>
Fri, 15 May 2015 12:30:09 +0000 (14:30 +0200)
30 files changed:
server/sonar-server/src/main/java/org/sonar/server/component/db/ComponentDao.java
server/sonar-server/src/main/java/org/sonar/server/computation/db/AnalysisReportDao.java
server/sonar-server/src/main/java/org/sonar/server/debt/DebtModelBackup.java
server/sonar-server/src/main/java/org/sonar/server/debt/DebtModelOperations.java
server/sonar-server/src/main/java/org/sonar/server/issue/IssueCommentService.java
server/sonar-server/src/main/java/org/sonar/server/qualityprofile/db/ActiveRuleDao.java
server/sonar-server/src/main/java/org/sonar/server/rule/db/RuleDao.java
server/sonar-server/src/main/java/org/sonar/server/user/db/GroupDao.java
server/sonar-server/src/test/java/org/sonar/server/batch/IssuesActionTest.java
server/sonar-server/src/test/java/org/sonar/server/component/ws/ProvisionedProjectsActionTest.java
server/sonar-server/src/test/java/org/sonar/server/component/ws/SearchActionTest.java
server/sonar-server/src/test/java/org/sonar/server/computation/ComputationServiceTest.java
server/sonar-server/src/test/java/org/sonar/server/computation/ReportQueueTest.java
server/sonar-server/src/test/java/org/sonar/server/computation/issue/RuleCacheLoaderTest.java
server/sonar-server/src/test/java/org/sonar/server/computation/step/PersistIssuesStepTest.java
server/sonar-server/src/test/java/org/sonar/server/computation/step/PurgeRemovedViewsStepTest.java
server/sonar-server/src/test/java/org/sonar/server/design/ws/ShowActionTest.java
server/sonar-server/src/test/java/org/sonar/server/issue/IssueCommentServiceTest.java
server/sonar-server/src/test/java/org/sonar/server/qualityprofile/ws/QProfileDeleteActionTest.java
server/sonar-server/src/test/java/org/sonar/server/qualityprofile/ws/QProfileProjectsActionTest.java
server/sonar-server/src/test/java/org/sonar/server/source/ws/HashActionTest.java
server/sonar-server/src/test/java/org/sonar/server/source/ws/LinesActionTest.java
server/sonar-server/src/test/java/org/sonar/server/source/ws/ScmActionTest.java
server/sonar-server/src/test/java/org/sonar/server/test/ws/TestsListActionTest.java
server/sonar-server/src/test/java/org/sonar/server/user/db/GroupDaoTest.java
server/sonar-server/src/test/java/org/sonar/server/user/ws/ChangePasswordActionTest.java
server/sonar-server/src/test/java/org/sonar/server/user/ws/CreateActionTest.java
server/sonar-server/src/test/java/org/sonar/server/user/ws/UpdateActionTest.java
server/sonar-server/src/test/java/org/sonar/server/view/index/ViewIndexerTest.java
sonar-core/src/main/java/org/sonar/core/i18n/DefaultI18n.java

index 96d3eb0ab44e36e7d3ac5a0f8c1b6237dbba52b0..a57572426b65fe59d4b72b129d42fff66f8c6740 100644 (file)
@@ -53,10 +53,6 @@ import java.util.Map;
 @ServerSide
 public class ComponentDao extends BaseDao<ComponentMapper, ComponentDto, String> implements DaoComponent {
 
-  public ComponentDao() {
-    this(System2.INSTANCE);
-  }
-
   public ComponentDao(System2 system) {
     super(ComponentMapper.class, system);
   }
index f08edbf5b3a05744e96dbe7e7e8fad7c24fbc74a..115364cb4f4012e59d6cb93835d46fe2c654c87f 100644 (file)
@@ -38,10 +38,6 @@ public class AnalysisReportDao implements DaoComponent {
 
   private System2 system2;
 
-  public AnalysisReportDao() {
-    this(System2.INSTANCE);
-  }
-
   public AnalysisReportDao(System2 system2) {
     this.system2 = system2;
   }
index 560363f81cbcb7e14aa3e32cc0f117b0d62c2b27..90659cc54a43f3e31f464b5828b999c9c1a94433 100644 (file)
@@ -73,14 +73,6 @@ public class DebtModelBackup {
   private final UserSession userSession;
 
   public DebtModelBackup(DbClient dbClient, DebtModelOperations debtModelOperations, RuleOperations ruleOperations,
-    DebtModelPluginRepository debtModelPluginRepository, DebtCharacteristicsXMLImporter characteristicsXMLImporter, DebtRulesXMLImporter rulesXMLImporter,
-    DebtModelXMLExporter debtModelXMLExporter, RuleDefinitionsLoader defLoader, UserSession userSession) {
-    this(dbClient, debtModelOperations, ruleOperations, debtModelPluginRepository, characteristicsXMLImporter, rulesXMLImporter, debtModelXMLExporter,
-      defLoader, System2.INSTANCE, userSession);
-  }
-
-  @VisibleForTesting
-  DebtModelBackup(DbClient dbClient, DebtModelOperations debtModelOperations, RuleOperations ruleOperations,
     DebtModelPluginRepository debtModelPluginRepository, DebtCharacteristicsXMLImporter characteristicsXMLImporter, DebtRulesXMLImporter rulesXMLImporter,
     DebtModelXMLExporter debtModelXMLExporter, RuleDefinitionsLoader defLoader, System2 system2, UserSession userSession) {
     this.dbClient = dbClient;
index dcfd747e0b036ceae3d74541ac3a9e4119f4514e..532a310d8bceac0835438e92deb88008b9b59f03 100644 (file)
@@ -52,12 +52,7 @@ public class DebtModelOperations {
   private final System2 system2;
   private final UserSession userSession;
 
-  public DebtModelOperations(DbClient dbClient, UserSession userSession) {
-    this(dbClient, System2.INSTANCE, userSession);
-  }
-
-  @VisibleForTesting
-  DebtModelOperations(DbClient dbClient, System2 system2, UserSession userSession) {
+  public DebtModelOperations(DbClient dbClient, System2 system2, UserSession userSession) {
     this.dbClient = dbClient;
     this.system2 = system2;
     this.userSession = userSession;
index 34cedf47052cf94cd3194ecb345710f679f65496..3852b4c286deb222f7e98b83a5bf062624e2993f 100644 (file)
@@ -54,13 +54,11 @@ public class IssueCommentService {
   private final DbClient dbClient;
   private final IssueService issueService;
   private final IssueUpdater updater;
-  private final IssueChangeDao changeDao;
 
-  public IssueCommentService(DbClient dbClient, IssueService issueService, IssueUpdater updater, IssueChangeDao changeDao) {
+  public IssueCommentService(DbClient dbClient, IssueService issueService, IssueUpdater updater) {
     this.dbClient = dbClient;
     this.issueService = issueService;
     this.updater = updater;
-    this.changeDao = changeDao;
   }
 
   public List<DefaultIssueComment> findComments(String issueKey) {
@@ -81,11 +79,11 @@ public class IssueCommentService {
   }
 
   public List<DefaultIssueComment> findComments(DbSession session, Collection<String> issueKeys) {
-    return changeDao.selectCommentsByIssues(session, issueKeys);
+    return dbClient.issueChangeDao().selectCommentsByIssues(session, issueKeys);
   }
 
   public IssueComment findComment(String commentKey) {
-    return changeDao.selectCommentByKey(commentKey);
+    return dbClient.issueChangeDao().selectCommentByKey(commentKey);
   }
 
   public IssueComment addComment(String issueKey, String text, UserSession userSession) {
@@ -114,7 +112,7 @@ public class IssueCommentService {
   }
 
   public IssueComment deleteComment(String commentKey, UserSession userSession) {
-    DefaultIssueComment comment = changeDao.selectCommentByKey(commentKey);
+    DefaultIssueComment comment = dbClient.issueChangeDao().selectCommentByKey(commentKey);
     if (comment == null) {
       throw new NotFoundException("Comment not found: " + commentKey);
     }
@@ -125,12 +123,12 @@ public class IssueCommentService {
     // check authorization
     issueService.getByKey(comment.issueKey());
 
-    changeDao.delete(commentKey);
+    dbClient.issueChangeDao().delete(commentKey);
     return comment;
   }
 
   public IssueComment editComment(String commentKey, String text, UserSession userSession) {
-    DefaultIssueComment comment = changeDao.selectCommentByKey(commentKey);
+    DefaultIssueComment comment = dbClient.issueChangeDao().selectCommentByKey(commentKey);
     if (StringUtils.isBlank(text)) {
       throw new BadRequestException("Cannot add empty comments to an issue");
     }
@@ -147,7 +145,7 @@ public class IssueCommentService {
     IssueChangeDto dto = IssueChangeDto.of(comment);
     dto.setUpdatedAt(System2.INSTANCE.now());
     dto.setChangeData(text);
-    changeDao.update(dto);
+    dbClient.issueChangeDao().update(dto);
 
     return comment;
   }
index 76994dbf49bc58ddc771bbe6bf34337f6484fece..20a0bfeef22c2664ea6041fc74dd2d376dbc379d 100644 (file)
@@ -20,7 +20,6 @@
 
 package org.sonar.server.qualityprofile.db;
 
-import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
 import org.sonar.api.utils.System2;
 import org.sonar.core.persistence.DbSession;
@@ -36,6 +35,7 @@ import org.sonar.server.rule.db.RuleDao;
 import org.sonar.server.search.IndexDefinition;
 
 import javax.annotation.CheckForNull;
+
 import java.util.List;
 
 public class ActiveRuleDao extends BaseDao<ActiveRuleMapper, ActiveRuleDto, ActiveRuleKey> {
@@ -54,11 +54,6 @@ public class ActiveRuleDao extends BaseDao<ActiveRuleMapper, ActiveRuleDto, Acti
   private final RuleDao ruleDao;
   private final QualityProfileDao profileDao;
 
-  public ActiveRuleDao(QualityProfileDao profileDao, RuleDao ruleDao) {
-    this(profileDao, ruleDao, System2.INSTANCE);
-  }
-
-  @VisibleForTesting
   public ActiveRuleDao(QualityProfileDao profileDao, RuleDao ruleDao, System2 system) {
     super(IndexDefinition.ACTIVE_RULE, ActiveRuleMapper.class, system);
     this.ruleDao = ruleDao;
index 101659f898e3d37bbcc82dedd3a28f44c3a5705a..de1b473f1e117ccc88da0b4a80f675e0a87700f0 100644 (file)
@@ -19,7 +19,6 @@
  */
 package org.sonar.server.rule.db;
 
-import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Preconditions;
 import org.sonar.api.rule.RuleKey;
 import org.sonar.api.utils.System2;
@@ -31,15 +30,11 @@ import org.sonar.server.db.BaseDao;
 import org.sonar.server.search.IndexDefinition;
 
 import javax.annotation.CheckForNull;
+
 import java.util.List;
 
 public class RuleDao extends BaseDao<RuleMapper, RuleDto, RuleKey> {
 
-  public RuleDao() {
-    this(System2.INSTANCE);
-  }
-
-  @VisibleForTesting
   public RuleDao(System2 system) {
     super(IndexDefinition.RULE, RuleMapper.class, system);
   }
index 174dd1e959d970f34f692318840daebec8b45c82..56210ce94340f1f8ea482bf43c4430f22dc2b90d 100644 (file)
@@ -20,7 +20,6 @@
 
 package org.sonar.server.user.db;
 
-import com.google.common.annotations.VisibleForTesting;
 import org.sonar.api.utils.System2;
 import org.sonar.core.persistence.DbSession;
 import org.sonar.core.user.GroupDto;
@@ -34,11 +33,6 @@ import java.util.List;
  */
 public class GroupDao extends BaseDao<GroupMapper, GroupDto, String> {
 
-  public GroupDao() {
-    this(System2.INSTANCE);
-  }
-
-  @VisibleForTesting
   public GroupDao(System2 system) {
     super(GroupMapper.class, system);
   }
index 95ee7c4603e941b9c746f44c0871903cb6621228..7c377e4be94e0df456330a92301c74082e90bcd7 100644 (file)
@@ -20,9 +20,6 @@
 
 package org.sonar.server.batch;
 
-import java.io.ByteArrayInputStream;
-import java.util.Arrays;
-import javax.annotation.Nullable;
 import org.junit.After;
 import org.junit.Before;
 import org.junit.ClassRule;
@@ -56,6 +53,11 @@ import org.sonar.server.tester.UserSessionRule;
 import org.sonar.server.ws.WsTester;
 import org.sonar.test.DbTests;
 
+import javax.annotation.Nullable;
+
+import java.io.ByteArrayInputStream;
+import java.util.Arrays;
+
 import static com.google.common.collect.Lists.newArrayList;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.mockito.Mockito.mock;
@@ -91,12 +93,12 @@ public class IssuesActionTest {
     es.truncateIndices();
     this.session = db.myBatis().openSession(false);
 
-    DbClient dbClient = new DbClient(db.database(), db.myBatis(), new IssueDao(db.myBatis()), new ComponentDao());
+    componentDao = new ComponentDao(mock(System2.class));
+    DbClient dbClient = new DbClient(db.database(), db.myBatis(), new IssueDao(db.myBatis()), componentDao);
     issueIndex = new IssueIndex(es.client(), System2.INSTANCE, userSessionRule);
     issueIndexer = new IssueIndexer(null, es.client());
     issueAuthorizationIndexer = new IssueAuthorizationIndexer(null, es.client());
     issuesAction = new IssuesAction(dbClient, issueIndex, userSessionRule);
-    componentDao = new ComponentDao();
 
     tester = new WsTester(new BatchWs(new BatchIndex(mock(Server.class)), issuesAction));
   }
index 9ea902ced16ab63de685f9f677645655b58f6392..669c30545462f0ce3248c5634d836b4bc7a57791 100644 (file)
@@ -45,6 +45,7 @@ import org.sonar.server.ws.WsTester;
 import org.sonar.test.JsonAssert;
 
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.Mockito.mock;
 
 public class ProvisionedProjectsActionTest {
 
@@ -57,6 +58,7 @@ public class ProvisionedProjectsActionTest {
   DbClient dbClient;
   DbSession dbSession;
   ComponentDao componentDao;
+  System2 system2 = mock(System2.class);
 
   @After
   public void tearDown() {
@@ -65,7 +67,7 @@ public class ProvisionedProjectsActionTest {
 
   @Before
   public void setUp() {
-    dbClient = new DbClient(db.database(), db.myBatis(), new ComponentDao(), new SnapshotDao(System2.INSTANCE));
+    dbClient = new DbClient(db.database(), db.myBatis(), new ComponentDao(system2), new SnapshotDao(System2.INSTANCE));
     dbSession = dbClient.openSession(false);
     componentDao = dbClient.componentDao();
     db.truncateTables();
index c63ff362391d97e9c33a0751a7ea0bf53e5c7ff1..629b2ff864a33523a01115b51ce1f55bfb8ac8fe 100644 (file)
@@ -25,6 +25,7 @@ import org.junit.ClassRule;
 import org.junit.Rule;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
+import org.sonar.api.utils.System2;
 import org.sonar.api.web.UserRole;
 import org.sonar.core.persistence.DbTester;
 import org.sonar.core.user.AuthorizationDao;
@@ -53,7 +54,7 @@ public class SearchActionTest {
   public void setUp() {
     dbTester.truncateTables();
     DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(),
-      new ComponentDao(), new AuthorizationDao(dbTester.myBatis()), new ComponentIndexDao()
+      new ComponentDao(mock(System2.class)), new AuthorizationDao(dbTester.myBatis()), new ComponentIndexDao()
       );
     tester = new WsTester(new ComponentsWs(mock(ComponentAppAction.class), new SearchAction(dbClient, userSessionRule)));
   }
index ad635fbf7e7d92a602aee36f70300a3e372db0af..38967d78e622de1762229ce2f20748570cb9131b 100644 (file)
@@ -74,7 +74,7 @@ public class ComputationServiceTest {
 
   @Before
   public void setUp() {
-    DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao(), new SnapshotDao(system));
+    DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao(system), new SnapshotDao(system));
     sut = new ComputationService(dbClient, steps, activityService, settingsFactory, tempFolder, system);
 
     // db contains project with key "P1"
index ba5ec765781a4e6f065d86fc6d7babf693cac1ff..90655f3b337a51c6da39cd5a6df08a1da6179c29 100644 (file)
@@ -71,7 +71,7 @@ public class ReportQueueTest {
     settings.setProperty(ProcessProperties.PATH_DATA, dataDir.getAbsolutePath());
     when(system.now()).thenReturn(NOW);
 
-    DbClient dbClient = new DbClient(db.database(), db.myBatis(), new ComponentDao(), new AnalysisReportDao(system));
+    DbClient dbClient = new DbClient(db.database(), db.myBatis(), new ComponentDao(system), new AnalysisReportDao(system));
     sut = new ReportQueue(dbClient, settings);
 
     try (DbSession session = dbClient.openSession(false)) {
index 23fb6501a82939ccbc5e581967e03194d671a06d..d65db7e530710f938342228e0a815a2983527101 100644 (file)
@@ -23,6 +23,7 @@ import org.junit.Before;
 import org.junit.ClassRule;
 import org.junit.Test;
 import org.sonar.api.rule.RuleKey;
+import org.sonar.api.utils.System2;
 import org.sonar.core.persistence.DbTester;
 import org.sonar.server.db.DbClient;
 import org.sonar.server.rule.db.RuleDao;
@@ -31,6 +32,7 @@ import java.util.Collections;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.Assert.fail;
+import static org.mockito.Mockito.mock;
 
 public class RuleCacheLoaderTest {
 
@@ -45,7 +47,7 @@ public class RuleCacheLoaderTest {
   @Test
   public void load_by_key() {
     dbTester.prepareDbUnit(getClass(), "shared.xml");
-    DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new RuleDao());
+    DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new RuleDao(mock(System2.class)));
     RuleCacheLoader loader = new RuleCacheLoader(dbClient);
 
     assertThat(loader.load(RuleKey.of("squid", "R001")).getName()).isEqualTo("Rule One");
@@ -54,7 +56,7 @@ public class RuleCacheLoaderTest {
 
   @Test
   public void load_by_keys_is_not_supported() {
-    DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new RuleDao());
+    DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new RuleDao(mock(System2.class)));
     RuleCacheLoader loader = new RuleCacheLoader(dbClient);
     try {
       loader.loadAll(Collections.<RuleKey>emptyList());
index 80a1e0fc79debd2a420b2a067a787be4f1fe956a..0aa8bfa21f6f486cded26c40fceb14af2da0d60d 100644 (file)
@@ -69,7 +69,7 @@ public class PersistIssuesStepTest extends BaseStepTest {
   public void setup() throws Exception {
     dbTester.truncateTables();
     session = dbTester.myBatis().openSession(false);
-    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new IssueDao(dbTester.myBatis()), new RuleDao());
+    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new IssueDao(dbTester.myBatis()), new RuleDao(system2));
 
     issueCache = new IssueCache(temp.newFile(), System2.INSTANCE);
     system2 = mock(System2.class);
index a8ebac72f819c217ca8dd5c5889385e156fe0d96..d91e2e35fdae9e86eb8a37c8735db3b0f4bba078 100644 (file)
@@ -23,6 +23,7 @@ package org.sonar.server.computation.step;
 import org.junit.*;
 import org.sonar.api.config.Settings;
 import org.sonar.api.resources.Qualifiers;
+import org.sonar.api.utils.System2;
 import org.sonar.core.component.ComponentDto;
 import org.sonar.core.persistence.DbSession;
 import org.sonar.core.persistence.DbTester;
@@ -59,7 +60,7 @@ public class PurgeRemovedViewsStepTest extends BaseStepTest {
   public void setUp() {
     esTester.truncateIndices();
     session = db.myBatis().openSession(false);
-    dbClient = new DbClient(db.database(), db.myBatis(), new IssueDao(db.myBatis()), new ComponentDao());
+    dbClient = new DbClient(db.database(), db.myBatis(), new IssueDao(db.myBatis()), new ComponentDao(mock(System2.class)));
     sut = new PurgeRemovedViewsStep(new ViewIndex(esTester.client()), dbClient);
   }
 
index ecc362f865e80ba0a16764590da3b51b1269e0fd..b5771b3e0ebb77f1b4c12191b1b901267285bb1f 100644 (file)
@@ -46,6 +46,7 @@ import org.sonar.server.ws.WsTester;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.failBecauseExceptionWasNotThrown;
+import static org.mockito.Mockito.mock;
 
 public class ShowActionTest {
 
@@ -74,7 +75,7 @@ public class ShowActionTest {
   @Before
   public void setUp() {
     dbTester.truncateTables();
-    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao(), new SnapshotDao(System2.INSTANCE), new FileDependencyDao());
+    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao(mock(System2.class)), new SnapshotDao(System2.INSTANCE), new FileDependencyDao());
     session = dbClient.openSession(false);
     tester = new WsTester(new DependenciesWs(new ShowAction(dbClient, userSessionRule)));
     controller = tester.controller("api/dependencies");
index 49f18628b803c993ed7f1fcdc850341363aab730..c4921fddb2a9a876d8462d7d8f1044214e8f2ab5 100644 (file)
@@ -85,8 +85,9 @@ public class IssueCommentServiceTest {
   @Before
   public void setUp() {
     when(dbClient.openSession(false)).thenReturn(session);
+    when(dbClient.issueChangeDao()).thenReturn(changeDao);
 
-    issueCommentService = new IssueCommentService(dbClient, issueService, updater, changeDao);
+    issueCommentService = new IssueCommentService(dbClient, issueService, updater);
   }
 
   @Test
index ec94fb144457d7cfb030f0114ac57c0de9e7fd61..64045de06198b508315d1654a8a6782a1465c1d6 100644 (file)
@@ -67,13 +67,15 @@ public class QProfileDeleteActionTest {
 
   private DbSession session;
 
+  System2 system2 = mock(System2.class);
+
   @Before
   public void setUp() {
     dbTester.truncateTables();
     qualityProfileDao = new QualityProfileDao(dbTester.myBatis(), mock(System2.class));
-    componentDao = new ComponentDao();
+    componentDao = new ComponentDao(mock(System2.class));
 
-    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), qualityProfileDao, new ActiveRuleDao(qualityProfileDao, new RuleDao()));
+    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), qualityProfileDao, new ActiveRuleDao(qualityProfileDao, new RuleDao(system2), system2));
     session = dbClient.openSession(false);
 
     xoo1 = LanguageTesting.newLanguage("xoo1");
index 23fec4a8451656041c8af2874edc1650cb9d6cb3..865db5a8539bdecdabe0b1e46a43e2536f642336 100644 (file)
@@ -68,12 +68,14 @@ public class QProfileProjectsActionTest {
 
   private RoleDao roleDao;
 
+  System2 system2 = mock(System2.class);
+
   @Before
   public void setUp() {
     dbTester.truncateTables();
     dbClient = new DbClient(dbTester.database(), dbTester.myBatis(),
-      new QualityProfileDao(dbTester.myBatis(), mock(System2.class)),
-      new ComponentDao(),
+      new QualityProfileDao(dbTester.myBatis(), system2),
+      new ComponentDao(system2),
       new AuthorizationDao(dbTester.myBatis()));
     roleDao = new RoleDao();
     session = dbClient.openSession(false);
index 3b1dde59945faf4655d6747e78a8f2d477ca597e..4131fc8ab9fccbd98d61413c9e6e4663bf440ea5 100644 (file)
@@ -20,6 +20,7 @@
 package org.sonar.server.source.ws;
 
 import org.junit.Rule;
+import org.sonar.api.utils.System2;
 import org.sonar.server.source.db.FileSourceDao;
 
 import org.junit.After;
@@ -41,6 +42,7 @@ import org.sonar.server.ws.WsTester;
 import org.sonar.test.DbTests;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.junit.Assert.fail;
+import static org.mockito.Mockito.mock;
 
 @RunWith(MockitoJUnitRunner.class)
 @Category(DbTests.class)
@@ -63,7 +65,7 @@ public class HashActionTest {
     db.truncateTables();
     this.session = db.myBatis().openSession(false);
 
-    DbClient dbClient = new DbClient(db.database(), db.myBatis(), new FileSourceDao(db.myBatis()), new ComponentDao());
+    DbClient dbClient = new DbClient(db.database(), db.myBatis(), new FileSourceDao(db.myBatis()), new ComponentDao(mock(System2.class)));
 
     tester = new WsTester(new SourcesWs(new HashAction(dbClient, userSessionRule)));
   }
index b93a8a23b3d7422caed57850cdafb77454e8901c..462264000ab87c75b29bb35949a85813167ac936 100644 (file)
@@ -28,6 +28,7 @@ import org.junit.ClassRule;
 import org.junit.Rule;
 import org.junit.Test;
 import org.sonar.api.config.Settings;
+import org.sonar.api.utils.System2;
 import org.sonar.api.web.UserRole;
 import org.sonar.core.component.ComponentDto;
 import org.sonar.core.persistence.DbSession;
@@ -48,6 +49,7 @@ import org.sonar.server.ws.WsTester;
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.assertj.core.api.Assertions.failBecauseExceptionWasNotThrown;
 import static org.junit.Assert.fail;
+import static org.mockito.Mockito.mock;
 
 public class LinesActionTest {
 
@@ -79,7 +81,7 @@ public class LinesActionTest {
 
     htmlSourceDecorator = new HtmlSourceDecorator();
     sourceLineIndex = new SourceLineIndex(esTester.client());
-    componentDao = new ComponentDao();
+    componentDao = new ComponentDao(mock(System2.class));
     DbClient dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), componentDao);
     session = dbClient.openSession(false);
     wsTester = new WsTester(new SourcesWs(new LinesAction(dbClient, sourceLineIndex, htmlSourceDecorator, userSessionRule)));
index 25de3e5c916688044880cafc379706b532efd573..74fae06142ca3fc4a66e3c0540d3f42a318ad8b3 100644 (file)
@@ -28,6 +28,7 @@ import org.junit.Rule;
 import org.junit.Test;
 import org.sonar.api.config.Settings;
 import org.sonar.api.utils.DateUtils;
+import org.sonar.api.utils.System2;
 import org.sonar.api.web.UserRole;
 import org.sonar.core.component.ComponentDto;
 import org.sonar.core.persistence.DbSession;
@@ -43,6 +44,8 @@ import org.sonar.server.source.index.SourceLineIndexDefinition;
 import org.sonar.server.tester.UserSessionRule;
 import org.sonar.server.ws.WsTester;
 
+import static org.mockito.Mockito.mock;
+
 public class ScmActionTest {
 
   private static final String FILE_KEY = "FILE_KEY";
@@ -66,7 +69,7 @@ public class ScmActionTest {
   public void setUp() {
     dbTester.truncateTables();
     esTester.truncateIndices();
-    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao());
+    dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao(mock(System2.class)));
     session = dbClient.openSession(false);
 
     tester = new WsTester(new SourcesWs(new ScmAction(dbClient, new SourceLineIndex(esTester.client()), userSessionRule)));
index 27862aeb2b7409e0ce1cbcaad531f48f7770174b..d6901eaccb7cb4c81f956207482cfc1d833a673b 100644 (file)
@@ -28,6 +28,7 @@ import org.junit.ClassRule;
 import org.junit.Rule;
 import org.junit.Test;
 import org.sonar.api.config.Settings;
+import org.sonar.api.utils.System2;
 import org.sonar.api.web.UserRole;
 import org.sonar.core.component.ComponentDto;
 import org.sonar.core.persistence.DbSession;
@@ -43,6 +44,8 @@ import org.sonar.server.test.index.TestIndexDefinition;
 import org.sonar.server.tester.UserSessionRule;
 import org.sonar.server.ws.WsTester;
 
+import static org.mockito.Mockito.mock;
+
 public class TestsListActionTest {
   DbClient dbClient;
   DbSession dbSession;
@@ -59,7 +62,7 @@ public class TestsListActionTest {
 
   @Before
   public void setUp() {
-    dbClient = new DbClient(db.database(), db.myBatis(), new ComponentDao());
+    dbClient = new DbClient(db.database(), db.myBatis(), new ComponentDao(mock(System2.class)));
     dbSession = dbClient.openSession(false);
     db.truncateTables();
     es.truncateIndices();
index 744c742b77d2fe2fc53e93566ec52cf8beed70d1..fc2baa3dd1c370ccbdc7c40137b5986d947fbd38 100644 (file)
@@ -55,7 +55,7 @@ public class GroupDaoTest extends AbstractDaoTestCase {
   public void select_by_key() {
     setupData("select_by_key");
 
-    GroupDto group = new GroupDao().getByKey(session, "sonar-users");
+    GroupDto group = new GroupDao(system2).getByKey(session, "sonar-users");
     assertThat(group).isNotNull();
     assertThat(group.getId()).isEqualTo(1L);
     assertThat(group.getName()).isEqualTo("sonar-users");
index 407337ab5aa795b36517b9c098ef492c2cb06cdd..ee6c1e4b1720cb6dffa816d0327ecc1a7bcb3817 100644 (file)
@@ -84,7 +84,7 @@ public class ChangePasswordActionTest {
     System2 system2 = new System2();
     UserDao userDao = new UserDao(dbTester.myBatis(), system2);
     UserGroupDao userGroupDao = new UserGroupDao();
-    GroupDao groupDao = new GroupDao();
+    GroupDao groupDao = new GroupDao(system2);
     dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), userDao, userGroupDao, groupDao);
     session = dbClient.openSession(false);
     groupDao.insert(session, new GroupDto().setName("sonar-users"));
index aa41c94be68efbc6a7555dbab84fa9f0ccb38d2d..42a4008c7dd1d254baba7f8ad5625c27194cd412 100644 (file)
@@ -94,7 +94,7 @@ public class CreateActionTest {
     System2 system2 = new System2();
     UserDao userDao = new UserDao(dbTester.myBatis(), system2);
     UserGroupDao userGroupDao = new UserGroupDao();
-    GroupDao groupDao = new GroupDao();
+    GroupDao groupDao = new GroupDao(system2);
     dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), userDao, userGroupDao, groupDao);
     session = dbClient.openSession(false);
     groupDao.insert(session, new GroupDto().setName("sonar-users"));
index 84f19cd13a43116fa8728ac21eb5a605473a177c..6ed505a8d225e153a70ef194cc53be08a14f3800 100644 (file)
@@ -82,7 +82,7 @@ public class UpdateActionTest {
     System2 system2 = new System2();
     UserDao userDao = new UserDao(dbTester.myBatis(), system2);
     UserGroupDao userGroupDao = new UserGroupDao();
-    GroupDao groupDao = new GroupDao();
+    GroupDao groupDao = new GroupDao(system2);
     dbClient = new DbClient(dbTester.database(), dbTester.myBatis(), userDao, userGroupDao, groupDao);
     session = dbClient.openSession(false);
     groupDao.insert(session, new GroupDto().setName("sonar-users"));
index 21db4db9539d22660aebc57e05d1ef7bdf7f80c3..ccd5484e7137fd608b8c2b4859477958b81f28e0 100644 (file)
@@ -27,6 +27,7 @@ import org.junit.ClassRule;
 import org.junit.Test;
 import org.junit.experimental.categories.Category;
 import org.sonar.api.config.Settings;
+import org.sonar.api.utils.System2;
 import org.sonar.core.persistence.DbTester;
 import org.sonar.server.component.db.ComponentDao;
 import org.sonar.server.db.DbClient;
@@ -38,6 +39,7 @@ import java.util.Map;
 
 import static com.google.common.collect.Lists.newArrayList;
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.Mockito.mock;
 
 @Category(DbTests.class)
 public class ViewIndexerTest {
@@ -54,7 +56,7 @@ public class ViewIndexerTest {
   public void setUp() {
     dbTester.truncateTables();
     esTester.truncateIndices();
-    indexer = new ViewIndexer(new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao()), esTester.client());
+    indexer = new ViewIndexer(new DbClient(dbTester.database(), dbTester.myBatis(), new ComponentDao(mock(System2.class))), esTester.client());
     indexer.setEnabled(true);
   }
 
index 8fbddc73f12ae1f20d5563eee00699825c27d465..4267fc7c77b71e2c487cf0162111edc9878e4f1f 100644 (file)
@@ -68,12 +68,7 @@ public class DefaultI18n implements I18n, Startable {
   private ClassLoader classloader;
   private Map<String, String> propertyToBundles;
 
-  public DefaultI18n(PluginRepository pluginRepository) {
-    this(pluginRepository, System2.INSTANCE);
-  }
-
-  @VisibleForTesting
-  DefaultI18n(PluginRepository pluginRepository, System2 system2) {
+  public DefaultI18n(PluginRepository pluginRepository, System2 system2) {
     this.pluginRepository = pluginRepository;
     this.system2 = system2;
     // SONAR-2927