Commit 7171bc6f authored by Helen Li's avatar Helen Li Committed by Commit Bot

[cronet] Shut down Executor in CronetStressTest#testLargeNumberOfUploads

Shut down executor in CronetStressTest.

Bug: 771366
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester
Change-Id: Id94e90c22ed2c3760da674f731a4dbcd4c6e5dbe
Reviewed-on: https://chromium-review.googlesource.com/702037
Commit-Queue: Helen Li <xunjieli@chromium.org>
Reviewed-by: default avatarAndrei Kapishnikov <kapishnikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#506742}
parent 7e4697db
...@@ -9,6 +9,8 @@ import static org.junit.Assert.assertTrue; ...@@ -9,6 +9,8 @@ import static org.junit.Assert.assertTrue;
import static org.chromium.net.CronetTestRule.getContext; import static org.chromium.net.CronetTestRule.getContext;
import android.support.test.filters.LargeTest;
import org.junit.After; import org.junit.After;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule; import org.junit.Rule;
...@@ -16,7 +18,6 @@ import org.junit.Test; ...@@ -16,7 +18,6 @@ import org.junit.Test;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.chromium.base.test.BaseJUnit4ClassRunner; import org.chromium.base.test.BaseJUnit4ClassRunner;
import org.chromium.base.test.util.DisabledTest;
import org.chromium.base.test.util.Feature; import org.chromium.base.test.util.Feature;
import org.chromium.net.CronetTestRule.CronetTestFramework; import org.chromium.net.CronetTestRule.CronetTestFramework;
import org.chromium.net.CronetTestRule.OnlyRunNativeCronet; import org.chromium.net.CronetTestRule.OnlyRunNativeCronet;
...@@ -43,8 +44,7 @@ public class CronetStressTest { ...@@ -43,8 +44,7 @@ public class CronetStressTest {
} }
@Test @Test
//@LargeTest @LargeTest
@DisabledTest(message = "see crbug.com/771366")
@OnlyRunNativeCronet @OnlyRunNativeCronet
@Feature({"Cronet"}) @Feature({"Cronet"})
public void testLargeNumberOfUploads() throws Exception { public void testLargeNumberOfUploads() throws Exception {
...@@ -65,6 +65,7 @@ public class CronetStressTest { ...@@ -65,6 +65,7 @@ public class CronetStressTest {
UrlRequest request = builder.build(); UrlRequest request = builder.build();
request.start(); request.start();
callback.blockForDone(); callback.blockForDone();
callback.shutdownExecutor();
assertEquals(200, callback.mResponseInfo.getHttpStatusCode()); assertEquals(200, callback.mResponseInfo.getHttpStatusCode());
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment