Tests for supervised users password sync:

chrome/browser/chromeos/login/managed/managed_user_password_browsertest.cc

+// Copyright 2013 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_display_host_impl.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/supervised_user_manager.h"
+#include "chrome/browser/chromeos/login/webui_login_view.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/managed_mode/managed_user_constants.h"
+#include "chrome/browser/managed_mode/managed_user_registration_utility.h"
+#include "chrome/browser/managed_mode/managed_user_registration_utility_stub.h"
+#include "chrome/browser/managed_mode/managed_user_shared_settings_service.h"
+#include "chrome/browser/managed_mode/managed_user_shared_settings_service_factory.h"
+#include "chrome/browser/managed_mode/managed_user_sync_service.h"
+#include "chrome/browser/managed_mode/managed_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::testing::ManagedUserTestBase;
+using chromeos::testing::kTestSupervisedUserDisplayName;
+using chromeos::testing::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,
+                       PRE_PRE_PRE_PasswordChangeFromUserTest) {
+  PrepareUsers();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       PRE_PRE_PasswordChangeFromUserTest) {
+  StartFlowLoginAsManager();
+  FillNewUserData(kTestSupervisedUserDisplayName);
+  StartUserCreation("managed-user-creation-next-button",
+                    kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,

[Nikita (slow), 2014/04/23 09:50:17]

This file doesn't have a single comment.
Please at least add short comments for each test case describing what it does.

[Denis Kuznetsov (DE-MUC), 2014/04/24 17:44:09]

Done.

+                       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, managed_users::kChromeOSPasswordData, password, true);
+  content::RunAllPendingInMessageLoop();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest, 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,
+                       PRE_PRE_PRE_PasswordChangeFromManagerTest) {
+  PrepareUsers();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       PRE_PRE_PasswordChangeFromManagerTest) {
+  StartFlowLoginAsManager();
+  FillNewUserData(kTestSupervisedUserDisplayName);
+  StartUserCreation("managed-user-creation-next-button",
+                    kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       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);
+  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, managed_users::kChromeOSPasswordData, password, true);
+  content::RunAllPendingInMessageLoop();
+
+  ::testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       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,
+                       PRE_PRE_PRE_PRE_PasswordChangeUserAndManagerTest) {
+  PrepareUsers();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       PRE_PRE_PRE_PasswordChangeUserAndManagerTest) {
+  StartFlowLoginAsManager();
+  FillNewUserData(kTestSupervisedUserDisplayName);
+  StartUserCreation("managed-user-creation-next-button",
+                    kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       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, managed_users::kChromeOSPasswordData, password, true);
+  content::RunAllPendingInMessageLoop();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       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);
+  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, managed_users::kChromeOSPasswordData, password, true);
+  content::RunAllPendingInMessageLoop();
+
+  ::testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserPasswordTest,
+                       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