Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(213)

Side by Side Diff: components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java

Issue 1856273002: Change all FlakyTest import to use org.chromium.base.test.util.FlakyTest (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Change @FlakyTest bug message Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 package org.chromium.net; 5 package org.chromium.net;
6 6
7 import static org.chromium.base.CollectionUtil.newHashSet; 7 import static org.chromium.base.CollectionUtil.newHashSet;
8 8
9 import android.content.Context; 9 import android.content.Context;
10 import android.content.ContextWrapper; 10 import android.content.ContextWrapper;
11 import android.os.ConditionVariable; 11 import android.os.ConditionVariable;
12 import android.os.Handler; 12 import android.os.Handler;
13 import android.os.Looper; 13 import android.os.Looper;
14 import android.test.FlakyTest;
15 import android.test.suitebuilder.annotation.SmallTest; 14 import android.test.suitebuilder.annotation.SmallTest;
16 15
17 import org.chromium.base.PathUtils; 16 import org.chromium.base.PathUtils;
18 import org.chromium.base.annotations.JNINamespace; 17 import org.chromium.base.annotations.JNINamespace;
19 import org.chromium.base.test.util.DisabledTest; 18 import org.chromium.base.test.util.DisabledTest;
20 import org.chromium.base.test.util.Feature; 19 import org.chromium.base.test.util.Feature;
20 import org.chromium.base.test.util.FlakyTest;
21 import org.chromium.net.CronetEngine.UrlRequestInfo; 21 import org.chromium.net.CronetEngine.UrlRequestInfo;
22 import org.chromium.net.TestUrlRequestCallback.ResponseStep; 22 import org.chromium.net.TestUrlRequestCallback.ResponseStep;
23 import org.chromium.net.test.EmbeddedTestServer; 23 import org.chromium.net.test.EmbeddedTestServer;
24 24
25 import java.io.BufferedReader; 25 import java.io.BufferedReader;
26 import java.io.File; 26 import java.io.File;
27 import java.io.FileReader; 27 import java.io.FileReader;
28 import java.util.Arrays; 28 import java.util.Arrays;
29 import java.util.HashSet; 29 import java.util.HashSet;
30 import java.util.LinkedList; 30 import java.util.LinkedList;
(...skipping 303 matching lines...) Expand 10 before | Expand all | Expand 10 after
334 assertTrue(metrics.getTotalTimeMs() > 0); 334 assertTrue(metrics.getTotalTimeMs() > 0);
335 assertTrue(metrics.getTotalTimeMs() >= metrics.getTtfbMs()); 335 assertTrue(metrics.getTotalTimeMs() >= metrics.getTtfbMs());
336 assertTrue(metrics.getReceivedBytesCount() > 0); 336 assertTrue(metrics.getReceivedBytesCount() > 0);
337 mTestFramework.mCronetEngine.shutdown(); 337 mTestFramework.mCronetEngine.shutdown();
338 } 338 }
339 339
340 /* 340 /*
341 @SmallTest 341 @SmallTest
342 @Feature({"Cronet"}) 342 @Feature({"Cronet"})
343 @SuppressWarnings("deprecation") 343 @SuppressWarnings("deprecation")
344 https://crbug.com/592444
345 */ 344 */
346 @FlakyTest 345 @FlakyTest(message = "https://crbug.com/592444")
347 public void testRequestFinishedListenerFailedRequest() throws Exception { 346 public void testRequestFinishedListenerFailedRequest() throws Exception {
348 String connectionRefusedUrl = "http://127.0.0.1:3"; 347 String connectionRefusedUrl = "http://127.0.0.1:3";
349 mTestFramework = startCronetTestFramework(); 348 mTestFramework = startCronetTestFramework();
350 TestExecutor testExecutor = new TestExecutor(); 349 TestExecutor testExecutor = new TestExecutor();
351 TestRequestFinishedListener requestFinishedListener = new TestRequestFin ishedListener(); 350 TestRequestFinishedListener requestFinishedListener = new TestRequestFin ishedListener();
352 mTestFramework.mCronetEngine.enableNetworkQualityEstimator(testExecutor) ; 351 mTestFramework.mCronetEngine.enableNetworkQualityEstimator(testExecutor) ;
353 mTestFramework.mCronetEngine.addRequestFinishedListener(requestFinishedL istener); 352 mTestFramework.mCronetEngine.addRequestFinishedListener(requestFinishedL istener);
354 TestUrlRequestCallback callback = new TestUrlRequestCallback(); 353 TestUrlRequestCallback callback = new TestUrlRequestCallback();
355 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(connection RefusedUrl, 354 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(connection RefusedUrl,
356 callback, callback.getExecutor(), mTestFramework.mCronetEngine); 355 callback, callback.getExecutor(), mTestFramework.mCronetEngine);
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
545 mTestFramework.mCronetEngine.shutdown(); 544 mTestFramework.mCronetEngine.shutdown();
546 fail("Should throw an exception"); 545 fail("Should throw an exception");
547 } catch (Exception e) { 546 } catch (Exception e) {
548 assertEquals("Engine is shut down.", e.getMessage()); 547 assertEquals("Engine is shut down.", e.getMessage());
549 } 548 }
550 } 549 }
551 550
552 /* 551 /*
553 @SmallTest 552 @SmallTest
554 @Feature({"Cronet"}) 553 @Feature({"Cronet"})
555 https://crbug.com/592444
556 */ 554 */
557 @FlakyTest 555 @FlakyTest(message = "https://crbug.com/592444")
558 public void testShutdownAfterError() throws Exception { 556 public void testShutdownAfterError() throws Exception {
559 mTestFramework = startCronetTestFramework(); 557 mTestFramework = startCronetTestFramework();
560 TestUrlRequestCallback callback = new ShutdownTestUrlRequestCallback(); 558 TestUrlRequestCallback callback = new ShutdownTestUrlRequestCallback();
561 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(MOCK_CRONE T_TEST_FAILED_URL, 559 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(MOCK_CRONE T_TEST_FAILED_URL,
562 callback, callback.getExecutor(), mTestFramework.mCronetEngine); 560 callback, callback.getExecutor(), mTestFramework.mCronetEngine);
563 urlRequestBuilder.build().start(); 561 urlRequestBuilder.build().start();
564 callback.blockForDone(); 562 callback.blockForDone();
565 assertTrue(callback.mOnErrorCalled); 563 assertTrue(callback.mOnErrorCalled);
566 } 564 }
567 565
(...skipping 445 matching lines...) Expand 10 before | Expand all | Expand 10 after
1013 try { 1011 try {
1014 // ensureInitialized() calls native code to check the version right after library load 1012 // ensureInitialized() calls native code to check the version right after library load
1015 // and will error with the message below if library loading was skip ped 1013 // and will error with the message below if library loading was skip ped
1016 CronetLibraryLoader.ensureInitialized(getContext(), builder); 1014 CronetLibraryLoader.ensureInitialized(getContext(), builder);
1017 fail("Native library should not be loaded"); 1015 fail("Native library should not be loaded");
1018 } catch (UnsatisfiedLinkError e) { 1016 } catch (UnsatisfiedLinkError e) {
1019 assertTrue(loader.wasCalled()); 1017 assertTrue(loader.wasCalled());
1020 } 1018 }
1021 } 1019 }
1022 } 1020 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698