fix(bitbucketdatacenter): ignore branch deletion events in ParsePayload

When a branch is deleted in Bitbucket Data Center, the repo:refs_changed
webhook sends a change with toHash set to all-zeros. Previously, Pipelines
as Code treated this like a push event, which caused errors when trying
to validate a missing commit.

This patch detects and ignores such events early in ParsePayload.

Fixes #2035

Author: infernus01

pkg/provider/bitbucketdatacenter/parse_payload.go

@@ -15,8 +15,7 @@ import (
 	"github.com/openshift-pipelines/pipelines-as-code/pkg/provider/bitbucketdatacenter/types"
 )
 
-// checkValidPayload checks if the payload is valid.
-func checkValidPayload(e *types.PullRequestEvent) error {
+func checkValidPullRequestPayload(e *types.PullRequestEvent) error {
 	if e.PullRequest.ToRef.Repository.Project == nil {
 		return fmt.Errorf("bitbucket toRef project is nil")
 	}
@@ -132,7 +131,7 @@ func (v *Provider) ParsePayload(_ context.Context, run *params.Run, request *htt
 			processedEvent.TriggerComment = e.Comment.Text
 		}
 
-		if err := checkValidPayload(e); err != nil {
+		if err := checkValidPullRequestPayload(e); err != nil {
 			return nil, err
 		}
 
@@ -164,6 +163,13 @@ func (v *Provider) ParsePayload(_ context.Context, run *params.Run, request *htt
 			return nil, fmt.Errorf("push event contains no commits under 'changes'; cannot proceed")
 		}
 
+		// Check for branch deletion - if any change is a DELETE type with zero hash, skip processing
+		for _, change := range e.Changes {
+			if provider.IsZeroSHA(change.ToHash) && change.Type == "DELETE" {
+				return nil, nil
+			}
+		}
+
 		if len(e.Commits) == 0 {
 			return nil, fmt.Errorf("push event contains no commits; cannot proceed")
 		}

pkg/provider/bitbucketdatacenter/parse_payload_test.go

@@ -545,7 +545,7 @@ func TestCheckValidPayload(t *testing.T) {
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			err := checkValidPayload(&tt.payloadEvent)
+			err := checkValidPullRequestPayload(&tt.payloadEvent)
 			if tt.wantErrString != "" {
 				assert.ErrorContains(t, err, tt.wantErrString)
 				return
@@ -709,6 +709,19 @@ func TestParsePayload(t *testing.T) {
 			expEvent:     ev1,
 			wantSHA:      "abcd",
 		},
+		{
+			name:      "branch/deleted with zero hash",
+			eventType: "repo:refs_changed",
+			payloadEvent: bbv1test.MakePushEvent(ev1, []types.PushRequestEventChange{
+				{
+					ToHash: "0000000000000000000000000000000000000000",
+					RefID:  "refs/heads/feature-branch",
+					Type:   "DELETE",
+				},
+			}, []types.Commit{},
+			),
+			expEvent: nil,
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
@@ -735,10 +748,15 @@ func TestParsePayload(t *testing.T) {
 				return
 			}
 			assert.NilError(t, err)
-
+			// Handle case where expEvent is nil (e.g., branch deletion)
+			if tt.expEvent == nil {
+				if got != nil {
+					t.Fatalf("expected event to be nil, got: %+v", got)
+				}
+				return
+			}
 			// assert SHA ID
 			assert.Equal(t, tt.wantSHA, got.SHA)
-
 			assert.Equal(t, got.AccountID, tt.expEvent.AccountID)
 
 			// test that we got slashed