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

Unified Diff: chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncCustomizationFragmentTest.java

Issue 1222203008: [Sync] Add priority preferences datatype for mobile (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix test Created 5 years, 5 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/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncCustomizationFragmentTest.java
diff --git a/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncCustomizationFragmentTest.java b/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncCustomizationFragmentTest.java
index 5a279b0f0c196b4b7f9058a4dbb473bf1d0a03eb..d365a922f50c942dcfea9dc564b397f7ca136e57 100644
--- a/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncCustomizationFragmentTest.java
+++ b/chrome/android/sync_shell/javatests/src/org/chromium/chrome/browser/sync/SyncCustomizationFragmentTest.java
@@ -60,6 +60,8 @@ public class SyncCustomizationFragmentTest extends SyncTestBase {
UI_DATATYPES.put(ModelType.PASSWORD, SyncCustomizationFragment.PREFERENCE_SYNC_PASSWORDS);
UI_DATATYPES.put(ModelType.PROXY_TABS,
SyncCustomizationFragment.PREFERENCE_SYNC_RECENT_TABS);
+ UI_DATATYPES.put(ModelType.PREFERENCE,
+ SyncCustomizationFragment.PREFERENCE_SYNC_SETTINGS);
}
private Activity mActivity;
@@ -149,12 +151,6 @@ public class SyncCustomizationFragmentTest extends SyncTestBase {
assertFalse(AndroidSyncSettings.isChromeSyncEnabled(mContext));
}
- /**
- * Make sure that the encryption UI presents the correct options.
- *
- * By default it should show the CUSTOM and KEYSTORE options, in that order.
- * KEYSTORE should be selected but both should be enabled.
- */
@SmallTest
@Feature({"Sync"})
public void testSettingDataTypes() throws Exception {
@@ -172,6 +168,9 @@ public class SyncCustomizationFragmentTest extends SyncTestBase {
}
Set<ModelType> expectedTypes = EnumSet.copyOf(dataTypes.keySet());
+ // TODO(zea): update this once preferences are supported.
+ expectedTypes.remove(ModelType.PREFERENCE);
+ expectedTypes.add(ModelType.PRIORITY_PREFERENCE);
assertDataTypesAre(expectedTypes);
togglePreference(dataTypes.get(ModelType.AUTOFILL));
togglePreference(dataTypes.get(ModelType.PASSWORD));
@@ -184,6 +183,12 @@ public class SyncCustomizationFragmentTest extends SyncTestBase {
assertDataTypesAre(expectedTypes);
}
+ /**
+ * Make sure that the encryption UI presents the correct options.
+ *
+ * By default it should show the CUSTOM and KEYSTORE options, in that order.
+ * KEYSTORE should be selected but both should be enabled.
+ */
@SmallTest
@Feature({"Sync"})
public void testDefaultEncryptionOptions() throws Exception {

Powered by Google App Engine
This is Rietveld 408576698