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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/invalidation/ChromeBrowserSyncAdapterTest.java

Issue 2860843002: Revert of Reland: Convert ChromeActivityTestCaseBase direct children to JUnit4 (Closed)
Patch Set: Created 3 years, 8 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: chrome/android/javatests/src/org/chromium/chrome/browser/invalidation/ChromeBrowserSyncAdapterTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/invalidation/ChromeBrowserSyncAdapterTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/invalidation/ChromeBrowserSyncAdapterTest.java
index a58bf2f9f64d3fc8ad1ab4866fa6315559aef36d..80b64f1d6b2fe2c076da8f374ddea4983a9529ca 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/invalidation/ChromeBrowserSyncAdapterTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/invalidation/ChromeBrowserSyncAdapterTest.java
@@ -12,25 +12,15 @@
import android.content.Intent;
import android.content.SyncResult;
import android.os.Bundle;
-import android.support.test.InstrumentationRegistry;
import android.support.test.filters.MediumTest;
-
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.runner.RunWith;
import org.chromium.base.ApplicationStatus;
import org.chromium.base.CommandLine;
-import org.chromium.base.test.util.CommandLineFlags;
import org.chromium.base.test.util.Feature;
import org.chromium.base.test.util.RetryOnFailure;
import org.chromium.base.test.util.ScalableTimeout;
import org.chromium.chrome.browser.ChromeActivity;
-import org.chromium.chrome.browser.ChromeSwitches;
-import org.chromium.chrome.test.ChromeActivityTestRule;
-import org.chromium.chrome.test.ChromeJUnit4ClassRunner;
+import org.chromium.chrome.test.ChromeActivityTestCaseBase;
import org.chromium.components.invalidation.PendingInvalidation;
import org.chromium.components.signin.AccountManagerHelper;
import org.chromium.components.sync.AndroidSyncSettings;
@@ -40,14 +30,7 @@
/**
* Tests for ChromeBrowserSyncAdapter.
*/
-@RunWith(ChromeJUnit4ClassRunner.class)
-@CommandLineFlags.Add({ChromeSwitches.DISABLE_FIRST_RUN_EXPERIENCE,
- ChromeActivityTestRule.DISABLE_NETWORK_PREDICTION_FLAG})
-public class ChromeBrowserSyncAdapterTest {
- @Rule
- public ChromeActivityTestRule<ChromeActivity> mActivityTestRule =
- new ChromeActivityTestRule<>(ChromeActivity.class);
-
+public class ChromeBrowserSyncAdapterTest extends ChromeActivityTestCaseBase<ChromeActivity> {
private static final Account TEST_ACCOUNT =
AccountManagerHelper.createAccountFromName("test@gmail.com");
private static final long WAIT_FOR_LAUNCHER_MS = ScalableTimeout.scaleTimeout(10 * 1000);
@@ -82,18 +65,25 @@
}
}
- @Before
- public void setUp() throws Exception {
- mActivityTestRule.startMainActivityOnBlankPage();
- mSyncAdapter =
- new TestSyncAdapter(InstrumentationRegistry.getInstrumentation().getTargetContext(),
- mActivityTestRule.getActivity().getApplication());
+ public ChromeBrowserSyncAdapterTest() {
+ super(ChromeActivity.class);
+ }
+
+ @Override
+ protected void setUp() throws Exception {
+ super.setUp();
+ mSyncAdapter = new TestSyncAdapter(
+ getInstrumentation().getTargetContext(), getActivity().getApplication());
+ }
+
+ @Override
+ public void startMainActivity() throws InterruptedException {
+ startMainActivityOnBlankPage();
}
private void performSyncWithBundle(Bundle bundle) {
mSyncAdapter.onPerformSync(TEST_ACCOUNT, bundle,
- AndroidSyncSettings.getContractAuthority(mActivityTestRule.getActivity()), null,
- new SyncResult());
+ AndroidSyncSettings.getContractAuthority(getActivity()), null, new SyncResult());
}
private void sendChromeToBackground(Activity activity) {
@@ -113,18 +103,16 @@
return ApplicationStatus.hasVisibleActivities();
}
- @Test
@MediumTest
@Feature({"Sync"})
@RetryOnFailure
public void testRequestSyncNoInvalidationData() {
performSyncWithBundle(new Bundle());
- Assert.assertTrue(mSyncAdapter.mInvalidatedAllTypes);
- Assert.assertFalse(mSyncAdapter.mInvalidated);
- Assert.assertTrue(CommandLine.isInitialized());
+ assertTrue(mSyncAdapter.mInvalidatedAllTypes);
+ assertFalse(mSyncAdapter.mInvalidated);
+ assertTrue(CommandLine.isInitialized());
}
- @Test
@MediumTest
@Feature({"Sync"})
public void testRequestSyncSpecificDataType() {
@@ -136,28 +124,26 @@
performSyncWithBundle(
PendingInvalidation.createBundle(objectId, objectSource, version, payload));
- Assert.assertFalse(mSyncAdapter.mInvalidatedAllTypes);
- Assert.assertTrue(mSyncAdapter.mInvalidated);
- Assert.assertEquals(objectSource, mSyncAdapter.mObjectSource);
- Assert.assertEquals(objectId, mSyncAdapter.mObjectId);
- Assert.assertEquals(version, mSyncAdapter.mVersion);
- Assert.assertEquals(payload, mSyncAdapter.mPayload);
- Assert.assertTrue(CommandLine.isInitialized());
+ assertFalse(mSyncAdapter.mInvalidatedAllTypes);
+ assertTrue(mSyncAdapter.mInvalidated);
+ assertEquals(objectSource, mSyncAdapter.mObjectSource);
+ assertEquals(objectId, mSyncAdapter.mObjectId);
+ assertEquals(version, mSyncAdapter.mVersion);
+ assertEquals(payload, mSyncAdapter.mPayload);
+ assertTrue(CommandLine.isInitialized());
}
- @Test
@MediumTest
@Feature({"Sync"})
@RetryOnFailure
public void testRequestSyncWhenChromeInBackground() {
- sendChromeToBackground(mActivityTestRule.getActivity());
+ sendChromeToBackground(getActivity());
performSyncWithBundle(new Bundle());
- Assert.assertFalse(mSyncAdapter.mInvalidatedAllTypes);
- Assert.assertFalse(mSyncAdapter.mInvalidated);
- Assert.assertTrue(CommandLine.isInitialized());
+ assertFalse(mSyncAdapter.mInvalidatedAllTypes);
+ assertFalse(mSyncAdapter.mInvalidated);
+ assertTrue(CommandLine.isInitialized());
}
- @Test
@MediumTest
@Feature({"Sync"})
@RetryOnFailure
@@ -165,7 +151,7 @@
Bundle extras = new Bundle();
extras.putBoolean(ContentResolver.SYNC_EXTRAS_INITIALIZE, true);
performSyncWithBundle(extras);
- Assert.assertFalse(mSyncAdapter.mInvalidatedAllTypes);
- Assert.assertFalse(mSyncAdapter.mInvalidated);
+ assertFalse(mSyncAdapter.mInvalidatedAllTypes);
+ assertFalse(mSyncAdapter.mInvalidated);
}
}

Powered by Google App Engine
This is Rietveld 408576698