Fixed suggesting adding more users into multi-profile session when we don't have more

chrome/browser/ui/ash/session_state_delegate_chromeos_unittest.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 "chrome/browser/ui/ash/session_state_delegate_chromeos.h"
 
 #include <string>
 #include <vector>
 
 #include "base/run_loop.h"
@@ skipping 211 matching lines @@
       "subject", "issuer", base::Time(), base::Time()));
   service->OnTrustAnchorsChanged(certificates);
   EXPECT_TRUE(service->has_policy_certificates());
   EXPECT_FALSE(
       session_state_delegate()->IsMultiProfileAllowedByPrimaryUserPolicy());
 
   // Flush tasks posted to IO.
   base::RunLoop().RunUntilIdle();
 }
 
+// Make sure adding users to multiprofiles disabled by reaching maximum
+// number of users in sessions.
+TEST_F(SessionStateDelegateChromeOSTest,
+       AddUserToMultiprofileDisallowedByMaximumUsers) {
+  InitForMultiProfile();
+  ash::SessionStateDelegate::MultiProfileAddUserError add_user_error;
+
+  EXPECT_TRUE(
+      session_state_delegate()->CanAddUserToMultiProfile(&add_user_error));
+  const std::string user_email(kUser);
+  user_manager()->LoginUser(user_email);
+  while (session_state_delegate()->NumberOfLoggedInUsers() <
+         session_state_delegate()->GetMaximumNumberOfLoggedInUsers()) {
+    UserAddedToSession("bb@b.b");
+  }
+  EXPECT_FALSE(
+      session_state_delegate()->CanAddUserToMultiProfile(&add_user_error));
+  EXPECT_EQ(ash::SessionStateDelegate::
+                MULTIPROFILE_ADD_USER_ERROR_MAXIMUM_OF_USERS_REACHED,
+            add_user_error);
+}
+
+// Make sure adding users to multiprofiles disabled by logging all possible

[James Cook, 2014/09/23 16:02:12]

nit: "logging" -> "logging in"

[Roman Sorokin (ftl), 2014/09/24 08:30:47]

Done.

+// users.
+TEST_F(SessionStateDelegateChromeOSTest,
+       AddUserToMultiprofileDisallowedByAllUsersLogged) {
+  InitForMultiProfile();
+  ash::SessionStateDelegate::MultiProfileAddUserError add_user_error;
+
+  EXPECT_TRUE(
+      session_state_delegate()->CanAddUserToMultiProfile(&add_user_error));
+  const std::string user_email(kUser);
+  user_manager()->LoginUser(user_email);
+  UserAddedToSession("bb@b.b");
+  EXPECT_FALSE(
+      session_state_delegate()->CanAddUserToMultiProfile(&add_user_error));
+  EXPECT_EQ(ash::SessionStateDelegate::MULTIPROFILE_ADD_USER_ERROR_OUT_OF_USERS,
+            add_user_error);
+}
+
+// Make sure adding users to multiprofiles disabled by primary user policy.
+TEST_F(SessionStateDelegateChromeOSTest,
+       AddUserToMultiprofileDisallowedByPrimaryUserPolicy) {
+  InitForMultiProfile();
+  ash::SessionStateDelegate::MultiProfileAddUserError add_user_error;
+
+  EXPECT_TRUE(
+      session_state_delegate()->CanAddUserToMultiProfile(&add_user_error));
+  const std::string user_email(kUser);
+  user_manager()->LoginUser(user_email);
+  user_profile_->GetPrefs()->SetString(
+      prefs::kMultiProfileUserBehavior,
+      chromeos::MultiProfileUserController::kBehaviorNotAllowed);
+  user_manager()->AddUser("bb@b.b");
+  EXPECT_FALSE(
+      session_state_delegate()->CanAddUserToMultiProfile(&add_user_error));
+  EXPECT_EQ(ash::SessionStateDelegate::
+                MULTIPROFILE_ADD_USER_ERROR_NOT_ALLOWED_PRIMARY_USER,
+            add_user_error);
+}

[James Cook, 2014/09/23 16:02:12]

Nice test suite!

[Roman Sorokin (ftl), 2014/09/24 08:30:47]

Acknowledged.

+
 }  // namespace chromeos