Skip to content

Commit 86a788a

Browse files
M09Icclaude
andcommitted
fix(ci): fix go vet warnings in e2e_test.go
- Shadow loop variable `endpoint` to avoid capture-by-reference - Use t.Errorf instead of t.Fatalf in goroutine (not allowed) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 9844ef8 commit 86a788a

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

runner/e2e_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ func runReconfigureServerHelper(t *testing.T) {
181181
}
182182
go func() {
183183
if err := console.Run(); err != nil {
184-
t.Fatalf("console.Run: %v", err)
184+
t.Errorf("console.Run: %v", err)
185185
}
186186
}()
187187

@@ -2037,6 +2037,7 @@ func multiServeDualSOCKSConcurrentScenario() runner.Scenario {
20372037
var wg sync.WaitGroup
20382038
errs := make(chan error, reqsPerServe*2)
20392039
for _, endpoint := range []string{"socks_addr_1", "socks_addr_2"} {
2040+
endpoint := endpoint
20402041
addr := fx.Endpoint(endpoint)
20412042
for i := 0; i < reqsPerServe; i++ {
20422043
wg.Add(1)

0 commit comments

Comments
 (0)