Tame the proliferation of UserContext constructors

chrome/browser/chromeos/login/auth/online_attempt_unittest.cc

 // 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/bind.h"
 #include "base/memory/ref_counted.h"
 #include "base/run_loop.h"
 #include "chrome/browser/chromeos/login/auth/auth_attempt_state.h"
@@ skipping 15 matching lines @@
 using ::testing::Invoke;
 using ::testing::Return;
 using ::testing::_;
 using content::BrowserThread;
 
 namespace chromeos {
 
 class OnlineAttemptTest : public testing::Test {
  public:
   OnlineAttemptTest()
-      : state_(UserContext(), "", "", User::USER_TYPE_REGULAR, false),
+      : state_(UserContext(), false),
         attempt_(new OnlineAttempt(&state_, &resolver_)) {
   }
 
   void RunFailureTest(const GoogleServiceAuthError& error) {
     EXPECT_CALL(resolver_, Resolve())
         .Times(1)
         .RetiresOnSaturation();
 
     BrowserThread::PostTask(
         BrowserThread::UI, FROM_HERE,
@@ skipping 87 matching lines @@
   TestingProfile profile;
 
   base::RunLoop run_loop;
   EXPECT_CALL(resolver_, Resolve())
       .WillOnce(Invoke(&run_loop, &base::RunLoop::Quit))
       .RetiresOnSaturation();
 
   // This is how we inject fake URLFetcher objects, with a factory.
   MockURLFetcherFactory<HostedFetcher> factory;
 
-  TestAttemptState local_state(UserContext(), "", "",
-                               User::USER_TYPE_REGULAR, true);
+  TestAttemptState local_state(UserContext(), true);
   attempt_.reset(new OnlineAttempt(&local_state, &resolver_));
   attempt_->Initiate(&profile);
 
   run_loop.Run();
 
   EXPECT_EQ(error, local_state.online_outcome());
   EXPECT_EQ(LoginFailure::NETWORK_AUTH_FAILED,
             local_state.online_outcome().reason());
 }
 
 TEST_F(OnlineAttemptTest, FullLogin) {
   TestingProfile profile;
 
   base::RunLoop run_loop;
   EXPECT_CALL(resolver_, Resolve())
       .WillOnce(Invoke(&run_loop, &base::RunLoop::Quit))
       .RetiresOnSaturation();
 
   // This is how we inject fake URLFetcher objects, with a factory.
   MockURLFetcherFactory<SuccessFetcher> factory;
 
-  TestAttemptState local_state(UserContext(), "", "",
-                               User::USER_TYPE_REGULAR, true);
+  TestAttemptState local_state(UserContext(), true);
   attempt_.reset(new OnlineAttempt(&local_state, &resolver_));
   attempt_->Initiate(&profile);
 
   run_loop.Run();
 
   EXPECT_EQ(LoginFailure::LoginFailureNone(), local_state.online_outcome());
 }
 
 TEST_F(OnlineAttemptTest, LoginNetFailure) {
   RunFailureTest(
@@ skipping 40 matching lines @@
                  attempt_->weak_factory_.GetWeakPtr(),
                  error));
 
   // Force UI thread to finish tasks so I can verify |state_|.
   base::RunLoop().RunUntilIdle();
   EXPECT_TRUE(GoogleServiceAuthError::AuthErrorNone() ==
               state_.online_outcome().error());
 }
 
 }  // namespace chromeos