Merge pull request #1369 from hongweiliu17/comment_improvement

fix: create comment for all integration test status if no commitStatus

Author: hongweiliu17

status/reporter_github.go

@@ -481,7 +481,7 @@ func (csu *CommitStatusUpdater) UpdateStatus(ctx context.Context, report TestRep
 	}
 	// Create a comment when integration test is neither pending nor inprogress since comment for pending/inprogress is less meaningful and there is commitStatus for all statuses
 	_, isPullRequest := csu.snapshot.GetAnnotations()[gitops.PipelineAsCodePullRequestAnnotation]
-	if report.Status != intgteststat.IntegrationTestStatusPending && report.Status != intgteststat.IntegrationTestStatusInProgress && isPullRequest {
+	if isPullRequest {
 		statusCode, err := csu.updateStatusInComment(ctx, report)
 		if err != nil {
 			csu.logger.Error(err, "failed to update comment", "snapshot.NameSpace", csu.snapshot.Namespace, "snapshot.Name", csu.snapshot.Name, "scenarioName", report.ScenarioName)

status/reporter_gitlab.go

@@ -322,7 +322,7 @@ func (r *GitLabReporter) ReportStatus(ctx context.Context, report TestReport) (i
 
 	// Create a note when integration test is neither pending nor inprogress since comment for pending/inprogress is less meaningful
 	_, isMergeRequest := r.snapshot.GetAnnotations()[gitops.PipelineAsCodePullRequestAnnotation]
-	if report.Status != intgteststat.IntegrationTestStatusPending && report.Status != intgteststat.IntegrationTestStatusInProgress && report.Status != intgteststat.SnapshotCreationFailed && isMergeRequest {
+	if isMergeRequest {
 		statusCode, err := r.updateStatusInComment(report)
 		if err != nil {
 			return statusCode, err