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

Unified Diff: chrome/browser/extensions/token_cache/token_cache_unittest.cc

Issue 12380006: Build a new TokenCacheService so I can stop using TokenService for something it wasn't designed for. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Token Cache Service - more CR fixes per DCheng Created 7 years, 10 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/extensions/token_cache/token_cache_unittest.cc
diff --git a/chrome/browser/extensions/token_cache/token_cache_unittest.cc b/chrome/browser/extensions/token_cache/token_cache_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..1395a13082da0f01635f340935dc8ed9151be0a9
--- /dev/null
+++ b/chrome/browser/extensions/token_cache/token_cache_unittest.cc
@@ -0,0 +1,119 @@
+// Copyright (c) 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 "base/basictypes.h"
+#include "base/compiler_specific.h"
+#include "base/time.h"
+#include "chrome/browser/extensions/token_cache/token_cache.h"
+#include "chrome/common/chrome_notification_types.h"
+#include "content/public/browser/notification_details.h"
+#include "content/public/browser/notification_source.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+using base::Time;
+using base::TimeDelta;
+
+namespace extensions {
+
+class TokenCacheTest : public testing::Test {
+ public:
+ TokenCacheTest() : cache_(NULL) {}
+ virtual ~TokenCacheTest() {}
+
+ // Methods from testing::Test.
+ virtual void SetUp() OVERRIDE {}
+ virtual void TearDown() OVERRIDE {
+ cache_.token_cache_.clear();
dcheng 2013/03/06 00:26:35 Do we really need to do this? If not, just remove
Pete Williamson 2013/03/06 00:50:06 Done.
+ }
+
+ size_t CacheSize() {
+ return cache_.token_cache_.size();
+ }
+
+ bool HasMatch(const std::string& token_name) {
+ return cache_.FindMatch(token_name) != cache_.token_cache_.end();
+ }
+
+ void InsertExpiredToken(const std::string& token_name,
+ const std::string& token_value) {
+ EXPECT_TRUE(!HasMatch(token_name));
+
+ // Compute a time value for yesterday
+ Time now = Time::Now();
+ TimeDelta one_day = one_day.FromDays(1);
+ Time yesterday = now - one_day;
+
+ TokenCacheData token_data;
+ token_data.token = token_value;
+ token_data.expiration_time = yesterday;
+
+ cache_.token_cache_[token_name] = token_data;
+ }
+
+ protected:
+ TokenCacheService cache_;
+};
+
+TEST_F(TokenCacheTest, SaveTokenTest) {
+ TimeDelta zero;
+ cache_.StoreToken("foo", "bar", zero);
+
+ EXPECT_EQ(CacheSize(), (size_t) 1);
dcheng 2013/03/06 00:26:35 1U.
Pete Williamson 2013/03/06 00:50:06 Done.
+ EXPECT_TRUE(HasMatch("foo"));
+}
+
+TEST_F(TokenCacheTest, RetrieveTokenTest) {
+ TimeDelta zero;
+ cache_.StoreToken("Mozart", "Eine Kleine Nacht Musik", zero);
+ cache_.StoreToken("Bach", "Brandenburg Concerto #3", zero);
+ cache_.StoreToken("Beethoven", "Emperor Piano Concerto #5", zero);
+ cache_.StoreToken("Handel", "Water Music", zero);
+ cache_.StoreToken("Chopin", "Heroic", zero);
+
+ std::string found_token = cache_.RetrieveToken("Chopin");
+ EXPECT_EQ("Heroic", found_token);
+}
+
+TEST_F(TokenCacheTest, ReplaceTokenTest) {
+ TimeDelta zero;
+ cache_.StoreToken("Chopin", "Heroic", zero);
+
+ std::string found_token = cache_.RetrieveToken("Chopin");
+ EXPECT_EQ("Heroic", found_token);
+
+ cache_.StoreToken("Chopin", "Military", zero);
+
+ found_token = cache_.RetrieveToken("Chopin");
+ EXPECT_EQ("Military", found_token);
+ EXPECT_EQ(CacheSize(), 1U);
+}
+
+TEST_F(TokenCacheTest, SignoutTest) {
+ TimeDelta zero;
+ cache_.StoreToken("foo", "bar", zero);
+ content::Source<TokenCacheService> stub_source(&cache_);
+ content::NotificationDetails stub_details;
+
+ EXPECT_EQ(CacheSize(), 1U);
+ EXPECT_TRUE(HasMatch("foo"));
+
+ cache_.Observe(chrome::NOTIFICATION_GOOGLE_SIGNED_OUT,
+ stub_source, stub_details);
+
+ EXPECT_EQ(CacheSize(), 0U);
+ EXPECT_FALSE(HasMatch("foo"));
+}
+
+TEST_F(TokenCacheTest, TokenExpireTest) {
+ // Use the fact that we are friends to insert an expired token.
+ InsertExpiredToken("foo", "bar");
+
+ EXPECT_EQ(CacheSize(), 1U);
+
+ // If we attempt to find the token, the attempt should fail.
+ EXPECT_FALSE(HasMatch("foo"));
+ EXPECT_EQ(CacheSize(), 0U);
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698