| Index: chrome/browser/chromeos/login/managed/managed_user_test_base.cc
|
| diff --git a/chrome/browser/chromeos/login/managed/managed_user_test_base.cc b/chrome/browser/chromeos/login/managed/managed_user_test_base.cc
|
| index 05cdc7a90c865b9ee408a41be780f5fca544dc29..a6196f8a83507bc193443e554730832e61d6094f 100644
|
| --- a/chrome/browser/chromeos/login/managed/managed_user_test_base.cc
|
| +++ b/chrome/browser/chromeos/login/managed/managed_user_test_base.cc
|
| @@ -14,7 +14,6 @@
|
| #include "base/threading/sequenced_worker_pool.h"
|
| #include "chrome/browser/chrome_notification_types.h"
|
| #include "chrome/browser/chromeos/login/login_display_host_impl.h"
|
| -#include "chrome/browser/chromeos/login/login_manager_test.h"
|
| #include "chrome/browser/chromeos/login/managed/supervised_user_authentication.h"
|
| #include "chrome/browser/chromeos/login/startup_utils.h"
|
| #include "chrome/browser/chromeos/login/supervised_user_manager.h"
|
| @@ -160,18 +159,27 @@ void ManagedUsersSharedSettingsSyncTestAdapter::AddChange(
|
| }
|
|
|
| ManagedUserTestBase::ManagedUserTestBase()
|
| - : LoginManagerTest(true),
|
| - mock_async_method_caller_(NULL),
|
| + : mock_async_method_caller_(NULL),
|
| mock_homedir_methods_(NULL),
|
| network_portal_detector_(NULL),
|
| - registration_utility_stub_(NULL) {
|
| + registration_utility_stub_(NULL),
|
| + login_manager_test_helper_(new LoginManagerTestHelper(true)) {
|
| + set_exit_when_last_browser_closes(false);
|
| }
|
|
|
| ManagedUserTestBase::~ManagedUserTestBase() {
|
| }
|
|
|
| +void ManagedUserTestBase::SetUpOnMainThread() {
|
| + login_manager_test_helper_->SetUp();
|
| +}
|
| +
|
| +void ManagedUserTestBase::SetUpCommandLine(CommandLine* command_line) {
|
| + login_manager_test_helper_->SetUpCommandLine(command_line);
|
| +}
|
| +
|
| void ManagedUserTestBase::SetUpInProcessBrowserTestFixture() {
|
| - LoginManagerTest::SetUpInProcessBrowserTestFixture();
|
| + login_manager_test_helper_->SetUpLoginUtils();
|
| mock_async_method_caller_ = new cryptohome::MockAsyncMethodCaller;
|
| mock_async_method_caller_->SetUp(true, cryptohome::MOUNT_ERROR_NONE);
|
| cryptohome::AsyncMethodCaller::InitializeForTesting(
|
| @@ -200,7 +208,7 @@ void ManagedUserTestBase::SetUpInProcessBrowserTestFixture() {
|
| }
|
|
|
| void ManagedUserTestBase::CleanUpOnMainThread() {
|
| - LoginManagerTest::CleanUpOnMainThread();
|
| + login_manager_test_helper_->CleanUp();
|
| }
|
|
|
| void ManagedUserTestBase::TearDown() {
|
| @@ -208,7 +216,7 @@ void ManagedUserTestBase::TearDown() {
|
| cryptohome::HomedirMethods::Shutdown();
|
| mock_homedir_methods_ = NULL;
|
| mock_async_method_caller_ = NULL;
|
| - LoginManagerTest::TearDown();
|
| + InProcessBrowserTest::TearDown();
|
| }
|
|
|
| void ManagedUserTestBase::TearDownInProcessBrowserTestFixture() {
|
| @@ -216,13 +224,14 @@ void ManagedUserTestBase::TearDownInProcessBrowserTestFixture() {
|
| }
|
|
|
| void ManagedUserTestBase::JSEval(const std::string& script) {
|
| - EXPECT_TRUE(content::ExecuteScript(web_contents(), script));
|
| + EXPECT_TRUE(content::ExecuteScript(login_manager_test_helper_->web_contents(),
|
| + script));
|
| }
|
|
|
| void ManagedUserTestBase::JSExpectAsync(const std::string& function) {
|
| bool result;
|
| EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
|
| - web_contents(),
|
| + login_manager_test_helper_->web_contents(),
|
| StringPrintf(
|
| "(%s)(function() { window.domAutomationController.send(true); });",
|
| function.c_str()),
|
| @@ -240,8 +249,8 @@ void ManagedUserTestBase::JSSetTextField(const std::string& element_selector,
|
| }
|
|
|
| void ManagedUserTestBase::PrepareUsers() {
|
| - RegisterUser(kTestManager);
|
| - RegisterUser(kTestOtherUser);
|
| + login_manager_test_helper_->RegisterUser(kTestManager);
|
| + login_manager_test_helper_->RegisterUser(kTestOtherUser);
|
| chromeos::StartupUtils::MarkOobeCompleted();
|
| }
|
|
|
| @@ -250,13 +259,15 @@ void ManagedUserTestBase::StartFlowLoginAsManager() {
|
| JSEval("chrome.send('showLocallyManagedUserCreationScreen')");
|
|
|
| // Read intro and proceed.
|
| - JSExpect(StringPrintf("%s == 'intro'", kCurrentPage));
|
| + login_manager_test_helper_->JSExpect(
|
| + StringPrintf("%s == 'intro'", kCurrentPage));
|
|
|
| JSEval("$('managed-user-creation-start-button').click()");
|
|
|
| // Check that both users appear as managers, and test-manager@gmail.com is
|
| // the first one.
|
| - JSExpect(StringPrintf("%s == 'manager'", kCurrentPage));
|
| + login_manager_test_helper_->JSExpect(
|
| + StringPrintf("%s == 'manager'", kCurrentPage));
|
|
|
| std::string manager_pods =
|
| "document.querySelectorAll('#managed-user-creation-managers-pane "
|
| @@ -267,28 +278,33 @@ void ManagedUserTestBase::StartFlowLoginAsManager() {
|
|
|
| int managers_on_device = 2;
|
|
|
| - JSExpect(StringPrintf("%s.length == 1", selected_manager_pods.c_str()));
|
| + login_manager_test_helper_->JSExpect(
|
| + StringPrintf("%s.length == 1", selected_manager_pods.c_str()));
|
|
|
| - JSExpect(
|
| + login_manager_test_helper_->JSExpect(
|
| StringPrintf("$('managed-user-creation').managerList_.pods.length == %d",
|
| managers_on_device));
|
| - JSExpect(StringPrintf(
|
| + login_manager_test_helper_->JSExpect(StringPrintf(
|
| "%s.length == %d", manager_pods.c_str(), managers_on_device));
|
| - JSExpect(StringPrintf("%s[%d].user.emailAddress == '%s'",
|
| - manager_pods.c_str(),
|
| - 0,
|
| - kTestManager));
|
| + login_manager_test_helper_->JSExpect(
|
| + StringPrintf("%s[%d].user.emailAddress == '%s'",
|
| + manager_pods.c_str(),
|
| + 0,
|
| + kTestManager));
|
|
|
| // Select the first user as manager, and enter password.
|
| - JSExpect("$('managed-user-creation-next-button').disabled");
|
| + login_manager_test_helper_->JSExpect(
|
| + "$('managed-user-creation-next-button').disabled");
|
| JSSetTextField("#managed-user-creation .manager-pod.focused input",
|
| kTestManagerPassword);
|
|
|
| JSEval("$('managed-user-creation').updateNextButtonForManager_()");
|
|
|
| // Next button is now enabled.
|
| - JSExpect("!$('managed-user-creation-next-button').disabled");
|
| - SetExpectedCredentials(kTestManager, kTestManagerPassword);
|
| + login_manager_test_helper_->JSExpect(
|
| + "!$('managed-user-creation-next-button').disabled");
|
| + login_manager_test_helper_->SetExpectedCredentials(kTestManager,
|
| + kTestManagerPassword);
|
| content::WindowedNotificationObserver login_observer(
|
| chrome::NOTIFICATION_LOGIN_USER_PROFILE_PREPARED,
|
| content::NotificationService::AllSources());
|
| @@ -303,11 +319,13 @@ void ManagedUserTestBase::StartFlowLoginAsManager() {
|
| base::RunLoop().RunUntilIdle();
|
|
|
| // Check the page have changed.
|
| - JSExpect(StringPrintf("%s == 'username'", kCurrentPage));
|
| + login_manager_test_helper_->JSExpect(
|
| + StringPrintf("%s == 'username'", kCurrentPage));
|
| }
|
|
|
| void ManagedUserTestBase::FillNewUserData(const std::string& display_name) {
|
| - JSExpect("$('managed-user-creation-next-button').disabled");
|
| + login_manager_test_helper_->JSExpect(
|
| + "$('managed-user-creation-next-button').disabled");
|
| JSSetTextField("#managed-user-creation-name", display_name);
|
| JSEval("$('managed-user-creation').checkUserName_()");
|
|
|
| @@ -319,7 +337,8 @@ void ManagedUserTestBase::FillNewUserData(const std::string& display_name) {
|
| kTestSupervisedUserPassword);
|
|
|
| JSEval("$('managed-user-creation').updateNextButtonForUser_()");
|
| - JSExpect("!$('managed-user-creation-next-button').disabled");
|
| + login_manager_test_helper_->JSExpect(
|
| + "!$('managed-user-creation-next-button').disabled");
|
| }
|
|
|
| void ManagedUserTestBase::StartUserCreation(
|
| @@ -343,7 +362,8 @@ void ManagedUserTestBase::StartUserCreation(
|
| content::BrowserThread::GetBlockingPool()->FlushForTesting();
|
| base::RunLoop().RunUntilIdle();
|
|
|
| - JSExpect(StringPrintf("%s == 'created'", kCurrentPage));
|
| + login_manager_test_helper_->JSExpect(
|
| + StringPrintf("%s == 'created'", kCurrentPage));
|
| JSEval("$('managed-user-creation-gotit-button').click()");
|
| }
|
|
|
| @@ -360,7 +380,7 @@ void ManagedUserTestBase::SigninAsSupervisedUser(
|
| // Created supervised user have to be first in a list.
|
| const User* user = UserManager::Get()->GetUsers().at(user_index);
|
| ASSERT_EQ(base::UTF8ToUTF16(expected_display_name), user->display_name());
|
| - LoginUser(user->email());
|
| + login_manager_test_helper_->LogInUser(user->email());
|
| if (check_homedir_calls)
|
| ::testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
|
| Profile* profile = UserManager::Get()->GetProfileByUser(user);
|
| @@ -374,7 +394,7 @@ void ManagedUserTestBase::SigninAsManager(int user_index) {
|
|
|
| // Created supervised user have to be first in a list.
|
| const User* user = UserManager::Get()->GetUsers().at(user_index);
|
| - LoginUser(user->email());
|
| + login_manager_test_helper_->LogInUser(user->email());
|
| Profile* profile = UserManager::Get()->GetProfileByUser(user);
|
| shared_settings_adapter_.reset(
|
| new ManagedUsersSharedSettingsSyncTestAdapter(profile));
|
| @@ -393,22 +413,22 @@ void ManagedUserTestBase::RemoveSupervisedUser(
|
| ASSERT_EQ(base::UTF8ToUTF16(expected_display_name), user->display_name());
|
|
|
| // Open pod menu.
|
| - JSExpect(
|
| + login_manager_test_helper_->JSExpect(
|
| StringPrintf("!$('pod-row').pods[%d].isActionBoxMenuActive", user_index));
|
| JSEval(StringPrintf(
|
| "$('pod-row').pods[%d].querySelector('.action-box-button').click()",
|
| user_index));
|
| - JSExpect(
|
| + login_manager_test_helper_->JSExpect(
|
| StringPrintf("$('pod-row').pods[%d].isActionBoxMenuActive", user_index));
|
|
|
| // Select "Remove user" element.
|
| - JSExpect(StringPrintf(
|
| + login_manager_test_helper_->JSExpect(StringPrintf(
|
| "$('pod-row').pods[%d].actionBoxRemoveUserWarningElement.hidden",
|
| user_index));
|
| JSEval(StringPrintf(
|
| "$('pod-row').pods[%d].querySelector('.action-box-menu-remove').click()",
|
| user_index));
|
| - JSExpect(StringPrintf(
|
| + login_manager_test_helper_->JSExpect(StringPrintf(
|
| "!$('pod-row').pods[%d].actionBoxRemoveUserWarningElement.hidden",
|
| user_index));
|
|
|
|
|