diff --git a/xds/src/test/java/io/grpc/xds/RingHashLoadBalancerTest.java b/xds/src/test/java/io/grpc/xds/RingHashLoadBalancerTest.java index 9c7f8dc62e0..00d5e8996ff 100644 --- a/xds/src/test/java/io/grpc/xds/RingHashLoadBalancerTest.java +++ b/xds/src/test/java/io/grpc/xds/RingHashLoadBalancerTest.java @@ -719,17 +719,17 @@ public void allSubchannelsInTransientFailure() { // Map each server address to exactly one ring entry. RingHashConfig config = new RingHashConfig(3, 3, ""); List servers = createWeightedServerAddrs(1, 1, 1); - initializeLbSubchannels(config, servers); + List subchannelList = initializeLbSubchannels(config, servers); // Bring all subchannels to TRANSIENT_FAILURE. - for (Subchannel subchannel : subchannels.values()) { + for (Subchannel subchannel : subchannelList) { deliverSubchannelState(subchannel, ConnectivityStateInfo.forTransientFailure( Status.UNAVAILABLE.withDescription( subchannel.getAddresses().getAddresses() + " unreachable"))); } verify(helper, atLeastOnce()) .updateBalancingState(eq(TRANSIENT_FAILURE), pickerCaptor.capture()); - verifyConnection(PickFirstLoadBalancerProvider.isEnabledNewPickFirst() ? 1 : 2); + verifyConnection(2); // Picking subchannel triggers connection. RPC hash hits server0. PickSubchannelArgs args = getDefaultPickSubchannelArgsForServer(0);