From: Julien HENRY <julien.henry@sonarsource.com>
Date: Thu, 8 Jan 2015 09:05:16 +0000 (+0100)
Subject: SONAR-6014 Remove dependency on project ID.
X-Git-Tag: latest-silver-master-#65~300
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=f87b00be24a240481bfc1f761a039851bb8c0649;p=sonarqube.git

SONAR-6014 Remove dependency on project ID.
---

diff --git a/sonar-batch/src/main/java/org/sonar/batch/components/TimeMachineConfiguration.java b/sonar-batch/src/main/java/org/sonar/batch/components/TimeMachineConfiguration.java
index f5045866558..6baa999b22f 100644
--- a/sonar-batch/src/main/java/org/sonar/batch/components/TimeMachineConfiguration.java
+++ b/sonar-batch/src/main/java/org/sonar/batch/components/TimeMachineConfiguration.java
@@ -74,7 +74,7 @@ public class TimeMachineConfiguration implements BatchExtension {
   private Snapshot findSnapshot(Snapshot projectSnapshot) {
     String hql = "from " + Snapshot.class.getSimpleName() + " where resourceId=:resourceId and (rootId=:rootSnapshotId or id=:rootSnapshotId)";
     List<Snapshot> snapshots = session.createQuery(hql)
-      .setParameter("resourceId", project.getId())
+      .setParameter("resourceId", projectSnapshot.getResourceId())
       .setParameter("rootSnapshotId", projectSnapshot.getId())
       .setMaxResults(1)
       .getResultList();
diff --git a/sonar-batch/src/test/java/org/sonar/batch/components/TimeMachineConfigurationTest.java b/sonar-batch/src/test/java/org/sonar/batch/components/TimeMachineConfigurationTest.java
index 7e61880fffc..8af5c78517f 100644
--- a/sonar-batch/src/test/java/org/sonar/batch/components/TimeMachineConfigurationTest.java
+++ b/sonar-batch/src/test/java/org/sonar/batch/components/TimeMachineConfigurationTest.java
@@ -46,11 +46,12 @@ public class TimeMachineConfigurationTest extends AbstractDbUnitTestCase {
   public void get_project_past_snapshot() {
     Snapshot projectSnapshot = new Snapshot();
     projectSnapshot.setId(1010);
+    projectSnapshot.setResourceId(1);
     PastSnapshot projectPastSnapshot = new PastSnapshot("mode", new Date(), projectSnapshot);
 
     when(periodsDefinition.getRootProjectPastSnapshots()).thenReturn(newArrayList(projectPastSnapshot));
 
-    TimeMachineConfiguration timeMachineConfiguration = new TimeMachineConfiguration(getSession(), (Project) new Project("my:project").setId(1), periodsDefinition);
+    TimeMachineConfiguration timeMachineConfiguration = new TimeMachineConfiguration(getSession(), (Project) new Project("my:project"), periodsDefinition);
     assertThat(timeMachineConfiguration.periods()).hasSize(1);
     assertThat(timeMachineConfiguration.periods().get(0).getDate()).isNotNull();
     assertThat(timeMachineConfiguration.getProjectPastSnapshots()).hasSize(1);
@@ -61,11 +62,12 @@ public class TimeMachineConfigurationTest extends AbstractDbUnitTestCase {
   public void get_module_past_snapshot() {
     Snapshot projectSnapshot = new Snapshot();
     projectSnapshot.setId(1010);
+    projectSnapshot.setResourceId(2);
     PastSnapshot projectPastSnapshot = new PastSnapshot("mode", new Date(), projectSnapshot);
 
     when(periodsDefinition.getRootProjectPastSnapshots()).thenReturn(newArrayList(projectPastSnapshot));
 
-    TimeMachineConfiguration timeMachineConfiguration = new TimeMachineConfiguration(getSession(), (Project) new Project("my:module").setId(2), periodsDefinition);
+    TimeMachineConfiguration timeMachineConfiguration = new TimeMachineConfiguration(getSession(), (Project) new Project("my:module"), periodsDefinition);
     assertThat(timeMachineConfiguration.periods()).hasSize(1);
     assertThat(timeMachineConfiguration.periods().get(0).getDate()).isNotNull();
     assertThat(timeMachineConfiguration.getProjectPastSnapshots()).hasSize(1);