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

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

Issue 393343002: Rename "managed (mode|user)" to "supervised user" (part 7) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: more alphabetize (and rebase again) Created 6 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/browser/chromeos/login/managed/managed_user_password_browsertest.cc
diff --git a/chrome/browser/chromeos/login/managed/managed_user_password_browsertest.cc b/chrome/browser/chromeos/login/managed/managed_user_password_browsertest.cc
deleted file mode 100644
index 161c9914de2ea50d2885712d35d8e2acec2e5e97..0000000000000000000000000000000000000000
--- a/chrome/browser/chromeos/login/managed/managed_user_password_browsertest.cc
+++ /dev/null
@@ -1,261 +0,0 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-#include <string>
-
-#include "base/compiler_specific.h"
-#include "base/run_loop.h"
-#include "base/strings/utf_string_conversions.h"
-#include "base/threading/sequenced_worker_pool.h"
-#include "chrome/browser/chrome_notification_types.h"
-#include "chrome/browser/chromeos/login/login_manager_test.h"
-#include "chrome/browser/chromeos/login/managed/managed_user_test_base.h"
-#include "chrome/browser/chromeos/login/managed/supervised_user_authentication.h"
-#include "chrome/browser/chromeos/login/startup_utils.h"
-#include "chrome/browser/chromeos/login/ui/login_display_host_impl.h"
-#include "chrome/browser/chromeos/login/ui/webui_login_view.h"
-#include "chrome/browser/chromeos/login/users/supervised_user_manager.h"
-#include "chrome/browser/chromeos/net/network_portal_detector_test_impl.h"
-#include "chrome/browser/chromeos/settings/stub_cros_settings_provider.h"
-#include "chrome/browser/supervised_user/supervised_user_constants.h"
-#include "chrome/browser/supervised_user/supervised_user_registration_utility.h"
-#include "chrome/browser/supervised_user/supervised_user_registration_utility_stub.h"
-#include "chrome/browser/supervised_user/supervised_user_shared_settings_service.h"
-#include "chrome/browser/supervised_user/supervised_user_shared_settings_service_factory.h"
-#include "chrome/browser/supervised_user/supervised_user_sync_service.h"
-#include "chrome/browser/supervised_user/supervised_user_sync_service_factory.h"
-#include "chromeos/cryptohome/mock_async_method_caller.h"
-#include "chromeos/cryptohome/mock_homedir_methods.h"
-#include "content/public/browser/notification_service.h"
-#include "content/public/test/browser_test_utils.h"
-#include "content/public/test/test_utils.h"
-#include "sync/api/attachments/attachment_service_proxy_for_test.h"
-#include "sync/api/fake_sync_change_processor.h"
-#include "sync/api/sync_change.h"
-#include "sync/api/sync_error_factory_mock.h"
-#include "sync/protocol/sync.pb.h"
-
-using testing::_;
-using chromeos::ManagedUserTestBase;
-using chromeos::kTestSupervisedUserDisplayName;
-using chromeos::kTestManager;
-
-namespace chromeos {
-
-class SupervisedUserPasswordTest : public ManagedUserTestBase {
- public:
- SupervisedUserPasswordTest() : ManagedUserTestBase() {}
-
- private:
- DISALLOW_COPY_AND_ASSIGN(SupervisedUserPasswordTest);
-};
-
-class SupervisedUserPasswordManagerTest : public ManagedUserTestBase {
- public:
- SupervisedUserPasswordManagerTest() : ManagedUserTestBase() {}
-
- private:
- DISALLOW_COPY_AND_ASSIGN(SupervisedUserPasswordManagerTest);
-};
-
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PRE_PRE_PasswordChangeFromUserTest) {
- PrepareUsers();
-}
-
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PRE_PasswordChangeFromUserTest) {
- StartFlowLoginAsManager();
- FillNewUserData(kTestSupervisedUserDisplayName);
- StartUserCreation("managed-user-creation-next-button",
- kTestSupervisedUserDisplayName);
-}
-
-// Supervised user signs in, get sync notification about password update, and
-// schedules password migration.
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PasswordChangeFromUserTest) {
- SigninAsSupervisedUser(true, 0, kTestSupervisedUserDisplayName);
-
- const User* user = UserManager::Get()->GetUsers().at(0);
- std::string sync_id =
- UserManager::Get()->GetSupervisedUserManager()->GetUserSyncId(
- user->email());
- base::DictionaryValue password;
- password.SetIntegerWithoutPathExpansion(
- kSchemaVersion, SupervisedUserAuthentication::SCHEMA_SALT_HASHED);
- password.SetIntegerWithoutPathExpansion(kPasswordRevision, 2);
-
- password.SetStringWithoutPathExpansion(kPasswordSignature, "signature");
- password.SetStringWithoutPathExpansion(kEncryptedPassword,
- "new-encrypted-password");
-
- shared_settings_adapter_->AddChange(
- sync_id, supervised_users::kChromeOSPasswordData, password, true, false);
- content::RunAllPendingInMessageLoop();
-}
-
-// Supervised user signs in for second time, and actual password migration takes
-// place.
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PasswordChangeFromUserTest) {
- EXPECT_CALL(*mock_homedir_methods_, MountEx(_, _, _, _)).Times(1);
- EXPECT_CALL(*mock_homedir_methods_, UpdateKeyEx(_, _, _, _, _)).Times(1);
- SigninAsSupervisedUser(false, 0, kTestSupervisedUserDisplayName);
- testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
-}
-
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PRE_PRE_PasswordChangeFromManagerTest) {
- PrepareUsers();
-}
-
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PRE_PasswordChangeFromManagerTest) {
- StartFlowLoginAsManager();
- FillNewUserData(kTestSupervisedUserDisplayName);
- StartUserCreation("managed-user-creation-next-button",
- kTestSupervisedUserDisplayName);
-}
-
-// Manager signs in, gets sync notification about supervised user password
-// update, and performs migration.
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PasswordChangeFromManagerTest) {
- const User* managed_user = UserManager::Get()->GetUsers().at(0);
-
- SigninAsManager(1);
-
- EXPECT_CALL(*mock_homedir_methods_, AddKeyEx(_, _, _, _, _)).Times(1);
-
- std::string sync_id =
- UserManager::Get()->GetSupervisedUserManager()->GetUserSyncId(
- managed_user->email());
-
- ::sync_pb::ManagedUserSpecifics managed_user_proto;
-
- managed_user_proto.set_id(sync_id);
- managed_user_proto.set_name(kTestSupervisedUserDisplayName);
- managed_user_proto.set_acknowledged(true);
- managed_user_proto.set_master_key("master key");
- managed_user_proto.set_password_signature_key("signature_key");
- managed_user_proto.set_password_encryption_key("encryption_key");
-
- managed_users_adapter_->AddChange(managed_user_proto, false);
- content::RunAllPendingInMessageLoop();
-
- base::DictionaryValue password;
- password.SetIntegerWithoutPathExpansion(
- kSchemaVersion, SupervisedUserAuthentication::SCHEMA_SALT_HASHED);
- password.SetIntegerWithoutPathExpansion(kPasswordRevision, 2);
-
- password.SetStringWithoutPathExpansion(kPasswordSignature, "signature");
- password.SetStringWithoutPathExpansion(kEncryptedPassword,
- "new-encrypted-password");
- shared_settings_adapter_->AddChange(
- sync_id, supervised_users::kChromeOSPasswordData, password, true, false);
- content::RunAllPendingInMessageLoop();
-
- testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
-}
-
-// After that supervised user signs in, and no password change happens.
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PasswordChangeFromManagerTest) {
- EXPECT_CALL(*mock_homedir_methods_, MountEx(_, _, _, _)).Times(1);
- EXPECT_CALL(*mock_homedir_methods_, UpdateKeyEx(_, _, _, _, _)).Times(0);
- SigninAsSupervisedUser(false, 1, kTestSupervisedUserDisplayName);
- testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
-}
-
-IN_PROC_BROWSER_TEST_F(
- SupervisedUserPasswordTest,
- DISABLED_PRE_PRE_PRE_PRE_PasswordChangeUserAndManagerTest) {
- PrepareUsers();
-}
-
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PRE_PRE_PasswordChangeUserAndManagerTest) {
- StartFlowLoginAsManager();
- FillNewUserData(kTestSupervisedUserDisplayName);
- StartUserCreation("managed-user-creation-next-button",
- kTestSupervisedUserDisplayName);
-}
-
-// Supervised user signs in, get sync notification about password update, and
-// schedules password migration.
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PRE_PasswordChangeUserAndManagerTest) {
- SigninAsSupervisedUser(true, 0, kTestSupervisedUserDisplayName);
-
- const User* user = UserManager::Get()->GetUsers().at(0);
- std::string sync_id =
- UserManager::Get()->GetSupervisedUserManager()->GetUserSyncId(
- user->email());
- base::DictionaryValue password;
- password.SetIntegerWithoutPathExpansion(
- kSchemaVersion, SupervisedUserAuthentication::SCHEMA_SALT_HASHED);
- password.SetIntegerWithoutPathExpansion(kPasswordRevision, 2);
-
- password.SetStringWithoutPathExpansion(kPasswordSignature, "signature");
- password.SetStringWithoutPathExpansion(kEncryptedPassword,
- "new-encrypted-password");
-
- shared_settings_adapter_->AddChange(
- sync_id, supervised_users::kChromeOSPasswordData, password, true, false);
- content::RunAllPendingInMessageLoop();
-}
-
-// After that manager signs in, and also detects password change. Manager
-// performs the migration.
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PRE_PasswordChangeUserAndManagerTest) {
- const User* managed_user = UserManager::Get()->GetUsers().at(0);
-
- SigninAsManager(1);
-
- EXPECT_CALL(*mock_homedir_methods_, AddKeyEx(_, _, _, _, _)).Times(1);
-
- std::string sync_id =
- UserManager::Get()->GetSupervisedUserManager()->GetUserSyncId(
- managed_user->email());
-
- ::sync_pb::ManagedUserSpecifics managed_user_proto;
-
- managed_user_proto.set_id(sync_id);
- managed_user_proto.set_name(kTestSupervisedUserDisplayName);
- managed_user_proto.set_acknowledged(true);
- managed_user_proto.set_master_key("master key");
- managed_user_proto.set_password_signature_key("signature_key");
- managed_user_proto.set_password_encryption_key("encryption_key");
-
- managed_users_adapter_->AddChange(managed_user_proto, false);
- content::RunAllPendingInMessageLoop();
-
- base::DictionaryValue password;
- password.SetIntegerWithoutPathExpansion(
- kSchemaVersion, SupervisedUserAuthentication::SCHEMA_SALT_HASHED);
- password.SetIntegerWithoutPathExpansion(kPasswordRevision, 2);
-
- password.SetStringWithoutPathExpansion(kPasswordSignature, "signature");
- password.SetStringWithoutPathExpansion(kEncryptedPassword,
- "new-encrypted-password");
- shared_settings_adapter_->AddChange(
- sync_id, supervised_users::kChromeOSPasswordData, password, true, false);
- content::RunAllPendingInMessageLoop();
-
- testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
-}
-
-// When supervised user signs in, password is already migrated, so no migration
-// should be attempted.
-IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
- DISABLED_PasswordChangeUserAndManagerTest) {
- EXPECT_CALL(*mock_homedir_methods_, MountEx(_, _, _, _)).Times(1);
- EXPECT_CALL(*mock_homedir_methods_, UpdateKeyEx(_, _, _, _, _)).Times(0);
- SigninAsSupervisedUser(false, 1, kTestSupervisedUserDisplayName);
- testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
-}
-
-} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698