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

Unified Diff: components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/CronetUrlTest.java

Issue 884003004: Rename org.chromium.cronet_test_apk to org.chromium.net (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebased Created 5 years, 10 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 side-by-side diff with in-line comments
Download patch
Index: components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/CronetUrlTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/CronetUrlTest.java b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/CronetUrlTest.java
deleted file mode 100644
index 25c6cf903b4b633f9520b60510a30193d34dae9f..0000000000000000000000000000000000000000
--- a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/CronetUrlTest.java
+++ /dev/null
@@ -1,209 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-package org.chromium.cronet_test_apk;
-
-import android.test.suitebuilder.annotation.SmallTest;
-
-import org.chromium.base.PathUtils;
-import org.chromium.base.test.util.Feature;
-import org.chromium.net.HttpUrlRequest;
-import org.chromium.net.HttpUrlRequestFactoryConfig;
-
-import java.io.File;
-import java.util.HashMap;
-
-/**
- * Example test that just starts the cronet sample.
- */
-public class CronetUrlTest extends CronetTestBase {
- // URL used for base tests.
- private static final String URL = "http://127.0.0.1:8000";
-
- @SmallTest
- @Feature({"Cronet"})
- public void testLoadUrl() throws Exception {
- CronetTestActivity activity = launchCronetTestAppWithUrl(URL);
-
- // Make sure that the URL is set as expected.
- assertEquals(URL, activity.getUrl());
- assertEquals(200, activity.getHttpStatusCode());
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testInvalidUrl() throws Exception {
- CronetTestActivity activity = launchCronetTestAppWithUrl(
- "127.0.0.1:8000");
-
- // The load should fail.
- assertEquals(0, activity.getHttpStatusCode());
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testPostData() throws Exception {
- String[] commandLineArgs = {
- CronetTestActivity.POST_DATA_KEY, "test" };
- CronetTestActivity activity =
- launchCronetTestAppWithUrlAndCommandLineArgs(URL,
- commandLineArgs);
-
- // Make sure that the URL is set as expected.
- assertEquals(URL, activity.getUrl());
- assertEquals(200, activity.getHttpStatusCode());
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testNetLog() throws Exception {
- CronetTestActivity activity = launchCronetTestApp();
- File directory = new File(PathUtils.getDataDirectory(
- getInstrumentation().getTargetContext()));
- File file = File.createTempFile("cronet", "json", directory);
- activity.mRequestFactory.startNetLogToFile(file.getPath());
- // Starts a request.
- activity.startWithURL(URL);
- Thread.sleep(5000);
- activity.mRequestFactory.stopNetLog();
- assertTrue(file.exists());
- assertTrue(file.length() != 0);
- assertTrue(file.delete());
- assertTrue(!file.exists());
- }
-
- static class BadHttpUrlRequestListener extends TestHttpUrlRequestListener {
- static final String THROW_TAG = "BadListener";
-
- public BadHttpUrlRequestListener() {
- }
-
- @Override
- public void onResponseStarted(HttpUrlRequest request) {
- throw new NullPointerException(THROW_TAG);
- }
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testCalledByNativeException() throws Exception {
- CronetTestActivity activity = launchCronetTestAppWithUrl(URL);
-
- HashMap<String, String> headers = new HashMap<String, String>();
- BadHttpUrlRequestListener listener = new BadHttpUrlRequestListener();
-
- // Create request with bad listener to trigger an exception.
- HttpUrlRequest request = activity.mRequestFactory.createRequest(
- URL, HttpUrlRequest.REQUEST_PRIORITY_MEDIUM, headers, listener);
- request.start();
- listener.blockForComplete();
- assertTrue(request.isCanceled());
- assertNotNull(request.getException());
- assertEquals(listener.THROW_TAG,
- request.getException().getCause().getMessage());
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testSetUploadDataWithNullContentType() throws Exception {
- CronetTestActivity activity = launchCronetTestAppWithUrl(URL);
-
- HashMap<String, String> headers = new HashMap<String, String>();
- BadHttpUrlRequestListener listener = new BadHttpUrlRequestListener();
-
- // Create request.
- HttpUrlRequest request = activity.mRequestFactory.createRequest(
- URL, HttpUrlRequest.REQUEST_PRIORITY_MEDIUM, headers, listener);
- byte[] uploadData = new byte[] {1, 2, 3};
- try {
- request.setUploadData(null, uploadData);
- fail("setUploadData should throw on null content type");
- } catch (NullPointerException e) {
- // Nothing to do here.
- }
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void disabled_testQuicLoadUrl() throws Exception {
- HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
- // TODO(mef): Test Quic end-to-end using local QUIC server.
- String quicURL = "https://www.google.com:443";
- String quicNegotiatedProtocol = "quic/1+spdy/3";
- config.enableQUIC(true);
- config.addQuicHint("www.google.com", 443, 443);
- config.setExperimentalQuicConnectionOptions("PACE,IW10,FOO,DEADBEEF");
-
- String[] commandLineArgs = {
- CronetTestActivity.CONFIG_KEY, config.toString() };
- CronetTestActivity activity =
- launchCronetTestAppWithUrlAndCommandLineArgs(null,
- commandLineArgs);
- activity.startNetLog();
-
- HashMap<String, String> headers = new HashMap<String, String>();
- TestHttpUrlRequestListener listener = new TestHttpUrlRequestListener();
-
- // Try several times as first request may not use QUIC.
- // TODO(mef): Remove loop after adding http server properties manager.
- for (int i = 0; i < 10; ++i) {
- HttpUrlRequest request =
- activity.mRequestFactory.createRequest(
- quicURL,
- HttpUrlRequest.REQUEST_PRIORITY_MEDIUM,
- headers,
- listener);
- request.start();
- listener.blockForComplete();
- assertEquals(200, listener.mHttpStatusCode);
- if (listener.mNegotiatedProtocol.equals(quicNegotiatedProtocol)) {
- break;
- }
- Thread.sleep(1000);
- listener.resetComplete();
- }
-
- assertEquals(quicNegotiatedProtocol, listener.mNegotiatedProtocol);
- activity.stopNetLog();
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testLegacyLoadUrl() throws Exception {
- HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
- config.enableLegacyMode(true);
- // TODO(mef) fix tests so that library isn't loaded for legacy stack
- config.setLibraryName("cronet_tests");
-
- String[] commandLineArgs = {
- CronetTestActivity.CONFIG_KEY, config.toString() };
- CronetTestActivity activity =
- launchCronetTestAppWithUrlAndCommandLineArgs(URL,
- commandLineArgs);
-
- // Make sure that the URL is set as expected.
- assertEquals(URL, activity.getUrl());
- assertEquals(200, activity.getHttpStatusCode());
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testRequestHead() throws Exception {
- CronetTestActivity activity = launchCronetTestAppWithUrl(URL);
-
- HashMap<String, String> headers = new HashMap<String, String>();
- TestHttpUrlRequestListener listener = new TestHttpUrlRequestListener();
-
- // Create request.
- HttpUrlRequest request = activity.mRequestFactory.createRequest(
- URL, HttpUrlRequest.REQUEST_PRIORITY_MEDIUM, headers, listener);
- request.setHttpMethod("HEAD");
- request.start();
- listener.blockForComplete();
- assertEquals(200, listener.mHttpStatusCode);
- // HEAD requests do not get any response data and Content-Length must be
- // ignored.
- assertEquals(0, listener.mResponseAsBytes.length);
- }
-}

Powered by Google App Engine
This is Rietveld 408576698