Skip to content

Commit c69d264

Browse files
Copilotpelikhan
andauthored
Fix tests: Update to use sliceutil.MergeUnique
- Updated logs_report_test.go to use sliceutil.MergeUnique instead of removed addUniqueWorkflow - All tests passing - make agent-report-progress passes Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
1 parent 5694e2d commit c69d264

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

pkg/cli/logs_report_test.go

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,8 @@ import (
77
"path/filepath"
88
"testing"
99
"time"
10+
11+
"github.com/github/gh-aw/pkg/sliceutil"
1012
)
1113

1214
// TestRenderLogsConsoleUnified tests the unified console rendering
@@ -222,7 +224,7 @@ func TestAddUniqueWorkflow(t *testing.T) {
222224

223225
for _, tt := range tests {
224226
t.Run(tt.name, func(t *testing.T) {
225-
result := addUniqueWorkflow(tt.workflows, tt.workflow)
227+
result := sliceutil.MergeUnique(tt.workflows, tt.workflow)
226228
if len(result) != len(tt.expected) {
227229
t.Errorf("Expected length %d, got %d", len(tt.expected), len(result))
228230
}
@@ -429,7 +431,7 @@ func TestAggregateSummaryItems(t *testing.T) {
429431
},
430432
func(summary *MissingToolSummary, tool MissingToolReport) {
431433
summary.Count++
432-
summary.Workflows = addUniqueWorkflow(summary.Workflows, tool.WorkflowName)
434+
summary.Workflows = sliceutil.MergeUnique(summary.Workflows, tool.WorkflowName)
433435
summary.RunIDs = append(summary.RunIDs, tool.RunID)
434436
},
435437
func(summary *MissingToolSummary) {

0 commit comments

Comments
 (0)