carlos / platform Public

appconfig: the multi-upstream SyncExec test passes the ack parameter

Commit 8e16153 added a SyncExec caller beside the relay branch, which
gave SyncExec an eighth parameter; each was green against its own base
and main's gate caught the merge. No behaviour change.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

Paul Campbell pushed by paul@keymail.dev 1 month ago db9ccfda5873160c38b678bc5292f5b2899b1b8a parent 4316089

1 file changed, +1 −1

internal/appconfig/syncexec_test.go +1 −1

CI passed — run details

diff --git a/internal/appconfig/syncexec_test.go b/internal/appconfig/syncexec_test.go

index 97a04b0cf..b580080f9 100644

--- a/internal/appconfig/syncexec_test.go

+++ b/internal/appconfig/syncexec_test.go

@@ -704,7 +704,7 @@ func TestSyncExecOptedInMultiAddrRouteMaterializesFile(t *testing.T) {

if err := SyncExec(ctx, reg, s, pk, dd, func(string) error {

bounces++

return nil

- }, ""); err != nil {

+ }, "", nil); err != nil {

t.Fatalf("SyncExec: %v", err)

}

got, err := os.ReadFile(filepath.Join(dd, "woodstar.example.com.config.env"))