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

Unified Diff: components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/HttpUrlRequestFactoryTest.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/HttpUrlRequestFactoryTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/HttpUrlRequestFactoryTest.java b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/HttpUrlRequestFactoryTest.java
deleted file mode 100644
index 25e4e553ed7a552250357849058fcf02850e41bf..0000000000000000000000000000000000000000
--- a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/HttpUrlRequestFactoryTest.java
+++ /dev/null
@@ -1,146 +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.HttpUrlRequestFactory;
-import org.chromium.net.HttpUrlRequestFactoryConfig;
-import org.chromium.net.UrlRequestContextConfig;
-
-import java.util.HashMap;
-import java.util.regex.Pattern;
-
-/**
- * Tests for {@link HttpUrlRequestFactory}
- */
-public class HttpUrlRequestFactoryTest extends CronetTestBase {
- // URL used for base tests.
- private static final String URL = "http://127.0.0.1:8000";
-
- @SmallTest
- @Feature({"Cronet"})
- public void testCreateFactory() throws Throwable {
- HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
- config.enableQUIC(true);
- config.addQuicHint("www.google.com", 443, 443);
- config.addQuicHint("www.youtube.com", 443, 443);
- config.setLibraryName("cronet_tests");
- String[] commandLineArgs = {
- CronetTestActivity.CONFIG_KEY, config.toString() };
- CronetTestActivity activity =
- launchCronetTestAppWithUrlAndCommandLineArgs(URL,
- commandLineArgs);
- HttpUrlRequestFactory factory = activity.mRequestFactory;
- assertNotNull("Factory should be created", factory);
- assertTrue("Factory should be Chromium/n.n.n.n@r but is "
- + factory.getName(),
- Pattern.matches("Chromium/\\d+\\.\\d+\\.\\d+\\.\\d+@\\w+",
- factory.getName()));
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testCreateLegacyFactory() {
- HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
- config.enableLegacyMode(true);
-
- HttpUrlRequestFactory factory = HttpUrlRequestFactory.createFactory(
- getInstrumentation().getTargetContext(), config);
- assertNotNull("Factory should be created", factory);
- assertTrue("Factory should be HttpUrlConnection/n.n.n.n@r but is "
- + factory.getName(),
- Pattern.matches(
- "HttpUrlConnection/\\d+\\.\\d+\\.\\d+\\.\\d+@\\w+",
- factory.getName()));
- HashMap<String, String> headers = new HashMap<String, String>();
- TestHttpUrlRequestListener listener = new TestHttpUrlRequestListener();
- HttpUrlRequest request = factory.createRequest(
- URL, 0, headers, listener);
- request.start();
- listener.blockForComplete();
- assertEquals(200, listener.mHttpStatusCode);
- assertEquals("OK", listener.mHttpStatusText);
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testCreateLegacyFactoryUsingUrlRequestContextConfig() {
- UrlRequestContextConfig config = new UrlRequestContextConfig();
- config.enableLegacyMode(true);
-
- HttpUrlRequestFactory factory = HttpUrlRequestFactory.createFactory(
- getInstrumentation().getTargetContext(), config);
- assertNotNull("Factory should be created", factory);
- assertTrue("Factory should be HttpUrlConnection/n.n.n.n@r but is "
- + factory.getName(),
- Pattern.matches(
- "HttpUrlConnection/\\d+\\.\\d+\\.\\d+\\.\\d+@\\w+",
- factory.getName()));
- HashMap<String, String> headers = new HashMap<String, String>();
- TestHttpUrlRequestListener listener = new TestHttpUrlRequestListener();
- HttpUrlRequest request = factory.createRequest(
- URL, 0, headers, listener);
- request.start();
- listener.blockForComplete();
- assertEquals(200, listener.mHttpStatusCode);
- assertEquals("OK", listener.mHttpStatusText);
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testQuicHintHost() {
- HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
- config.addQuicHint("www.google.com", 443, 443);
- try {
- config.addQuicHint("https://www.google.com", 443, 443);
- } catch (IllegalArgumentException e) {
- return;
- }
- fail("IllegalArgumentException must be thrown");
- }
-
- /**
- * Returns the path for the test storage (http cache, QUIC server info).
- */
- public String getTestStoragePath() {
- return PathUtils.getDataDirectory(
- getInstrumentation().getTargetContext()) + "/test_storage";
- }
-
- @SmallTest
- @Feature({"Cronet"})
- public void testEnableHttpCache() {
- HttpUrlRequestFactoryConfig config = new HttpUrlRequestFactoryConfig();
- config.enableHttpCache(HttpUrlRequestFactoryConfig.HttpCache.DISABLED, 0);
- config.enableHttpCache(HttpUrlRequestFactoryConfig.HttpCache.IN_MEMORY, 0);
- try {
- config.enableHttpCache(HttpUrlRequestFactoryConfig.HttpCache.DISK, 0);
- fail("IllegalArgumentException must be thrown");
- } catch (IllegalArgumentException e) {
- assertEquals("Storage path must be set", e.getMessage());
- }
- try {
- config.enableHttpCache(
- HttpUrlRequestFactoryConfig.HttpCache.DISK_NO_HTTP, 0);
- fail("IllegalArgumentException must be thrown");
- } catch (IllegalArgumentException e) {
- assertEquals("Storage path must be set", e.getMessage());
- }
-
- config.setStoragePath(prepareTestStorage());
- config.enableHttpCache(HttpUrlRequestFactoryConfig.HttpCache.DISK, 100);
- config.enableHttpCache(HttpUrlRequestFactoryConfig.HttpCache.DISK_NO_HTTP, 100);
- try {
- config.enableHttpCache(HttpUrlRequestFactoryConfig.HttpCache.IN_MEMORY, 0);
- fail("IllegalArgumentException must be thrown");
- } catch (IllegalArgumentException e) {
- assertEquals("Storage path must be empty", e.getMessage());
- }
- }
-}

Powered by Google App Engine
This is Rietveld 408576698