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

Unified Diff: components/cronet/android/test/javatests/src/org/chromium/net/UploadTest.java

Issue 2085353006: [Cronet] Fix UploadTest#testUploadChannelWithReadError (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 6 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/cronet/android/test/javatests/src/org/chromium/net/UploadTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/UploadTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/UploadTest.java
index 61e5f3407ab52291712469c16dc153418efcabe1..6099939fd60c507ae0c3c910f103a20f5400cf21 100644
--- a/components/cronet/android/test/javatests/src/org/chromium/net/UploadTest.java
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/UploadTest.java
@@ -210,7 +210,7 @@ public class UploadTest extends CronetTestBase {
request.start();
listener.blockForComplete();
- assertEquals(200, listener.mHttpStatusCode);
+ assertEquals(0, listener.mHttpStatusCode);
assertEquals(UPLOAD_CHANNEL_DATA + "\0\0", listener.mResponseAsString);
mmenke 2016/06/23 21:47:07 Is this second check still true?
xunjieli 2016/06/23 21:49:13 I am not sure. Still running test. Will update onc
xunjieli 2016/06/23 22:13:07 Done. It is null. We aren't able to assert on the
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698