cleanup: remove redunant IsCancelComment func

removed IsCancelComment func as we're using same
func from provider package so it is redunant now.

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

Author: zakisk

pkg/opscomments/comments.go

@@ -88,10 +88,6 @@ func IsOkToTestComment(comment string) bool {
 	return oktotestRegex.MatchString(comment)
 }
 
-func IsCancelComment(comment string) bool {
-	return cancelAllRegex.MatchString(comment) || cancelSingleRegex.MatchString(comment)
-}
-
 // EventTypeBackwardCompat handle the backward compatibility we need to keep until
 // we have done the deprecated notice
 //

pkg/opscomments/comments_test.go

@@ -320,57 +320,6 @@ func TestIsOkToTestComment(t *testing.T) {
 	}
 }
 
-func TestCancelComment(t *testing.T) {
-	tests := []struct {
-		name    string
-		comment string
-		want    bool
-	}{
-		{
-			name:    "valid",
-			comment: "/cancel",
-			want:    true,
-		},
-		{
-			name:    "valid with some string before",
-			comment: "/lgtm \n/cancel",
-			want:    true,
-		},
-		{
-			name:    "valid with some string before and after",
-			comment: "hi, trigger the ci \n/cancel \n then report the status back",
-			want:    true,
-		},
-		{
-			name:    "valid comments",
-			comment: "/lgtm \n/cancel \n/approve",
-			want:    true,
-		},
-		{
-			name:    "invalid",
-			comment: "/ok",
-			want:    false,
-		},
-		{
-			name:    "invalid comment",
-			comment: "/ok-to-test abc",
-			want:    false,
-		},
-		{
-			name:    "cancel single pr",
-			comment: "/cancel abc",
-			want:    true,
-		},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			got := IsCancelComment(tt.comment)
-			assert.Equal(t, tt.want, got)
-		})
-	}
-}
-
 func TestIsTestRetestComment(t *testing.T) {
 	tests := []struct {
 		name    string

pkg/provider/github/parse_payload.go

@@ -525,7 +525,6 @@ func (v *Provider) handleCommitCommentEvent(ctx context.Context, event *github.C
 		err        error
 	)
 
-	// TODO: reuse the code from opscomments
 	// If it is a /test or /retest comment with pipelinerun name figure out the pipelinerun name
 	if provider.IsTestRetestComment(event.GetComment().GetBody()) {
 		prName, branchName, err = provider.GetPipelineRunAndBranchNameFromTestComment(event.GetComment().GetBody())