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

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

Issue 243103007: Tests for supervised users password sync: (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use sprint instead of concatenation, +1 test Created 6 years, 8 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_creation_browsertest.cc
diff --git a/chrome/browser/chromeos/login/managed/managed_user_creation_browsertest.cc b/chrome/browser/chromeos/login/managed/managed_user_creation_browsertest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..e759620172d58296ffdac20295656f7a8b4e2f5c
--- /dev/null
+++ b/chrome/browser/chromeos/login/managed/managed_user_creation_browsertest.cc
@@ -0,0 +1,175 @@
+// 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 SupervisedUserCreationTest : public ManagedUserTestBase {
+ public:
+ SupervisedUserCreationTest() : ManagedUserTestBase() {}
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(SupervisedUserCreationTest);
+};
+
+class SupervisedUserTransactionCleanupTest : public ManagedUserTestBase {
+ public:
+ SupervisedUserTransactionCleanupTest() : ManagedUserTestBase() {}
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(SupervisedUserTransactionCleanupTest);
+};
+
+class SupervisedUserOwnerCreationTest : public ManagedUserTestBase {
+ public:
+ SupervisedUserOwnerCreationTest() : ManagedUserTestBase() {}
+
+ virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+ ManagedUserTestBase::SetUpInProcessBrowserTestFixture();
+ cros_settings_provider_.reset(new StubCrosSettingsProvider());
+ cros_settings_provider_->Set(kDeviceOwner, base::StringValue(kTestManager));
+ }
+
+ private:
+ scoped_ptr<StubCrosSettingsProvider> cros_settings_provider_;
+ DISALLOW_COPY_AND_ASSIGN(SupervisedUserOwnerCreationTest);
+};
+
+class SupervisedUserTransactionCleanupTest2
+ : public SupervisedUserTransactionCleanupTest {
+ public:
+ SupervisedUserTransactionCleanupTest2()
+ : SupervisedUserTransactionCleanupTest() {}
+
+ virtual void SetUpInProcessBrowserTestFixture() OVERRIDE {
+ SupervisedUserTransactionCleanupTest::SetUpInProcessBrowserTestFixture();
+ EXPECT_CALL(*mock_async_method_caller_, AsyncRemove(_, _)).Times(1);
+ }
+
+ private:
+ DISALLOW_COPY_AND_ASSIGN(SupervisedUserTransactionCleanupTest2);
+};
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserCreationTest,
+ PRE_PRE_PRE_CreateAndRemoveSupervisedUser) {
+ PrepareUsers();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserCreationTest,
+ PRE_PRE_CreateAndRemoveSupervisedUser) {
+ StartFlowLoginAsManager();
+ FillNewUserData(kTestSupervisedUserDisplayName);
+ StartUserCreation("managed-user-creation-next-button",
+ kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserCreationTest,
+ PRE_CreateAndRemoveSupervisedUser) {
+ SigninAsSupervisedUser(true, 0, kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserCreationTest,
+ CreateAndRemoveSupervisedUser) {
+ RemoveSupervisedUser(3, 0, kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserOwnerCreationTest,
+ PRE_PRE_PRE_CreateAndRemoveSupervisedUser) {
+ PrepareUsers();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserOwnerCreationTest,
+ PRE_PRE_CreateAndRemoveSupervisedUser) {
+ StartFlowLoginAsManager();
+ FillNewUserData(kTestSupervisedUserDisplayName);
+ StartUserCreation("managed-user-creation-next-button",
+ kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserOwnerCreationTest,
+ PRE_CreateAndRemoveSupervisedUser) {
+ SigninAsSupervisedUser(true, 0, kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserOwnerCreationTest,
+ CreateAndRemoveSupervisedUser) {
+ RemoveSupervisedUser(3, 0, kTestSupervisedUserDisplayName);
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserTransactionCleanupTest,
+ PRE_PRE_CreateAndCancelSupervisedUser) {
+ PrepareUsers();
+}
+
+IN_PROC_BROWSER_TEST_F(SupervisedUserTransactionCleanupTest,
+ PRE_CreateAndCancelSupervisedUser) {
+ StartFlowLoginAsManager();
+ FillNewUserData(kTestSupervisedUserDisplayName);
+
+ EXPECT_CALL(*mock_homedir_methods_, MountEx(_, _, _, _)).Times(1);
+ EXPECT_CALL(*mock_homedir_methods_, AddKeyEx(_, _, _, _, _)).Times(1);
+
+ JSEval("$('managed-user-creation-next-button').click()");
+
+ ::testing::Mock::VerifyAndClearExpectations(mock_homedir_methods_);
+
+ EXPECT_TRUE(registration_utility_stub_->register_was_called());
+ EXPECT_EQ(registration_utility_stub_->display_name(),
+ base::UTF8ToUTF16(kTestSupervisedUserDisplayName));
+
+ std::string user_id = registration_utility_stub_->managed_user_id();
+ // Make sure user is already in list.
Nikita (slow) 2014/04/23 09:50:17 nit: Insert empty line before comment.
Denis Kuznetsov (DE-MUC) 2014/04/24 17:44:09 Done.
+ ASSERT_EQ(3UL, UserManager::Get()->GetUsers().size());
+ // We wait for token now. Press cancel button at this point.
Nikita (slow) 2014/04/23 09:50:17 nit: Insert empty line.
Denis Kuznetsov (DE-MUC) 2014/04/24 17:44:09 Done.
+ JSEval("$('cancel-add-user-button').click()");
+}
+
+IN_PROC_BROWSER_TEST_(
Nikita (slow) 2014/04/23 09:50:17 What this macros does?
Denis Kuznetsov (DE-MUC) 2014/04/24 17:44:09 this is expansion of IN_PROC_BROWSER_TEST_F. It l
Denis Kuznetsov (DE-MUC) 2014/04/24 17:44:09 It is the expansion of IN_PROC_BROWSER_TEST_F. Thi
+ SupervisedUserTransactionCleanupTest,
+ CreateAndCancelSupervisedUser,
+ SupervisedUserTransactionCleanupTest2,
+ ::testing::internal::GetTypeId<SupervisedUserTransactionCleanupTest>()) {
+ // Make sure there is no supervised user in list.
+ ASSERT_EQ(2UL, UserManager::Get()->GetUsers().size());
+}
+
+} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698