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

Unified Diff: chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc

Issue 2058083002: MD User Manager: Error message if pref doesn't allow creating supervised user. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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
Index: chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
diff --git a/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc b/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
index f4b52b3a53eefe8d405c6fb99e0fcace3a2f97bf..ea3db406c4d9ea3fcb11e13907a87af9c835c297 100644
--- a/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
+++ b/chrome/browser/ui/webui/signin/signin_create_profile_handler_unittest.cc
@@ -580,6 +580,18 @@ TEST_F(SigninCreateProfileHandlerTest, NotAllowedToCreateSupervisedUser) {
list_args.AppendString(custodian()->GetPath().value());
handler()->CreateProfile(&list_args);
- // Expect nothing to happen.
- EXPECT_EQ(0U, web_ui()->call_data().size());
+ // Expect a JS callbacks containing an error message.
+ EXPECT_EQ(1U, web_ui()->call_data().size());
+
+ EXPECT_EQ(kTestWebUIResponse, web_ui()->call_data()[0]->function_name());
+
+ std::string callback_name;
+ ASSERT_TRUE(web_ui()->call_data()[0]->arg1()->GetAsString(&callback_name));
+ EXPECT_EQ("create-profile-error", callback_name);
+
+ base::string16 expected_error_message = l10n_util::GetStringUTF16(
+ IDS_PROFILES_CREATE_SUPERVISED_NOT_ALLOWED_BY_POLICY);
+ base::string16 error_message;
+ ASSERT_TRUE(web_ui()->call_data()[0]->arg2()->GetAsString(&error_message));
+ EXPECT_EQ(expected_error_message, error_message);
}

Powered by Google App Engine
This is Rietveld 408576698