Indicates to Gerrit that the review is posted by an automated system

jenkinsci · Jan 31, 2022 · f2cf40f · f2cf40f
1 parent c4ff5c4
commit f2cf40f
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 2 deletions.
diff --git a/src/main/java/org/jenkinsci/plugins/sonargerrit/gerrit/GerritReviewBuilder.java b/src/main/java/org/jenkinsci/plugins/sonargerrit/gerrit/GerritReviewBuilder.java
@@ -46,6 +46,7 @@ public ReviewInput buildReview() {
     // review
     String reviewMessage = getReviewMessage(finalIssuesToComment);
     ReviewInput reviewInput = new ReviewInput().message(reviewMessage);
+    reviewInput.tag = "autogenerated:sonar";
 
     switch (reviewConfig.getCommentType()) {
       case STANDARD:

diff --git a/...s/sonargerrit/gerrit/CommentTypeTest.java → ...lugins/sonargerrit/gerrit/ReviewTest.java b/...s/sonargerrit/gerrit/CommentTypeTest.java → ...lugins/sonargerrit/gerrit/ReviewTest.java
@@ -31,7 +31,7 @@
 
 /** @author Réda Housni Alaoui */
 @EnableCluster
-class CommentTypeTest {
+class ReviewTest {
 
   private static final String MAVEN_TARGET =
       "clean verify sonar:sonar "
@@ -45,7 +45,7 @@ class CommentTypeTest {
   @BeforeAll
   static void beforeAll(Cluster cluster, @TempDir Path workTree) throws Exception {
 
-    CommentTypeTest.cluster = cluster;
+    ReviewTest.cluster = cluster;
 
     git = GerritGit.createAndCloneRepository(cluster.gerrit(), workTree);
 
@@ -122,6 +122,19 @@ void test2() throws Exception {
             });
   }
 
+  @Test
+  @DisplayName("Review tag is autogenerated:sonar")
+  void test3() throws Exception {
+    GerritChange change = createChangeViolatingS1186();
+    triggerAndAssertSuccess(createPipelineJob(change, ReviewCommentType.STANDARD));
+
+    ChangeInfo changeDetail = change.getDetail();
+    assertThat(changeDetail.labels.get(GerritServer.CODE_QUALITY_LABEL).all)
+        .hasSize(1)
+        .map(approvalInfo -> approvalInfo.tag)
+        .containsExactly("autogenerated:sonar");
+  }
+
   private GerritChange createChangeViolatingS1186()
       throws GitAPIException, IOException, RestApiException {
     git.addAndCommitFile(