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

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

Issue 525613002: Set up initial MockURLRequestJob tests for Cronet (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressed comments Created 6 years, 3 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/MockURLRequestJobTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java
new file mode 100644
index 0000000000000000000000000000000000000000..7f8a504f6e53bdcee96895cbf52bccf01b91c688
--- /dev/null
+++ b/components/cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockURLRequestJobTest.java
@@ -0,0 +1,90 @@
+// 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.os.ConditionVariable;
+import android.test.suitebuilder.annotation.SmallTest;
+import android.util.Log;
+
+import org.chromium.base.test.util.Feature;
+import org.chromium.net.HttpUrlRequest;
+import org.chromium.net.HttpUrlRequestListener;
+
+import java.util.HashMap;
+
+public class MockURLRequestJobTest extends CronetTestBase {
+ private static final String TAG = "MockURLRequestJobTest";
+ class MockHttpUrlRequestListener implements HttpUrlRequestListener {
+ ConditionVariable mComplete = new ConditionVariable();
+ public MockHttpUrlRequestListener() {
+ }
+
+ @Override
+ public void onResponseStarted(HttpUrlRequest request) {
+ Log.i(TAG, "****** Response Started, content length is "
+ + request.getContentLength());
mmenke 2014/09/03 18:02:35 nit: Binary operators should be put on the same l
+ Log.i(TAG, "*** Headers Are *** " + request.getAllHeaders());
+ }
+
+ public void blockForComplete() {
+ mComplete.block();
+ }
+
+ @Override
+ public void onRequestComplete(HttpUrlRequest request) {
+ mComplete.open();
+ Log.i(TAG, "****** Request Complete, status code is "
+ + request.getHttpStatusCode());
mmenke 2014/09/03 18:02:35 nit: Binary operators should be put on the same l
mmenke 2014/09/03 18:02:35 We should have the http status code in onResponseS
+ }
+ }
+
+ @SmallTest
+ @Feature({"Cronet"})
+ public void testCreateURLRequest() throws Exception {
+ CronetTestActivity activity = launchCronetTestApp();
+ String URL = "http://127.0.0.1:8000";
+
+ // Make sure the activity was created as expected.
+ assertNotNull(activity);
+
+ HashMap<String, String> headers = new HashMap<String, String>();
+ MockHttpUrlRequestListener listener = new MockHttpUrlRequestListener();
+
+ HttpUrlRequest request = activity.mChromiumRequestFactory.createRequest(
+ URL, HttpUrlRequest.REQUEST_PRIORITY_MEDIUM, headers, listener);
+ request.start();
+ listener.blockForComplete();
+
+ // Make sure that the URL is set as expected.
+ assertNotNull(activity);
+ assertEquals(200, request.getHttpStatusCode());
mmenke 2014/09/03 18:02:34 Are we talking to a real server here? If so, seem
+ }
+
+ @SmallTest
+ @Feature({"Cronet"})
+ public void testFailedURLRequest() throws Exception {
+ CronetTestActivity activity = launchCronetTestApp();
+ String URL = "http://127.0.0.1:8000";
+
+ // Fail this job.
+ URLRequestJobUtil.nativeFailURLRequestJob(URL);
+
+ // Make sure the activity was created as expected.
+ assertNotNull(activity);
+
+ HashMap<String, String> headers = new HashMap<String, String>();
+ MockHttpUrlRequestListener listener = new MockHttpUrlRequestListener();
+
+ HttpUrlRequest request = activity.mChromiumRequestFactory.createRequest(
+ URL, HttpUrlRequest.REQUEST_PRIORITY_MEDIUM, headers, listener);
+ request.start();
+ listener.blockForComplete();
+
+ // Make sure that the URL is set as expected.
+ assertNotNull(activity);
+ assertEquals(0, request.getHttpStatusCode());
+ }
+
+}

Powered by Google App Engine
This is Rietveld 408576698