fix(bitbucket-cloud): assign unique statue name in Bitbucket Cloud

Update Bitbucket Cloud provider to use GetCheckName helper for commit
status key. This enables unique status reporting per PipelineRun by
using the format "ApplicationName / PipelineRunName" instead of just
the application name.

Changes:
- Replace hardcoded ApplicationName with GetCheckName in CreateStatus
- Update tests to include OriginalPipelineRunName in StatusOpts
- Add assertion in MuxCreateCommitstatus to verify correct key format

Fixes: #2237

JIRA: https://issues.redhat.com/browse/SRVKP-9636

Signed-off-by: Zaki Shaikh <[email protected]>

Author: zakisk

pkg/provider/bitbucketcloud/bitbucket.go

@@ -109,7 +109,7 @@ func (v *Provider) CreateStatus(_ context.Context, event *info.Event, statusopts
 	}
 
 	cso := &bitbucket.CommitStatusOptions{
-		Key:         v.pacInfo.ApplicationName,
+		Key:         provider.GetCheckName(statusopts, v.pacInfo),
 		Url:         detailsURL,
 		State:       statusopts.Conclusion,
 		Description: statusopts.Title,

pkg/provider/bitbucketcloud/bitbucket_test.go

@@ -273,6 +273,7 @@ func TestGetCommitInfo(t *testing.T) {
 }
 
 func TestCreateStatus(t *testing.T) {
+	originalPipelineRunName := "hello-af9ch"
 	tests := []struct {
 		name                  string
 		wantErr               bool
@@ -283,60 +284,68 @@ func TestCreateStatus(t *testing.T) {
 		{
 			name: "skipped",
 			status: provider.StatusOpts{
-				Conclusion: "skipped",
+				Conclusion:              "skipped",
+				OriginalPipelineRunName: originalPipelineRunName,
 			},
 			expectedDescSubstr: "Skipping",
 		},
 		{
 			name: "neutral",
 			status: provider.StatusOpts{
-				Conclusion: "neutral",
+				Conclusion:              "neutral",
+				OriginalPipelineRunName: originalPipelineRunName,
 			},
 			expectedDescSubstr: "stopped",
 		},
 		{
 			name: "completed with comment",
 			status: provider.StatusOpts{
-				Conclusion: "success",
-				Status:     "completed",
-				Text:       "Happy as a bunny",
+				Conclusion:              "success",
+				Status:                  "completed",
+				OriginalPipelineRunName: originalPipelineRunName,
+				Text:                    "Happy as a bunny",
 			},
 			expectedDescSubstr:    "validated",
 			expectedCommentSubstr: "Happy as a bunny",
 		},
 		{
 			name: "failed",
 			status: provider.StatusOpts{
-				Conclusion: "failure",
+				Conclusion:              "failure",
+				OriginalPipelineRunName: originalPipelineRunName,
 			},
 			expectedDescSubstr: "Failed",
 		},
 		{
 			name: "details url",
 			status: provider.StatusOpts{
-				Conclusion: "failure",
-				DetailsURL: "http://fail.com",
+				Conclusion:              "failure",
+				DetailsURL:              "http://fail.com",
+				OriginalPipelineRunName: originalPipelineRunName,
 			},
 			expectedDescSubstr: "Failed",
 		},
 		{
 			name: "pending",
 			status: provider.StatusOpts{
-				Conclusion: "pending",
+				Conclusion:              "pending",
+				OriginalPipelineRunName: originalPipelineRunName,
 			},
 			expectedDescSubstr: "started",
 		},
 		{
 			name: "success",
 			status: provider.StatusOpts{
-				Conclusion: "success",
+				Conclusion:              "success",
+				OriginalPipelineRunName: originalPipelineRunName,
 			},
 			expectedDescSubstr: "validated",
 		},
 		{
 			name: "completed",
 			status: provider.StatusOpts{
-				Conclusion: "completed",
+				Conclusion:              "completed",
+				OriginalPipelineRunName: originalPipelineRunName,
 			},
 			expectedDescSubstr: "Completed",
 		},

pkg/provider/bitbucketcloud/test/bbcloudtest.go

@@ -13,6 +13,7 @@ import (
 
 	"github.com/ktrysmt/go-bitbucket"
 	"github.com/openshift-pipelines/pipelines-as-code/pkg/params/info"
+	"github.com/openshift-pipelines/pipelines-as-code/pkg/params/settings"
 	"github.com/openshift-pipelines/pipelines-as-code/pkg/provider"
 	"github.com/openshift-pipelines/pipelines-as-code/pkg/provider/bitbucketcloud/types"
 	"gotest.tools/v3/assert"
@@ -153,6 +154,8 @@ func MuxCreateCommitstatus(t *testing.T, mux *http.ServeMux, event *info.Event,
 		bit, _ := io.ReadAll(r.Body)
 		err := json.Unmarshal(bit, cso)
 		assert.NilError(t, err)
+		pacOpts := &info.PacOpts{Settings: settings.Settings{ApplicationName: settings.PACApplicationNameDefaultValue}}
+		assert.Equal(t, provider.GetCheckName(expStatus, pacOpts), cso.Key)
 
 		if expStatus.DetailsURL != "" {
 			assert.Equal(t, expStatus.DetailsURL, cso.Url)