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

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

Issue 2892013002: [Cronet] Clean up tests (Closed)
Patch Set: fix Created 3 years, 7 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/net/DiskStorageTest.java
diff --git a/components/cronet/android/test/javatests/src/org/chromium/net/DiskStorageTest.java b/components/cronet/android/test/javatests/src/org/chromium/net/DiskStorageTest.java
index 0154ceee8a98602045f9459f7ef5204e93b0d313..7833ed5a7e1c9c18dc845df0cb2b6295f1741024 100644
--- a/components/cronet/android/test/javatests/src/org/chromium/net/DiskStorageTest.java
+++ b/components/cronet/android/test/javatests/src/org/chromium/net/DiskStorageTest.java
@@ -21,7 +21,6 @@ import java.util.Arrays;
* Test CronetEngine disk storage.
*/
public class DiskStorageTest extends CronetTestBase {
- private CronetTestFramework mTestFramework;
private String mReadOnlyStoragePath;
@Override
@@ -55,16 +54,16 @@ public class DiskStorageTest extends CronetTestBase {
builder.setStoragePath(mReadOnlyStoragePath);
builder.enableHttpCache(CronetEngine.Builder.HTTP_CACHE_DISK, 1024 * 1024);
- mTestFramework = new CronetTestFramework(null, null, getContext(), builder);
+ CronetEngine cronetEngine = builder.build();
TestUrlRequestCallback callback = new TestUrlRequestCallback();
String url = NativeTestServer.getFileURL("/cacheable.txt");
- UrlRequest.Builder requestBuilder = mTestFramework.mCronetEngine.newUrlRequestBuilder(
- url, callback, callback.getExecutor());
+ UrlRequest.Builder requestBuilder =
+ cronetEngine.newUrlRequestBuilder(url, callback, callback.getExecutor());
UrlRequest urlRequest = requestBuilder.build();
urlRequest.start();
callback.blockForDone();
assertEquals(200, callback.mResponseInfo.getHttpStatusCode());
- mTestFramework.mCronetEngine.shutdown();
+ cronetEngine.shutdown();
FileInputStream newVersionFile = null;
// Make sure that version file is in readOnlyStoragePath.
File versionFile = new File(mReadOnlyStoragePath + "/version");
@@ -118,16 +117,16 @@ public class DiskStorageTest extends CronetTestBase {
builder.setStoragePath(CronetTestFramework.getTestStorage(getContext()));
builder.enableHttpCache(CronetEngine.Builder.HTTP_CACHE_DISK, 1024 * 1024);
- mTestFramework = new CronetTestFramework(null, null, getContext(), builder);
+ CronetEngine cronetEngine = builder.build();
TestUrlRequestCallback callback = new TestUrlRequestCallback();
String url = NativeTestServer.getFileURL("/cacheable.txt");
- UrlRequest.Builder requestBuilder = mTestFramework.mCronetEngine.newUrlRequestBuilder(
- url, callback, callback.getExecutor());
+ UrlRequest.Builder requestBuilder =
+ cronetEngine.newUrlRequestBuilder(url, callback, callback.getExecutor());
UrlRequest urlRequest = requestBuilder.build();
urlRequest.start();
callback.blockForDone();
assertEquals(200, callback.mResponseInfo.getHttpStatusCode());
- mTestFramework.mCronetEngine.shutdown();
+ cronetEngine.shutdown();
FileInputStream newVersionFile = null;
try {
newVersionFile = new FileInputStream(versionFile);
@@ -159,16 +158,16 @@ public class DiskStorageTest extends CronetTestBase {
builder.setStoragePath(CronetTestFramework.getTestStorage(getContext()));
builder.enableHttpCache(CronetEngine.Builder.HTTP_CACHE_DISK, 1024 * 1024);
- mTestFramework = new CronetTestFramework(null, null, getContext(), builder);
+ CronetEngine cronetEngine = builder.build();
TestUrlRequestCallback callback = new TestUrlRequestCallback();
String url = NativeTestServer.getFileURL("/cacheable.txt");
- UrlRequest.Builder requestBuilder = mTestFramework.mCronetEngine.newUrlRequestBuilder(
- url, callback, callback.getExecutor());
+ UrlRequest.Builder requestBuilder =
+ cronetEngine.newUrlRequestBuilder(url, callback, callback.getExecutor());
UrlRequest urlRequest = requestBuilder.build();
urlRequest.start();
callback.blockForDone();
assertEquals(200, callback.mResponseInfo.getHttpStatusCode());
- mTestFramework.mCronetEngine.shutdown();
+ cronetEngine.shutdown();
// Create a dummy file in storage directory.
String testStorage = CronetTestFramework.getTestStorage(getContext());

Powered by Google App Engine
This is Rietveld 408576698