Skip to content

Commit f67ec2e

Browse files
authored
xds: fix flow control test failure (#10773)
1 parent 7be2b6c commit f67ec2e

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

xds/src/test/java/io/grpc/xds/XdsClientImplTestBase.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@
2222
import static org.mockito.ArgumentMatchers.any;
2323
import static org.mockito.ArgumentMatchers.eq;
2424
import static org.mockito.ArgumentMatchers.isA;
25+
import static org.mockito.Mockito.atLeastOnce;
2526
import static org.mockito.Mockito.doAnswer;
2627
import static org.mockito.Mockito.mock;
2728
import static org.mockito.Mockito.never;
@@ -3004,7 +3005,7 @@ public void flowControlAbsent() throws Exception {
30043005
verifyResourceMetadataAcked(
30053006
CDS, CDS_RESOURCE, testClusterRoundRobin, VERSION_1, TIME_INCREMENT);
30063007
barrier.await();
3007-
verify(cdsResourceWatcher, times(1)).onChanged(any());
3008+
verify(cdsResourceWatcher, atLeastOnce()).onChanged(any());
30083009
String errorMsg = "CDS response Cluster 'cluster.googleapis.com2' validation error: "
30093010
+ "Cluster cluster.googleapis.com2: unspecified cluster discovery type";
30103011
call.verifyRequestNack(CDS, Arrays.asList(CDS_RESOURCE, anotherCdsResource), VERSION_1, "0001",
@@ -3067,8 +3068,8 @@ public void simpleFlowControl() throws Exception {
30673068
verifyResourceMetadataAcked(EDS, EDS_RESOURCE, testClusterLoadAssignment, VERSION_1,
30683069
TIME_INCREMENT);
30693070
barrier.await();
3070-
verify(edsResourceWatcher, times(1)).onChanged(edsUpdateCaptor.capture());
3071-
EdsUpdate edsUpdate = edsUpdateCaptor.getValue();
3071+
verify(edsResourceWatcher, atLeastOnce()).onChanged(edsUpdateCaptor.capture());
3072+
EdsUpdate edsUpdate = edsUpdateCaptor.getAllValues().get(0);
30723073
validateGoldenClusterLoadAssignment(edsUpdate);
30733074
barrier.await();
30743075
latch.await(10, TimeUnit.SECONDS);

0 commit comments

Comments
 (0)