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

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: 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.Feature; 18 import org.chromium.base.test.util.Feature;
19 import org.chromium.base.test.util.FlakyTest;
20 import org.chromium.net.CronetEngine.UrlRequestInfo; 20 import org.chromium.net.CronetEngine.UrlRequestInfo;
21 import org.chromium.net.TestUrlRequestCallback.ResponseStep; 21 import org.chromium.net.TestUrlRequestCallback.ResponseStep;
22 import org.chromium.net.test.EmbeddedTestServer; 22 import org.chromium.net.test.EmbeddedTestServer;
23 23
24 import java.io.BufferedReader; 24 import java.io.BufferedReader;
25 import java.io.File; 25 import java.io.File;
26 import java.io.FileReader; 26 import java.io.FileReader;
27 import java.util.Arrays; 27 import java.util.Arrays;
28 import java.util.HashSet; 28 import java.util.HashSet;
29 import java.util.LinkedList; 29 import java.util.LinkedList;
(...skipping 305 matching lines...) Expand 10 before | Expand all | Expand 10 after
335 assertTrue(metrics.getTotalTimeMs() > 0); 335 assertTrue(metrics.getTotalTimeMs() > 0);
336 assertTrue(metrics.getTotalTimeMs() >= metrics.getTtfbMs()); 336 assertTrue(metrics.getTotalTimeMs() >= metrics.getTtfbMs());
337 assertTrue(metrics.getReceivedBytesCount() > 0); 337 assertTrue(metrics.getReceivedBytesCount() > 0);
338 mTestFramework.mCronetEngine.shutdown(); 338 mTestFramework.mCronetEngine.shutdown();
339 } 339 }
340 340
341 /* 341 /*
342 @SmallTest 342 @SmallTest
343 @Feature({"Cronet"}) 343 @Feature({"Cronet"})
344 @SuppressWarnings("deprecation") 344 @SuppressWarnings("deprecation")
345 https://crbug.com/592444
346 */ 345 */
347 @FlakyTest 346 @FlakyTest(message = "https://crbug.com/592444")
348 public void testRequestFinishedListenerFailedRequest() throws Exception { 347 public void testRequestFinishedListenerFailedRequest() throws Exception {
349 String connectionRefusedUrl = "http://127.0.0.1:3"; 348 String connectionRefusedUrl = "http://127.0.0.1:3";
350 mTestFramework = startCronetTestFramework(); 349 mTestFramework = startCronetTestFramework();
351 TestExecutor testExecutor = new TestExecutor(); 350 TestExecutor testExecutor = new TestExecutor();
352 TestRequestFinishedListener requestFinishedListener = new TestRequestFin ishedListener(); 351 TestRequestFinishedListener requestFinishedListener = new TestRequestFin ishedListener();
353 mTestFramework.mCronetEngine.enableNetworkQualityEstimator(testExecutor) ; 352 mTestFramework.mCronetEngine.enableNetworkQualityEstimator(testExecutor) ;
354 mTestFramework.mCronetEngine.addRequestFinishedListener(requestFinishedL istener); 353 mTestFramework.mCronetEngine.addRequestFinishedListener(requestFinishedL istener);
355 TestUrlRequestCallback callback = new TestUrlRequestCallback(); 354 TestUrlRequestCallback callback = new TestUrlRequestCallback();
356 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(connection RefusedUrl, 355 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(connection RefusedUrl,
357 callback, callback.getExecutor(), mTestFramework.mCronetEngine); 356 callback, callback.getExecutor(), mTestFramework.mCronetEngine);
(...skipping 192 matching lines...) Expand 10 before | Expand all | Expand 10 after
550 mTestFramework.mCronetEngine.shutdown(); 549 mTestFramework.mCronetEngine.shutdown();
551 fail("Should throw an exception"); 550 fail("Should throw an exception");
552 } catch (Exception e) { 551 } catch (Exception e) {
553 assertEquals("Engine is shut down.", e.getMessage()); 552 assertEquals("Engine is shut down.", e.getMessage());
554 } 553 }
555 } 554 }
556 555
557 /* 556 /*
558 @SmallTest 557 @SmallTest
559 @Feature({"Cronet"}) 558 @Feature({"Cronet"})
560 https://crbug.com/592444
561 */ 559 */
562 @FlakyTest 560 @FlakyTest(message = "https://crbug.com/592444")
563 public void testShutdownAfterError() throws Exception { 561 public void testShutdownAfterError() throws Exception {
564 mTestFramework = startCronetTestFramework(); 562 mTestFramework = startCronetTestFramework();
565 TestUrlRequestCallback callback = new ShutdownTestUrlRequestCallback(); 563 TestUrlRequestCallback callback = new ShutdownTestUrlRequestCallback();
566 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(MOCK_CRONE T_TEST_FAILED_URL, 564 UrlRequest.Builder urlRequestBuilder = new UrlRequest.Builder(MOCK_CRONE T_TEST_FAILED_URL,
567 callback, callback.getExecutor(), mTestFramework.mCronetEngine); 565 callback, callback.getExecutor(), mTestFramework.mCronetEngine);
568 urlRequestBuilder.build().start(); 566 urlRequestBuilder.build().start();
569 callback.blockForDone(); 567 callback.blockForDone();
570 assertTrue(callback.mOnErrorCalled); 568 assertTrue(callback.mOnErrorCalled);
571 } 569 }
572 570
(...skipping 548 matching lines...) Expand 10 before | Expand all | Expand 10 after
1121 try { 1119 try {
1122 // ensureInitialized() calls native code to check the version right after library load 1120 // ensureInitialized() calls native code to check the version right after library load
1123 // and will error with the message below if library loading was skip ped 1121 // and will error with the message below if library loading was skip ped
1124 CronetLibraryLoader.ensureInitialized(getContext(), builder); 1122 CronetLibraryLoader.ensureInitialized(getContext(), builder);
1125 fail("Native library should not be loaded"); 1123 fail("Native library should not be loaded");
1126 } catch (UnsatisfiedLinkError e) { 1124 } catch (UnsatisfiedLinkError e) {
1127 assertTrue(loader.wasCalled()); 1125 assertTrue(loader.wasCalled());
1128 } 1126 }
1129 } 1127 }
1130 } 1128 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698