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

Unified Diff: chrome/browser/chromeos/login/managed/managed_user_test_base.cc

Issue 270563002: Componentize LoginManagerTest. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 7 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/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));

Powered by Google App Engine
This is Rietveld 408576698