Fix branch matching when there is a slashes in the base branch name

openshift-pipelines · Aug 3, 2023 · e1c8610 · e1c8610
1 parent 78c0159
commit e1c8610
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 4 deletions.
diff --git a/pkg/matcher/annotation_matcher.go b/pkg/matcher/annotation_matcher.go
@@ -26,7 +26,7 @@ const (
 
 func branchMatch(prunBranch, baseBranch string) bool {
 	// if target is refs/heads/.. and base is without ref (for pullRequest)
-	if strings.HasPrefix(prunBranch, "refs/heads") && !strings.Contains(baseBranch, "/") {
+	if strings.HasPrefix(prunBranch, "refs/heads") && !strings.Contains(baseBranch, "refs/heads") {
 		ref := "refs/heads/" + baseBranch
 		g := glob.MustCompile(prunBranch)
 		if g.Match(ref) {
@@ -35,7 +35,7 @@ func branchMatch(prunBranch, baseBranch string) bool {
 	}
 
 	// if base is refs/heads/.. and target is without ref (for push rerequested action)
-	if strings.HasPrefix(baseBranch, "refs/heads") && !strings.Contains(prunBranch, "/") {
+	if strings.HasPrefix(baseBranch, "refs/heads") && !strings.Contains(prunBranch, "refs/heads") {
 		prunRef := "refs/heads/" + prunBranch
 		g := glob.MustCompile(prunRef)
 		if g.Match(baseBranch) {

diff --git a/pkg/matcher/annotation_matcher_test.go b/pkg/matcher/annotation_matcher_test.go
@@ -833,6 +833,16 @@ func TestMatchPipelinerunByAnnotation(t *testing.T) {
 		},
 	}
 
+	pipelineWithSlashInBranchName := &tektonv1.PipelineRun{
+		ObjectMeta: metav1.ObjectMeta{
+			Name: "pipeline-withslashesinbranch",
+			Annotations: map[string]string{
+				keys.OnEvent:        "[pull_request, push]",
+				keys.OnTargetBranch: "[test/main]",
+			},
+		},
+	}
+
 	pipelineRefAll := &tektonv1.PipelineRun{
 		ObjectMeta: metav1.ObjectMeta{
 			Name: "pipeline-other",
@@ -1061,21 +1071,37 @@ func TestMatchPipelinerunByAnnotation(t *testing.T) {
 			wantErr: false,
 		},
 		{
-			name: "not-match-push-branch-matching",
+			name: "branch-matching-doesnot-match-for-push-event",
 			args: args{
 				runevent: info.Event{TriggerTarget: "push", EventType: "push", BaseBranch: "refs/heads/someothername/then/main"},
 				pruns:    []*tektonv1.PipelineRun{pipelineGood, pipelinePush},
 			},
 			wantErr: true,
 		},
 		{
-			name: "not-match-pull-request-branch-matching",
+			name: "branch-matching-doesnot-match-for-pull-request",
 			args: args{
 				runevent: info.Event{TriggerTarget: "pull_request", EventType: "pull_request", BaseBranch: "someothername/then/main"},
 				pruns:    []*tektonv1.PipelineRun{pipelineGood, pipelinePush},
 			},
 			wantErr: true,
 		},
+		{
+			name: "branch-matching-match-for-push-when-there-are-slashes-in-between-branch-name",
+			args: args{
+				runevent: info.Event{TriggerTarget: "push", EventType: "push", BaseBranch: "refs/heads/test/main"},
+				pruns:    []*tektonv1.PipelineRun{pipelineWithSlashInBranchName},
+			},
+			wantErr: false,
+		},
+		{
+			name: "branch-matching-match-for-pull_request-when-there-are-slashes-in-between-branch-name",
+			args: args{
+				runevent: info.Event{TriggerTarget: "pull_request", EventType: "pull_request", BaseBranch: "refs/heads/test/main"},
+				pruns:    []*tektonv1.PipelineRun{pipelineWithSlashInBranchName},
+			},
+			wantErr: false,
+		},
 	}
 
 	for _, tt := range tests {