Skip to content

Commit

Permalink
test: Solve CI flakiness for test/java/org/chromium/net/urlconnection…
Browse files Browse the repository at this point in the history
…:urlconnection_test (#2007)

Description: EM has a 1024 concurrent request limit that is now respected by Cronet tests
Risk Level: N/A
Testing: Solves CI "android_tests / java_tests_mac" failure
Docs Changes: N/A
Release Notes: N/A

Signed-off-by: Charles Le Borgne <[email protected]>
  • Loading branch information
carloseltuerto authored Jan 25, 2022
1 parent 3553b3c commit af4d895
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions test/java/org/chromium/net/CronetUrlRequestTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -2305,8 +2305,7 @@ public void testTagging() throws Exception {}
*/
public void testManyRequests() throws Exception {
String url = NativeTestServer.getMultiRedirectURL();
// Jelly Bean has a 2000 limit on global references, crbug.com/922656.
final int numRequests = Build.VERSION.SDK_INT >= Build.VERSION_CODES.KITKAT ? 2000 : 1500;
final int numRequests = 1000;
TestUrlRequestCallback callbacks[] = new TestUrlRequestCallback[numRequests];
UrlRequest requests[] = new UrlRequest[numRequests];
for (int i = 0; i < numRequests; i++) {
Expand Down

0 comments on commit af4d895

Please sign in to comment.