fix: skip reporting runs created via API (#622)

Fixes #618
leg100 · Oct 19, 2023 · 5d4527b · 5d4527b
1 parent 8a9b1f9
commit 5d4527b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/internal/run/reporter.go b/internal/run/reporter.go
@@ -68,8 +68,8 @@ func (r *Reporter) Start(ctx context.Context) error {
 }
 
 func (r *Reporter) handleRun(ctx context.Context, run *Run) error {
-	// Skip runs triggered via the UI
-	if run.Source == SourceUI {
+	// Skip runs triggered via the UI or API
+	if run.Source == SourceUI || run.Source == SourceAPI {
 		return nil
 	}
 

diff --git a/internal/run/reporter_test.go b/internal/run/reporter_test.go
@@ -57,6 +57,11 @@ func TestReporter_HandleRun(t *testing.T) {
 			run:  &Run{ID: "run-123", Source: SourceUI},
 			want: vcs.SetStatusOptions{},
 		},
+		{
+			name: "skip API-triggered run",
+			run:  &Run{ID: "run-123", Source: SourceAPI},
+			want: vcs.SetStatusOptions{},
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {