Changes to the sign in manager in preparation for one-click sign in. This CL

chrome/browser/signin/signin_manager_unittest.cc

 // Copyright (c) 2012 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/signin/signin_manager.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/compiler_specific.h"
+#include "base/stringprintf.h"
 #include "chrome/browser/password_manager/encryptor.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/signin/token_service.h"
 #include "chrome/browser/signin/token_service_unittest.h"
 #include "chrome/browser/webdata/web_data_service.h"
 #include "chrome/common/chrome_notification_types.h"
+#include "chrome/common/net/gaia/gaia_constants.h"
 #include "chrome/common/net/gaia/gaia_urls.h"
 #include "chrome/common/pref_names.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/test/test_url_fetcher_factory.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_status.h"
 
 #include "testing/gmock/include/gmock/gmock.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
+namespace {
+
+const char kGetTokenPairValidResponse[] =
+    "{"
+    "  \"refresh_token\": \"rt1\","
+    "  \"access_token\": \"at1\","
+    "  \"expires_in\": 3600,"
+    "  \"token_type\": \"Bearer\""
+    "}";
+
+const char kUberAuthTokenURLFormat[] =
+    "%s?source=%s&issueuberauth=1";
+
+}  // namespace
+
+
 class SigninManagerTest : public TokenServiceTestHarness {
  public:
   virtual void SetUp() OVERRIDE {
     TokenServiceTestHarness::SetUp();
     manager_.reset(new SigninManager());
     google_login_success_.ListenFor(
         chrome::NOTIFICATION_GOOGLE_SIGNIN_SUCCESSFUL,
         content::Source<Profile>(profile_.get()));
     google_login_failure_.ListenFor(chrome::NOTIFICATION_GOOGLE_SIGNIN_FAILED,
                                     content::Source<Profile>(profile_.get()));
   }
 
+  void SimulateValidResponseSignInWithCredentials() {
+    // Simulate the correct StartOAuthLoginTokenFetch response.  This involves
+    // two separate fetches.
+    TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
+    DCHECK(fetcher);
+    DCHECK(fetcher->delegate());
+
+    fetcher->set_url(
+      GURL(GaiaUrls::GetInstance()->client_login_to_oauth2_url()));
+    fetcher->set_status(net::URLRequestStatus());
+    fetcher->set_response_code(200);
+    fetcher->SetResponseString(kGetTokenPairValidResponse);
+    fetcher->delegate()->OnURLFetchComplete(fetcher);
+
+    fetcher = factory_.GetFetcherByID(0);
+    DCHECK(fetcher);
+    DCHECK(fetcher->delegate());
+
+    fetcher->set_url(
+        GURL(GaiaUrls::GetInstance()->oauth2_token_url()));
+    fetcher->set_status(net::URLRequestStatus());
+    fetcher->set_response_code(200);
+    fetcher->SetResponseString(kGetTokenPairValidResponse);
+    fetcher->delegate()->OnURLFetchComplete(fetcher);
+
+    // Simulate the correct StartUberAuthTokenFetch response.
+    GURL url(base::StringPrintf(
+        kUberAuthTokenURLFormat,
+        GaiaUrls::GetInstance()->oauth1_login_url().c_str(),
+        GaiaConstants::kChromeSource));
+    fetcher = factory_.GetFetcherByID(0);
+    DCHECK(fetcher);
+    DCHECK(fetcher->delegate());
+    fetcher->set_url(url);
+    fetcher->set_status(net::URLRequestStatus());
+    fetcher->set_response_code(200);
+    fetcher->SetResponseString("token");
+    fetcher->delegate()->OnURLFetchComplete(fetcher);
+
+    // Simulate the correct StartTokenAuth response.
+    fetcher = factory_.GetFetcherByID(0);
+    DCHECK(fetcher);
+    DCHECK(fetcher->delegate());
+
+    fetcher->set_url(
+        GURL(GaiaUrls::GetInstance()->token_auth_url()));
+    fetcher->set_status(net::URLRequestStatus());
+    fetcher->set_response_code(200);
+    net::ResponseCookies cookies;
+    cookies.push_back("SID=sid");
+    cookies.push_back("LSID=lsid");
+    fetcher->set_cookies(cookies);
+    fetcher->SetResponseString("data");
+    fetcher->delegate()->OnURLFetchComplete(fetcher);
+
+    // Simulate the correct GetUserInfo response.
+    fetcher = factory_.GetFetcherByID(0);
+    DCHECK(fetcher);
+    DCHECK(fetcher->delegate());
+    fetcher->set_url(GURL(GaiaUrls::GetInstance()->get_user_info_url()));
+    fetcher->set_status(net::URLRequestStatus());
+    fetcher->set_response_code(200);
+    fetcher->SetResponseString("email=user@gmail.com\nallServices=");
+    fetcher->delegate()->OnURLFetchComplete(fetcher);
+  }
+
   void SimulateValidResponseClientLogin(bool isGPlusUser) {
     // Simulate the correct ClientLogin response.
     std::string respons_string = isGPlusUser ?
         "email=user@gmail.com\nallServices=googleme" :
         "email=user@gmail.com\nallServices=";
     TestURLFetcher* fetcher = factory_.GetFetcherByID(0);
     DCHECK(fetcher);
     DCHECK(fetcher->delegate());
 
     fetcher->set_url(GURL(GaiaUrls::GetInstance()->client_login_url()));
@@ skipping 36 matching lines @@
   // Should go into token service and stop.
   EXPECT_EQ(1U, google_login_success_.size());
   EXPECT_EQ(0U, google_login_failure_.size());
 
   // Should persist across resets.
   manager_.reset(new SigninManager());
   manager_->Initialize(profile_.get());
   EXPECT_EQ("user@gmail.com", manager_->GetAuthenticatedUsername());
 }
 
+TEST_F(SigninManagerTest, SignInWithCredentials) {
+  manager_->Initialize(profile_.get());
+  EXPECT_TRUE(manager_->GetAuthenticatedUsername().empty());
+
+  manager_->StartSignInWithCredentials("username", "password");
+  EXPECT_TRUE(manager_->GetAuthenticatedUsername().empty());
+
+  SimulateValidResponseSignInWithCredentials();
+  EXPECT_FALSE(manager_->GetAuthenticatedUsername().empty());
+
+  // Should go into token service and stop.
+  EXPECT_EQ(1U, google_login_success_.size());

[Andrew T Wilson (Slow), 2012/02/27 19:00:10]

Is it difficult to validate that the correct password is passed with the SUCCESS
notification?

[Roger Tawa OOO till Jul 10th, 2012/02/29 21:02:54]

The password is cleared, so not sure what you mean.

[Andrew T Wilson (Slow), 2012/03/02 19:22:33]

I meant that all we're doing is detecting that LOGIN_SUCCESS notifications are
sent out, without checking that the right values are passed with it (like the
password). It'd be good to test that as well, if it's not hard.

[Roger Tawa OOO till Jul 10th, 2012/03/02 20:10:58]

Its not easy, probably why it was not done for the existing SignInClientLogin
test.  The notification sent out contains a pointer to a details object which is
on the stack in OnGetUserInfoSuccess(), so we can't check it here.  To check
this we could create specific tracker instead of using the generic
TestNotificationTracker, but I'd prefer to do that in a separate CL.

+  EXPECT_EQ(0U, google_login_failure_.size());
+
+  // Should persist across resets.
+  manager_.reset(new SigninManager());
+  manager_->Initialize(profile_.get());
+  EXPECT_EQ("user@gmail.com", manager_->GetAuthenticatedUsername());

[Andrew T Wilson (Slow), 2012/02/27 19:00:10]

Why does the authenticated username not match the username supplied to
StartSignInWithCredentials()?

[Roger Tawa OOO till Jul 10th, 2012/02/29 21:02:54]

The test passed because SimulateValidResponseSignInWithCredentials() used
"user@gmail.com".  I changed the code for this test t be consistent and use
"user@gmail.com" everywhere.

+}
+
 TEST_F(SigninManagerTest, SignInClientLoginNoGPlus) {
   manager_->Initialize(profile_.get());
   EXPECT_TRUE(manager_->GetAuthenticatedUsername().empty());
 
   manager_->StartSignIn("username", "password", "", "");
   EXPECT_TRUE(manager_->GetAuthenticatedUsername().empty());
 
   SimulateValidResponseClientLogin(false);
   EXPECT_FALSE(manager_->GetAuthenticatedUsername().empty());
   EXPECT_FALSE(profile_->GetPrefs()->GetBoolean(prefs::kIsGooglePlusUser));
@@ skipping 115 matching lines @@
 
 TEST_F(SigninManagerTest, SignOutMidConnect) {
   manager_->Initialize(profile_.get());
   manager_->StartSignIn("username", "password", "", "");
   manager_->SignOut();
   EXPECT_EQ(0U, google_login_success_.size());
   EXPECT_EQ(0U, google_login_failure_.size());
 
   EXPECT_TRUE(manager_->GetAuthenticatedUsername().empty());
 }