Skip to content

Commit 10484d1

Browse files
committed
Revert unnecessary change
1 parent 84556fc commit 10484d1

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

balancer/weightedtarget/weightedtarget_test.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -298,6 +298,7 @@ func (s) TestWeightedTarget(t *testing.T) {
298298
}
299299
verifyAddressInNewSubConn(t, cc, addr3)
300300

301+
// The subconn from the test_config_balancer should be shut down.
301302
scShutdown = <-cc.ShutdownSubConnCh
302303
// The same SubConn is closed by gracefulswitch and pickfirstleaf when they
303304
// are closed. Remove duplicate events.
@@ -306,7 +307,6 @@ func (s) TestWeightedTarget(t *testing.T) {
306307
// shutting down SubConns.
307308
<-cc.ShutdownSubConnCh
308309

309-
// The subconn from the test_config_balancer should be shut down.
310310
if scShutdown != sc2 {
311311
t.Fatalf("ShutdownSubConn, want %v, got %v", sc2, scShutdown)
312312
}

internal/balancergroup/balancergroup_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -360,8 +360,8 @@ func (s) TestBalancerGroup_locality_caching_not_read_within_timeout(t *testing.T
360360
ctx, cancel := context.WithTimeout(context.Background(), defaultTestTimeout)
361361
defer cancel()
362362
scToShutdown := map[balancer.SubConn]int{
363-
addrToSC[testBackendAddrs[2].Addr]: 3,
364-
addrToSC[testBackendAddrs[3].Addr]: 3,
363+
addrToSC[testBackendAddrs[2].Addr]: 1,
364+
addrToSC[testBackendAddrs[3].Addr]: 1,
365365
}
366366
for i := 0; i < len(scToShutdown); i++ {
367367
select {

0 commit comments

Comments
 (0)