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

Unified Diff: chrome/browser/sync/sync_ui_util_unittest.cc

Issue 2920853004: [sync] Display an error when sync settings aren't confirmed (Closed)
Patch Set: Reformat, remove browser_options change Created 3 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 | « chrome/browser/sync/sync_ui_util.cc ('k') | chrome/browser/ui/avatar_button_error_controller.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/sync_ui_util_unittest.cc
diff --git a/chrome/browser/sync/sync_ui_util_unittest.cc b/chrome/browser/sync/sync_ui_util_unittest.cc
index 0077699140d70340b29dc2caa1b867d26ec62314..4c599642d91767271216be9e267f83b95da4c00a 100644
--- a/chrome/browser/sync/sync_ui_util_unittest.cc
+++ b/chrome/browser/sync/sync_ui_util_unittest.cc
@@ -48,6 +48,7 @@ enum DistinctState {
STATUS_CASE_AUTH_ERROR,
STATUS_CASE_PROTOCOL_ERROR,
STATUS_CASE_PASSPHRASE_ERROR,
+ STATUS_CASE_CONFIRM_SYNC_SETTINGS,
STATUS_CASE_SYNCED,
STATUS_CASE_SYNC_DISABLED_BY_POLICY,
NUMBER_OF_STATUS_CASES
@@ -173,6 +174,16 @@ void GetDistinctCase(ProfileSyncServiceMock* service,
.WillRepeatedly(Return(false));
return;
}
+ case STATUS_CASE_CONFIRM_SYNC_SETTINGS: {
+ EXPECT_CALL(*service, IsSyncConfirmationNeeded())
+ .WillRepeatedly(Return(true));
+ EXPECT_CALL(*service, IsPassphraseRequired())
+ .WillRepeatedly(Return(false));
+ syncer::SyncEngine::Status status;
+ EXPECT_CALL(*service, QueryDetailedSyncStatus(_))
+ .WillRepeatedly(DoAll(SetArgPointee<0>(status), Return(false)));
+ return;
+ }
case STATUS_CASE_PASSPHRASE_ERROR: {
EXPECT_CALL(*service, IsFirstSetupComplete())
.WillRepeatedly(Return(true));
@@ -238,6 +249,8 @@ sync_ui_util::ActionType GetActionTypeforDistinctCase(int case_number) {
return sync_ui_util::UPGRADE_CLIENT;
case STATUS_CASE_PASSPHRASE_ERROR:
return sync_ui_util::ENTER_PASSPHRASE;
+ case STATUS_CASE_CONFIRM_SYNC_SETTINGS:
+ return sync_ui_util::CONFIRM_SYNC_SETTINGS;
case STATUS_CASE_SYNCED:
return sync_ui_util::NO_ACTION;
case STATUS_CASE_SYNC_DISABLED_BY_POLICY:
@@ -413,3 +426,23 @@ TEST_F(SyncUIUtilTest, ActionableErrorWithPassiveMessage) {
EXPECT_NE(first_actionable_error_status_label,
second_actionable_error_status_label);
}
+
+TEST_F(SyncUIUtilTest, SyncSettingsConfirmationNeededTest) {
+ std::unique_ptr<Profile> profile(MakeSignedInTestingProfile());
+ SigninManagerBase* signin =
+ SigninManagerFactory::GetForProfile(profile.get());
+
+ ProfileSyncServiceMock service(
+ CreateProfileSyncServiceParamsForTest(profile.get()));
+ EXPECT_CALL(service, IsSyncConfirmationNeeded()).WillRepeatedly(Return(true));
+
+ base::string16 actionable_error_status_label;
+ base::string16 link_label;
+ sync_ui_util::ActionType action_type = sync_ui_util::NO_ACTION;
+
+ sync_ui_util::GetStatusLabels(
+ profile.get(), &service, *signin, sync_ui_util::PLAIN_TEXT,
+ &actionable_error_status_label, &link_label, &action_type);
+
+ EXPECT_EQ(action_type, sync_ui_util::CONFIRM_SYNC_SETTINGS);
+}
« no previous file with comments | « chrome/browser/sync/sync_ui_util.cc ('k') | chrome/browser/ui/avatar_button_error_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698