fix: Match source and head branch in full path

before this, when source or head branch in cancel-in-progress
matching code having full path weren't being matched.
e.g. refs/heads/branch

Signed-off-by: Zaki Shaikh <zashaikh@redhat.com>

Author: zakisk

pkg/pipelineascode/cancel_pipelineruns.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"fmt"
 	"strconv"
+	"strings"
 	"sync"
 
 	tektonv1 "github.com/tektoncd/pipeline/pkg/apis/pipeline/v1"
@@ -52,7 +53,7 @@ func (p *PacRun) cancelAllInProgressBelongingToClosedPullRequest(ctx context.Con
 		// Note: The 'selection.NotIn' operator is used to exclude PipelineRuns that have the
 		// 'cancel-in-progress' annotation explicitly set to 'false', effectively opting them out of cancellation.
 		labelsMap = map[string]string{keys.CancelInProgress: "false"}
-		operator = selection.NotIn
+		operator = selection.NotIn //codespell:ignore 'NotIn'
 	} else {
 		// When the 'cancel-in-progress' setting is disabled globally via the Pipelines-as-Code ConfigMap,
 		// filter and list only those PipelineRuns that explicitly override the global setting by having the
@@ -153,7 +154,7 @@ func (p *PacRun) cancelInProgressMatchingPipelineRun(ctx context.Context, matchP
 			// it means we only cancel pipelinerun of the same name that runs to
 			// the unique branch. Note: HeadBranch is the branch from where the PR
 			// comes from in git jargon.
-			if sourceBranch != p.event.HeadBranch {
+			if strings.TrimPrefix(sourceBranch, "refs/heads/") != strings.TrimPrefix(p.event.HeadBranch, "refs/heads/") {
 				p.logger.Infof("cancel-in-progress: skipping pipelinerun %v/%v as it is not from the same branch, annotation source-branch: %s event headbranch: %s", pr.GetNamespace(), pr.GetName(), sourceBranch, p.event.HeadBranch)
 				return false
 			}

pkg/pipelineascode/cancel_pipelineruns_test.go

@@ -1033,6 +1033,62 @@ func TestCancelInProgressMatchingPipelineRun(t *testing.T) {
 			},
 			wantLog: "cancel-in-progress for event push is enabled via PipelineRun annotation",
 		},
+		{
+			name: "matching PipelineRun when source branch annotation is having full path refs/heads/",
+			event: &info.Event{
+				Repository:        "foo",
+				SHA:               "foosha",
+				HeadBranch:        "head",
+				EventType:         string(triggertype.Push),
+				TriggerTarget:     triggertype.Push,
+				PullRequestNumber: pullReqNumber,
+			},
+			pipelineRuns: []*pipelinev1.PipelineRun{
+				{
+					ObjectMeta: metav1.ObjectMeta{
+						Name:      "pr-foo-1",
+						Namespace: "foo",
+						Labels: map[string]string{
+							keys.EventType:      string(triggertype.Push),
+							keys.OriginalPRName: "pr-foo",
+							keys.URLRepository:  formatting.CleanValueKubernetes("foo"),
+							keys.SHA:            formatting.CleanValueKubernetes("foosha"),
+						},
+						Annotations: map[string]string{
+							keys.OriginalPRName:   "pr-foo",
+							keys.Repository:       "foo",
+							keys.SourceBranch:     "refs/heads/head",
+							keys.CancelInProgress: "true",
+						},
+					},
+					Spec: pipelinev1.PipelineRunSpec{},
+				},
+				{
+					ObjectMeta: metav1.ObjectMeta{
+						Name:      "pr-foo-2",
+						Namespace: "foo",
+						Labels: map[string]string{
+							keys.OriginalPRName: "pr-foo",
+							keys.URLRepository:  formatting.CleanValueKubernetes("foo"),
+							keys.SHA:            formatting.CleanValueKubernetes("foosha"),
+							keys.EventType:      string(triggertype.Push),
+						},
+						Annotations: map[string]string{
+							keys.OriginalPRName:   "pr-foo",
+							keys.Repository:       "foo",
+							keys.SourceBranch:     "head",
+							keys.CancelInProgress: "true",
+						},
+					},
+					Spec: pipelinev1.PipelineRunSpec{},
+				},
+			},
+			repo: fooRepo,
+			cancelledPipelineRuns: map[string]bool{
+				"pr-foo-2": true,
+			},
+			wantLog: "cancel-in-progress for event push is enabled via PipelineRun annotation",
+		},
 		{
 			name: "skip/cancel in progress with concurrency limit",
 			event: &info.Event{
@@ -1466,8 +1522,8 @@ func TestGetLabelSelector(t *testing.T) {
 			labelsMap: map[string]string{
 				keys.CancelInProgress: "false",
 			},
-			operator: selection.NotIn,
-			want:     "pipelinesascode.tekton.dev/cancel-in-progress notin (false)",
+			operator: selection.NotIn,                                               //codespell:ignore 'NotIn'
+			want:     "pipelinesascode.tekton.dev/cancel-in-progress notin (false)", //codespell:ignore 'NotIn'
 		},
 	}