added ignoreTypeInStatusName and tests

Author: mifitous

src/main/java/io/jenkins/plugins/gitlabbranchsource/BuildStatusNameCustomPartTrait.java

@@ -16,6 +16,7 @@ public class BuildStatusNameCustomPartTrait extends SCMSourceTrait {
     private String buildStatusNameCustomPart = "";
 
     private boolean buildStatusNameOverwrite;
+    private boolean ignoreTypeInStatusName;
 
     /**
      * Constructor for stapler.
@@ -38,12 +39,18 @@ public void setBuildStatusNameOverwrite(@NonNull Boolean buildStatusNameOverwrit
         this.buildStatusNameOverwrite = buildStatusNameOverwrite;
     }
 
+    @DataBoundSetter
+    public void setIgnoreTypeInStatusName(@NonNull Boolean ignoreTypeInStatusName) {
+        this.ignoreTypeInStatusName = ignoreTypeInStatusName;
+    }
+
     @Override
     protected void decorateContext(SCMSourceContext<?, ?> context) {
         if (context instanceof GitLabSCMSourceContext) {
             GitLabSCMSourceContext ctx = (GitLabSCMSourceContext) context;
             ctx.withBuildStatusNameCustomPart(getBuildStatusNameCustomPart());
             ctx.withBuildStatusNameOverwrite(getBuildStatusNameOverwrite());
+            ctx.withIgnoreTypeInStatusName(getIgnoreTypeInStatusName());
         }
     }
 
@@ -62,7 +69,18 @@ public String getBuildStatusNameCustomPart() {
      *
      * @return build status name overwrite option
      */
-    public boolean getBuildStatusNameOverwrite() { return buildStatusNameOverwrite; }
+    public boolean getBuildStatusNameOverwrite() {
+        return buildStatusNameOverwrite;
+    }
+
+    /**
+     * Getter method for the build status name overwrite
+     *
+     * @return build status name overwrite option
+     */
+    public boolean getIgnoreTypeInStatusName() {
+        return ignoreTypeInStatusName;
+    }
 
     /**
      * Our descriptor.

src/main/java/io/jenkins/plugins/gitlabbranchsource/GitLabSCMSourceContext.java

@@ -53,6 +53,8 @@ public class GitLabSCMSourceContext
 
     private boolean buildStatusNameOverwrite;
 
+    private boolean ignoreTypeInStatusName;
+
     private boolean alwaysBuildMROpen = true;
 
     private boolean alwaysBuildMRReOpen = true;
@@ -190,6 +192,10 @@ public boolean getBuildStatusNameOverwrite() {
         return buildStatusNameOverwrite;
     }
 
+    public boolean getIgnoreTypeInStatusName() {
+        return ignoreTypeInStatusName;
+    }
+
     @NonNull
     public GitLabSCMSourceContext wantBranches(boolean include) {
         wantBranches = wantBranches || include;
@@ -302,6 +308,11 @@ public final GitLabSCMSourceContext withBuildStatusNameOverwrite(final Boolean b
         return this;
     }
 
+    public final GitLabSCMSourceContext withIgnoreTypeInStatusName(final Boolean ignoreTypeInStatusName) {
+        this.ignoreTypeInStatusName = ignoreTypeInStatusName;
+        return this;
+    }
+
     @NonNull
     @Override
     public GitLabSCMSourceRequest newRequest(@NonNull SCMSource source,

src/main/java/io/jenkins/plugins/gitlabbranchsource/helpers/GitLabPipelineStatusNotifier.java

@@ -103,7 +103,7 @@ static String getStatusName(final GitLabSCMSourceContext sourceContext, final St
         if (revision instanceof BranchSCMRevision) {
             type = "branch";
         } else if (revision instanceof MergeRequestSCMRevision) {
-            type = getMrBuildName((MergeRequestSCMRevision)revision);
+            type = getMrBuildName((MergeRequestSCMRevision) revision);
         } else if (revision instanceof GitTagSCMRevision) {
             type = "tag";
         } else {
@@ -112,16 +112,31 @@ static String getStatusName(final GitLabSCMSourceContext sourceContext, final St
                     + revision.getClass().getName() + ", append" + type + " to status name");
         }
 
+        // Some comments regarding this implementation :
+        // Old code was :
+        // GITLAB_PIPELINE_STATUS_PREFIX + GITLAB_PIPELINE_STATUS_DELIMITER + (optional:
+        // pipelinePrefix + GITLAB_PIPELINE_STATUS_DELIMITER) + type
+        // New implementation:
+        // 1- BuildStatusNameCustomPart empty : use default
+        // GITLAB_PIPELINE_STATUS_PREFIX
+        // 2- else: depending on BuildStatusNameOverwrite : use
+        // BuildStatusNameCustomPart appended to GITLAB_PIPELINE_STATUS_PREFIX or use it
+        // as is
+        // 3- add type only if required (no ignoreTypeInStatusName)
+
         String pipelinePrefix = sourceContext.getBuildStatusNameCustomPart().trim();
         if (!pipelinePrefix.isEmpty()) {
-            pipelinePrefix = envVars.expand(pipelinePrefix) + GITLAB_PIPELINE_STATUS_DELIMITER;
+            pipelinePrefix = envVars.expand(pipelinePrefix);
             if (!sourceContext.getBuildStatusNameOverwrite()) {
                 pipelinePrefix = GITLAB_PIPELINE_STATUS_PREFIX + GITLAB_PIPELINE_STATUS_DELIMITER + pipelinePrefix;
             }
         } else {
-            pipelinePrefix = GITLAB_PIPELINE_STATUS_PREFIX + GITLAB_PIPELINE_STATUS_DELIMITER;
+            pipelinePrefix = GITLAB_PIPELINE_STATUS_PREFIX;
+        }
+        if (!sourceContext.getIgnoreTypeInStatusName()) {
+            pipelinePrefix = pipelinePrefix + GITLAB_PIPELINE_STATUS_DELIMITER + type;
         }
-        final String statusName = pipelinePrefix + type;
+        final String statusName = pipelinePrefix;
         LOGGER.log(Level.FINEST, () -> "Retrieved status name is: " + statusName);
         return statusName;
     }

src/main/resources/io/jenkins/plugins/gitlabbranchsource/BuildStatusNameCustomPartTrait/config.jelly

@@ -6,4 +6,7 @@
   <f:entry title="${%Overwrite full name}" field="buildStatusNameOverwrite">
     <f:checkbox default="unchecked"/>
   </f:entry>
+  <f:entry title="${%Do not append type in status name}" field="ignoreTypeInStatusName">
+    <f:checkbox default="unchecked"/>
+  </f:entry>
 </j:jelly>

src/test/java/io/jenkins/plugins/gitlabbranchsource/helpers/GitLabPipelineStatusNotifierTest.java

@@ -40,12 +40,46 @@ public void should_set_branch_status_name() {
                 + "branch"));
     }
 
+    @Test
+    public void should_set_branch_status_name_withBuildStatusNameCustomPart() {
+        GitLabSCMSourceContext sourceContext = new GitLabSCMSourceContext(null, null);
+        sourceContext.withBuildStatusNameCustomPart("CUSTOM");
+        sourceContext.withBuildStatusNameOverwrite(false);
+
+        BranchSCMHead head = new BranchSCMHead("head");
+        SCMRevision revision = new BranchSCMRevision(head, "hash");
+
+        String statusName = GitLabPipelineStatusNotifier.getStatusName(sourceContext, null, revision,
+                new hudson.EnvVars());
+
+        assertThat(statusName, is(GitLabPipelineStatusNotifier.GITLAB_PIPELINE_STATUS_PREFIX
+                + GitLabPipelineStatusNotifier.GITLAB_PIPELINE_STATUS_DELIMITER
+                + "CUSTOM"
+                + GitLabPipelineStatusNotifier.GITLAB_PIPELINE_STATUS_DELIMITER
+                + "branch"));
+    }
+
+    @Test
+    public void should_set_branch_status_name_withIgnoreTypeInStatusName() {
+        GitLabSCMSourceContext sourceContext = new GitLabSCMSourceContext(null, null);
+        sourceContext.withIgnoreTypeInStatusName(true);
+
+        BranchSCMHead head = new BranchSCMHead("head");
+        SCMRevision revision = new BranchSCMRevision(head, "hash");
+
+        String statusName = GitLabPipelineStatusNotifier.getStatusName(sourceContext, null, revision,
+                new hudson.EnvVars());
+
+        assertThat(statusName, is(GitLabPipelineStatusNotifier.GITLAB_PIPELINE_STATUS_PREFIX));
+    }
+
     @Test
     public void should_set_merge_request_head_status_name() {
         GitLabSCMSourceContext sourceContext = new GitLabSCMSourceContext(null, null);
 
         BranchSCMHead targetHead = new BranchSCMHead("target");
-        MergeRequestSCMHead head = new MergeRequestSCMHead("head", 0, targetHead, ChangeRequestCheckoutStrategy.HEAD, null, null, null, null, null);
+        MergeRequestSCMHead head = new MergeRequestSCMHead("head", 0, targetHead, ChangeRequestCheckoutStrategy.HEAD,
+                null, null, null, null, null);
 
         BranchSCMRevision target = new BranchSCMRevision(targetHead, "target-hash");
         BranchSCMRevision source = new BranchSCMRevision(new BranchSCMHead("source"), "source-hash");
@@ -64,7 +98,8 @@ public void should_set_merge_request_merge_status_name() {
         GitLabSCMSourceContext sourceContext = new GitLabSCMSourceContext(null, null);
 
         BranchSCMHead targetHead = new BranchSCMHead("target");
-        MergeRequestSCMHead head = new MergeRequestSCMHead("head", 0, targetHead, ChangeRequestCheckoutStrategy.MERGE, null, null, null, null, null);
+        MergeRequestSCMHead head = new MergeRequestSCMHead("head", 0, targetHead, ChangeRequestCheckoutStrategy.MERGE,
+                null, null, null, null, null);
 
         BranchSCMRevision target = new BranchSCMRevision(targetHead, "target-hash");
         BranchSCMRevision source = new BranchSCMRevision(new BranchSCMHead("source"), "source-hash");