Skip to content

Commit ac5819a

Browse files
committed
fix: update test calls to AddFilesToTargetRepoBranch after signature change
1 parent 6882a54 commit ac5819a

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

examples-copier/services/github_write_to_target_test.go

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -250,7 +250,7 @@ func TestAddFilesToTargetRepoBranch_Succeeds(t *testing.T) {
250250
},
251251
}
252252

253-
services.AddFilesToTargetRepoBranch(nil)
253+
services.AddFilesToTargetRepoBranch()
254254

255255
info := httpmock.GetCallCountInfo()
256256
require.Equal(t, 1, info["GET "+baseRefURL])
@@ -343,7 +343,7 @@ func TestAddFilesToTargetRepoBranch_ViaPR_Succeeds(t *testing.T) {
343343
},
344344
}
345345

346-
services.AddFilesToTargetRepoBranch(nil)
346+
services.AddFilesToTargetRepoBranch()
347347

348348
// Assertions
349349
require.Equal(t, 1, test.CountByMethodAndURLRegexp("POST",
@@ -425,7 +425,7 @@ func TestAddFiles_DirectConflict_NonFastForward(t *testing.T) {
425425
}
426426

427427
// Run – should not panic; error is handled/logged internally.
428-
services.AddFilesToTargetRepoBranch(nil)
428+
services.AddFilesToTargetRepoBranch()
429429

430430
info := httpmock.GetCallCountInfo()
431431
require.Equal(t, 1, info["GET "+baseRefURL])
@@ -508,7 +508,7 @@ func TestAddFiles_ViaPR_MergeConflict_Dirty_NotMerged(t *testing.T) {
508508
},
509509
}
510510

511-
services.AddFilesToTargetRepoBranch(nil)
511+
services.AddFilesToTargetRepoBranch()
512512

513513
// Assertions
514514
info := httpmock.GetCallCountInfo()
@@ -571,7 +571,7 @@ func TestPriority_Strategy_ConfigOverridesEnv_And_MessageFallbacks(t *testing.T)
571571
{RepoName: repo, BranchPath: "refs/heads/" + baseBranch, CommitStrategy: cfg.CopierCommitStrategy}: {TargetBranch: baseBranch, Content: files},
572572
}
573573

574-
services.AddFilesToTargetRepoBranch(nil) // No longer takes parameters - uses FilesToUpload map
574+
services.AddFilesToTargetRepoBranch() // No longer takes parameters - uses FilesToUpload map
575575

576576
info := httpmock.GetCallCountInfo()
577577
require.Equal(t, 1, info["GET "+baseRefURL])
@@ -650,7 +650,7 @@ func TestPriority_PRTitleDefaultsToCommitMessage_And_NoAutoMergeWhenConfigPresen
650650
// cfg := types.Configs{TargetRepo: repo, TargetBranch: baseBranch /* MergeWithoutReview: false (zero value) */}
651651
services.FilesToUpload = map[types.UploadKey]types.UploadFileContent{{RepoName: repo, BranchPath: "refs/heads/" + baseBranch, RuleName: "", CommitStrategy: "pr"}: {TargetBranch: baseBranch, Content: files, CommitStrategy: "pr"}}
652652

653-
services.AddFilesToTargetRepoBranch(nil) // No longer takes parameters - uses FilesToUpload map
653+
services.AddFilesToTargetRepoBranch() // No longer takes parameters - uses FilesToUpload map
654654

655655
// Ensure a PR was created but no merge occurred
656656
require.Equal(t, 1, test.CountByMethodAndURLRegexp("POST", regexp.MustCompile(`/pulls$`)))

0 commit comments

Comments
 (0)