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

Side by Side Diff: components/cronet/android/test/javatests/src/org/chromium/net/CronetUploadTest.java

Issue 2406273002: [Cronet] Test the libcronet that's shipped, not libcronet_test (Closed)
Patch Set: address two remaining comments Created 4 years, 1 month 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 android.test.suitebuilder.annotation.SmallTest; 7 import android.test.suitebuilder.annotation.SmallTest;
8 8
9 import org.chromium.base.test.util.Feature; 9 import org.chromium.base.test.util.Feature;
10 import org.chromium.net.CronetTestBase.OnlyRunNativeCronet; 10 import org.chromium.net.CronetTestBase.OnlyRunNativeCronet;
(...skipping 17 matching lines...) Expand all
28 @Override 28 @Override
29 @SuppressWarnings("PrimitiveArrayPassedToVarargsMethod") 29 @SuppressWarnings("PrimitiveArrayPassedToVarargsMethod")
30 protected void setUp() throws Exception { 30 protected void setUp() throws Exception {
31 super.setUp(); 31 super.setUp();
32 startCronetTestFramework(); 32 startCronetTestFramework();
33 ExecutorService executor = Executors.newSingleThreadExecutor(); 33 ExecutorService executor = Executors.newSingleThreadExecutor();
34 List<byte[]> reads = Arrays.asList("hello".getBytes()); 34 List<byte[]> reads = Arrays.asList("hello".getBytes());
35 mDataProvider = new TestDrivenDataProvider(executor, reads); 35 mDataProvider = new TestDrivenDataProvider(executor, reads);
36 mUploadDataStream = new CronetUploadDataStream(mDataProvider, executor); 36 mUploadDataStream = new CronetUploadDataStream(mDataProvider, executor);
37 mHandler = new TestUploadDataStreamHandler( 37 mHandler = new TestUploadDataStreamHandler(
38 mUploadDataStream.createUploadDataStreamForTesting()); 38 getContext(), mUploadDataStream.createUploadDataStreamForTesting ());
39 } 39 }
40 40
41 @Override 41 @Override
42 protected void tearDown() throws Exception { 42 protected void tearDown() throws Exception {
43 // Destroy handler's native objects. 43 // Destroy handler's native objects.
44 mHandler.destroyNativeObjects(); 44 mHandler.destroyNativeObjects();
45 super.tearDown(); 45 super.tearDown();
46 } 46 }
47 47
48 /** 48 /**
(...skipping 246 matching lines...) Expand 10 before | Expand all | Expand 10 after
295 // will then destroy the CronetUploadDataStreamAdapter. 295 // will then destroy the CronetUploadDataStreamAdapter.
296 mHandler.destroyNativeObjects(); 296 mHandler.destroyNativeObjects();
297 297
298 // Signal rewind completes, and wait for init to complete. 298 // Signal rewind completes, and wait for init to complete.
299 mDataProvider.onRewindSucceeded(mUploadDataStream); 299 mDataProvider.onRewindSucceeded(mUploadDataStream);
300 300
301 assertEquals(1, mDataProvider.getNumRewindCalls()); 301 assertEquals(1, mDataProvider.getNumRewindCalls());
302 assertEquals(1, mDataProvider.getNumReadCalls()); 302 assertEquals(1, mDataProvider.getNumReadCalls());
303 } 303 }
304 } 304 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698