Blimp OAuth2 token retreival on application start up.

blimp/client/core/session/identity_source_unittest.cc

Index: blimp/client/core/session/identity_source_unittest.cc
diff --git a/blimp/client/core/session/identity_source_unittest.cc b/blimp/client/core/session/identity_source_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..fb5e4dcadab7201a107bb1130f605b8ab17dec6a
--- /dev/null
+++ b/blimp/client/core/session/identity_source_unittest.cc
@@ -0,0 +1,145 @@
+// Copyright 2016 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 "blimp/client/core/session/identity_source.h"
+
+#include "base/macros.h"
+#include "base/memory/ptr_util.h"
+#include "blimp/client/test/test_blimp_client_context_delegate.h"
+#include "google_apis/gaia/fake_identity_provider.h"
+#include "google_apis/gaia/fake_oauth2_token_service.h"
+#include "google_apis/gaia/google_service_auth_error.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace blimp {
+namespace client {
+namespace {
+
+class MockIdentitySource : public IdentitySource {
+ public:
+  explicit MockIdentitySource(
+      std::unique_ptr<IdentityProvider> identity_provider)
+      : IdentitySource(std::move(identity_provider)),
+        success_(0),
+        fail_(0),
+        refresh_(0) {}
+  ~MockIdentitySource() override{};
+
+  void OnGetTokenSuccess(const OAuth2TokenService::Request* request,
+                         const std::string& access_token,
+                         const base::Time& expiration_time) override {
+    IdentitySource::OnGetTokenSuccess(request, access_token, expiration_time);
+    success_++;
+    token_ = access_token;
+  }
+
+  void OnGetTokenFailure(const OAuth2TokenService::Request* request,
+                         const GoogleServiceAuthError& error) override {
+    IdentitySource::OnGetTokenFailure(request, error);
+    fail_++;
+    token_.clear();
+  }
+
+  void OnRefreshTokenAvailable(const std::string& account_id) override {
+    IdentitySource::OnRefreshTokenAvailable(account_id);
+    refresh_++;
+  }
+
+  int Succeeded() { return success_; }
+  int Failed() { return fail_; }
+  int Refreshed() { return refresh_; }
+  const std::string& Token() { return token_; }
+
+ private:
+  std::string token_;
+  int success_;
+  int fail_;
+  int refresh_;
+
+  DISALLOW_COPY_AND_ASSIGN(MockIdentitySource);
+};
+
+class IdentitySourceTest : public testing::Test {
+ public:
+  IdentitySourceTest() = default;
+  ~IdentitySourceTest() override = default;
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(IdentitySourceTest);
+};
+
+TEST_F(IdentitySourceTest, TestConnect) {
+  std::unique_ptr<FakeOAuth2TokenService> token_service =
+      base::MakeUnique<FakeOAuth2TokenService>();
+  FakeIdentityProvider* id_provider =
+      new FakeIdentityProvider(token_service.get());
+  std::unique_ptr<IdentityProvider> id_ptr =
+      base::WrapUnique<IdentityProvider>((IdentityProvider*)id_provider);
+
+  MockIdentitySource auth(std::move(id_ptr));
+  TestBlimpClientContextDelegate mock_blimp_delegate;
+  auth.SetDelegate(&mock_blimp_delegate);
+
+  // Connect when user is not signed in. Nothing happens.
+  IdentitySource::TokenCallback token_callback;

[nyquist, 2016/08/16 18:38:06]

Should this test ensure that the callback is invoked on success, and not invoked
on failure?

[xingliu, 2016/08/17 00:32:24]

Fixed, also move a couple of fake objects into MockBlimpClientContextDelegate.

Also it tests duplicate connect calls, to ensure all the callbacks working
correctly.

+  id_provider->LogOut();
+  auth.Connect(token_callback);
+  DCHECK_EQ(auth.Succeeded(), 0);
+  DCHECK_EQ(auth.Failed(), 0);
+  DCHECK_EQ(auth.Refreshed(), 0);
+  DCHECK_EQ(auth.Token(), std::string());
+
+  FakeOAuth2TokenServiceDelegate* mock_token_service_delegate =
+      token_service->GetFakeOAuth2TokenServiceDelegate();
+
+  // Connect when user signed in, but no refresh token, refresh token observer
+  // should be added.
+  std::string account = "mock_account";
+  id_provider->LogIn(account);
+  mock_token_service_delegate->RevokeCredentials(account);
+  auth.Connect(token_callback);
+  DCHECK_EQ(auth.Succeeded(), 0);
+  DCHECK_EQ(auth.Failed(), 0);
+  DCHECK_EQ(auth.Refreshed(), 0);
+
+  // Issue refresh token, listener should be triggered, and request should be
+  // sent.
+  mock_token_service_delegate->UpdateCredentials(account, "mock_refresh_token");
+  DCHECK_EQ(auth.Succeeded(), 0);
+  DCHECK_EQ(auth.Failed(), 0);
+  DCHECK_EQ(auth.Refreshed(), 1);
+
+  // Fire access token success, first request should be fulfilled.
+  base::Time time;
+  std::string mock_access_token = "mock_access_token";
+  token_service->IssueAllTokensForAccount(account, mock_access_token, time);
+  DCHECK_EQ(auth.Succeeded(), 1);
+  DCHECK_EQ(auth.Failed(), 0);
+  DCHECK_EQ(auth.Token(), mock_access_token);
+
+  // Connect again and fire access token failed.
+  GoogleServiceAuthError error(GoogleServiceAuthError::State::REQUEST_CANCELED);
+  auth.Connect(token_callback);
+  token_service->IssueErrorForAllPendingRequestsForAccount(account, error);
+  DCHECK_EQ(auth.Succeeded(), 1);
+  DCHECK_EQ(auth.Failed(), 1);
+  DCHECK_EQ(auth.Token(), std::string());
+
+  // Refresh token listener should have been removed.
+  mock_token_service_delegate->UpdateCredentials(account, "mock_refresh_token");
+  DCHECK_EQ(auth.Refreshed(), 1);
+
+  // Direct connect with refresh token_callbacktoken, and no listener should be
+  // added.
+  auth.Connect(token_callback);
+  token_service->IssueAllTokensForAccount(account, mock_access_token, time);
+  DCHECK_EQ(auth.Succeeded(), 2);
+  DCHECK_EQ(auth.Token(), mock_access_token);
+  mock_token_service_delegate->UpdateCredentials(account, "mock_refresh_token");
+  DCHECK_EQ(auth.Refreshed(), 1);
+}
+
+}  // namespace
+}  // namespace client
+}  // namespace blimp