SONAR-11421 Add sha1 to ProjectAnalysis

SonarSource · Jan 8, 2019 · ea906c4 · ea906c4
1 parent ba01aad
commit ea906c4
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 2 deletions.
diff --git a/...java/org/sonar/ce/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutor.java b/...java/org/sonar/ce/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutor.java
@@ -125,7 +125,9 @@ private ProjectAnalysisImpl createProjectAnalysis(CeTask.Status status) {
       getAnalysis().map(a -> a.getDate().getTime()).orElse(system2.now()),
       ScannerContextImpl.from(reportReader.readContextProperties()),
       status == SUCCESS ? createQualityGate() : null,
-      createBranch());
+      createBranch(),
+      reportReader.readMetadata().getScmRevisionId()
+    );
   }
 
   @CheckForNull
@@ -223,10 +225,11 @@ private static class ProjectAnalysisImpl implements PostProjectAnalysisTask.Proj
     private final Branch branch;
     @Nullable
     private final Analysis analysis;
+    private final String scmRevisionId;
 
     private ProjectAnalysisImpl(@Nullable Organization organization, CeTask ceTask, Project project,
       @Nullable Analysis analysis, long date,
-      ScannerContext scannerContext, @Nullable QualityGate qualityGate, @Nullable Branch branch) {
+      ScannerContext scannerContext, @Nullable QualityGate qualityGate, @Nullable Branch branch, String scmRevisionId) {
       this.organization = organization;
       this.ceTask = requireNonNull(ceTask, "ceTask can not be null");
       this.project = requireNonNull(project, "project can not be null");
@@ -235,6 +238,7 @@ private ProjectAnalysisImpl(@Nullable Organization organization, CeTask ceTask,
       this.scannerContext = requireNonNull(scannerContext, "scannerContext can not be null");
       this.qualityGate = qualityGate;
       this.branch = branch;
+      this.scmRevisionId = scmRevisionId;
     }
 
     @Override
@@ -283,6 +287,11 @@ public ScannerContext getScannerContext() {
       return scannerContext;
     }
 
+    @Override
+    public String getScmRevisionId() {
+      return scmRevisionId;
+    }
+
     @Override
     public String toString() {
       return "ProjectAnalysis{" +

diff --git a/.../org/sonar/ce/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutorTest.java b/.../org/sonar/ce/task/projectanalysis/api/posttask/PostProjectAnalysisTasksExecutorTest.java
@@ -114,6 +114,8 @@ public void setUp() throws Exception {
       .setOrganizationsEnabled(new Random().nextBoolean())
       .setOrganization(Organization.from(
         new OrganizationDto().setKey(organizationKey).setName(organizationName).setUuid(organizationUuid).setDefaultQualityGateUuid("foo")));
+
+    reportReader.setMetadata(ScannerReport.Metadata.newBuilder().build());
   }
 
   @Test

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/ce/posttask/PostProjectAnalysisTask.java b/sonar-plugin-api/src/main/java/org/sonar/api/ce/posttask/PostProjectAnalysisTask.java
@@ -122,5 +122,10 @@ interface ProjectAnalysis {
      * @since 6.1
      */
     ScannerContext getScannerContext();
+
+    /**
+     * Revision Id that has been analysed
+     */
+    String getScmRevisionId();
   }
 }
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/ce/posttask/PostProjectAnalysisTaskTester.java b/sonar-plugin-api/src/main/java/org/sonar/api/ce/posttask/PostProjectAnalysisTaskTester.java
@@ -781,6 +781,11 @@ public ScannerContext getScannerContext() {
           return scannerContext;
         }
 
+        @Override
+        public String getScmRevisionId() {
+          return null;
+        }
+
         @Override
         public String toString() {
           return "ProjectAnalysis{" +