Skip to content
This repository was archived by the owner on Sep 26, 2023. It is now read-only.

Commit 2cca0bf

Browse files
authored
fix: Remove a flacky test in FlowControllerTest (#1360)
It is a temporary workaround. More details here: #1359
1 parent 18e4f8b commit 2cca0bf

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

gax/src/test/java/com/google/api/gax/batching/FlowControllerTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@
5050
import java.util.concurrent.TimeUnit;
5151
import java.util.concurrent.TimeoutException;
5252
import java.util.concurrent.atomic.AtomicInteger;
53+
import org.junit.Ignore;
5354
import org.junit.Test;
5455
import org.junit.runner.RunWith;
5556
import org.junit.runners.JUnit4;
@@ -538,6 +539,9 @@ public void testConcurrentUpdateThresholds_blocking() throws Exception {
538539
testBlockingReserveRelease(flowController, 0, expectedValue);
539540
}
540541

542+
// This test is very flaky. Remove @Ignore once https://github.com/googleapis/gax-java/issues/1359
543+
// is fixed.
544+
@Ignore
541545
@Test
542546
public void testConcurrentUpdateThresholds_nonBlocking() throws Exception {
543547
int initialValue = 5000;

0 commit comments

Comments
 (0)