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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: chrome/browser/extensions/api/feedback_private/log_source_access_manager_unittest.cc

Issue 2840103002: Add new API function: feedbackPrivate.readLogSource (Closed)
Patch Set: Add comments to SingleLogSourceFactory; Add new histogram enum Created 3 years, 7 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
« chrome/browser/extensions/api/feedback_private/log_source_access_manager.h ('K') | « chrome/browser/extensions/api/feedback_private/log_source_access_manager.cc ('k') | chrome/browser/extensions/api/feedback_private/log_source_resource.h » ('j') | chrome/browser/extensions/api/feedback_private/single_log_source_factory.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/extensions/api/feedback_private/log_source_access_manager_unittest.cc
diff --git a/chrome/browser/extensions/api/feedback_private/log_source_access_manager_unittest.cc b/chrome/browser/extensions/api/feedback_private/log_source_access_manager_unittest.cc
new file mode 100644
index 0000000000000000000000000000000000000000..a249bc7fdd1c528bd39103f7732a07c455bd36ef
--- /dev/null
+++ b/chrome/browser/extensions/api/feedback_private/log_source_access_manager_unittest.cc
@@ -0,0 +1,230 @@
+// Copyright 2017 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/extensions/api/feedback_private/log_source_access_manager.h"
+
+#include <string>
+#include <utility>
+
+#include "base/macros.h"
+#include "testing/gtest/include/gtest/gtest.h"
+
+namespace extensions {
+
+namespace {
+
+using extensions::api::feedback_private::LOG_SOURCE_MESSAGES;
+using extensions::api::feedback_private::LOG_SOURCE_UI_LATEST;
+using SourceAndExtension = LogSourceAccessManager::SourceAndExtension;
+
+} // namespace
+
+TEST(LogSourceAccessManagerTest, Empty) {
+ LogSourceAccessManager manager;
+
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_TRUE(
+ manager
+ .GetLastExtensionAccessTime(std::make_pair(LOG_SOURCE_UI_LATEST, ""))
+ .is_null());
+}
+
+TEST(LogSourceAccessManagerTest, OneSourceOneExtension) {
+ LogSourceAccessManager manager;
+
+ const SourceAndExtension key = std::make_pair(LOG_SOURCE_MESSAGES, "aaaa");
+
+ EXPECT_TRUE(manager.AddExtension(key));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.AddExtension(key));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ EXPECT_TRUE(manager.RemoveExtension(key));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.RemoveExtension(key));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+}
+
+TEST(LogSourceAccessManagerTest, OneSourceMultipleExtensions) {
+ LogSourceAccessManager manager;
+
+ const SourceAndExtension key_a = std::make_pair(LOG_SOURCE_MESSAGES, "aaaa");
+ const SourceAndExtension key_b = std::make_pair(LOG_SOURCE_MESSAGES, "bbbb");
+ const SourceAndExtension key_c = std::make_pair(LOG_SOURCE_MESSAGES, "cccc");
+ const SourceAndExtension key_d = std::make_pair(LOG_SOURCE_MESSAGES, "dddd");
+
+ // Add extensions "aaaa", "bbbb", and "cccc".
+ EXPECT_TRUE(manager.AddExtension(key_a));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.AddExtension(key_a));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ EXPECT_TRUE(manager.AddExtension(key_b));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.AddExtension(key_b));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ EXPECT_TRUE(manager.AddExtension(key_c));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.AddExtension(key_c));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ // Remove extensions "bbbb" and "aaaa".
+ EXPECT_TRUE(manager.RemoveExtension(key_b));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.RemoveExtension(key_b));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ EXPECT_TRUE(manager.RemoveExtension(key_a));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.RemoveExtension(key_a));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ // Add extension "dddd".
+ EXPECT_TRUE(manager.AddExtension(key_d));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.AddExtension(key_d));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ // Remove extensions "cccc" and "dddd".
+ EXPECT_TRUE(manager.RemoveExtension(key_c));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.RemoveExtension(key_c));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+
+ EXPECT_TRUE(manager.RemoveExtension(key_d));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_FALSE(manager.RemoveExtension(key_d));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+}
+
+TEST(LogSourceAccessManagerTest, MultipleSourcesMultipleExtensions) {
+ LogSourceAccessManager manager;
+
+ const SourceAndExtension key_a0 = std::make_pair(LOG_SOURCE_MESSAGES, "aaaa");
+ const SourceAndExtension key_a1 =
+ std::make_pair(LOG_SOURCE_UI_LATEST, "aaaa");
+ const SourceAndExtension key_b0 = std::make_pair(LOG_SOURCE_MESSAGES, "bbbb");
+ const SourceAndExtension key_b1 =
+ std::make_pair(LOG_SOURCE_UI_LATEST, "bbbb");
+ const SourceAndExtension key_c0 = std::make_pair(LOG_SOURCE_MESSAGES, "cccc");
+ const SourceAndExtension key_c1 =
+ std::make_pair(LOG_SOURCE_UI_LATEST, "cccc");
+
+ // Add the above entries.
+ EXPECT_TRUE(manager.AddExtension(key_a0));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_TRUE(manager.AddExtension(key_a1));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.AddExtension(key_a0));
+ EXPECT_FALSE(manager.AddExtension(key_a1));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ EXPECT_TRUE(manager.AddExtension(key_b0));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_TRUE(manager.AddExtension(key_b1));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.AddExtension(key_b0));
+ EXPECT_FALSE(manager.AddExtension(key_b1));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ EXPECT_TRUE(manager.AddExtension(key_c1));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.AddExtension(key_c1));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_TRUE(manager.AddExtension(key_c0));
+ EXPECT_FALSE(manager.AddExtension(key_c0));
+ EXPECT_FALSE(manager.AddExtension(key_c1));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ // Remove all the entries.
+ EXPECT_TRUE(manager.RemoveExtension(key_a0));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.RemoveExtension(key_a0));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ EXPECT_TRUE(manager.RemoveExtension(key_b0));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.RemoveExtension(key_b0));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ EXPECT_TRUE(manager.RemoveExtension(key_c0));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.RemoveExtension(key_c0));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(3U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ EXPECT_TRUE(manager.RemoveExtension(key_c1));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.RemoveExtension(key_c1));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(2U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ EXPECT_TRUE(manager.RemoveExtension(key_b1));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.RemoveExtension(key_b1));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(1U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+
+ EXPECT_TRUE(manager.RemoveExtension(key_a1));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+ EXPECT_FALSE(manager.RemoveExtension(key_a1));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_MESSAGES));
+ EXPECT_EQ(0U, manager.GetNumActiveExtensionsForSource(LOG_SOURCE_UI_LATEST));
+}
+
+TEST(LogSourceAccessManagerTest, LastAccessTime) {
+ LogSourceAccessManager manager;
+
+ const SourceAndExtension key_a0 = std::make_pair(LOG_SOURCE_MESSAGES, "aaaa");
+ const SourceAndExtension key_a1 =
+ std::make_pair(LOG_SOURCE_UI_LATEST, "aaaa");
+ const SourceAndExtension key_b0 = std::make_pair(LOG_SOURCE_MESSAGES, "bbbb");
+ const SourceAndExtension key_b1 =
+ std::make_pair(LOG_SOURCE_UI_LATEST, "bbbb");
+ const SourceAndExtension key_c0 = std::make_pair(LOG_SOURCE_MESSAGES, "cccc");
+ const SourceAndExtension key_c1 =
+ std::make_pair(LOG_SOURCE_UI_LATEST, "cccc");
+
+ EXPECT_TRUE(manager.GetLastExtensionAccessTime(key_a0).is_null());
+ EXPECT_TRUE(manager.GetLastExtensionAccessTime(key_a1).is_null());
+ EXPECT_TRUE(manager.GetLastExtensionAccessTime(key_b0).is_null());
+ EXPECT_TRUE(manager.GetLastExtensionAccessTime(key_b1).is_null());
+ EXPECT_TRUE(manager.GetLastExtensionAccessTime(key_c0).is_null());
+ EXPECT_TRUE(manager.GetLastExtensionAccessTime(key_c1).is_null());
+
+ manager.AccessSourceFromExtension(key_a0);
+ manager.AccessSourceFromExtension(key_b0);
+ manager.AccessSourceFromExtension(key_c0);
+ manager.AccessSourceFromExtension(key_a1);
+ manager.AccessSourceFromExtension(key_b1);
+ manager.AccessSourceFromExtension(key_c1);
+
+ EXPECT_FALSE(manager.GetLastExtensionAccessTime(key_a0).is_null());
+ EXPECT_FALSE(manager.GetLastExtensionAccessTime(key_a1).is_null());
+ EXPECT_FALSE(manager.GetLastExtensionAccessTime(key_b0).is_null());
+ EXPECT_FALSE(manager.GetLastExtensionAccessTime(key_b1).is_null());
+ EXPECT_FALSE(manager.GetLastExtensionAccessTime(key_c0).is_null());
+ EXPECT_FALSE(manager.GetLastExtensionAccessTime(key_c1).is_null());
+}
+
+} // namespace extensions
« chrome/browser/extensions/api/feedback_private/log_source_access_manager.h ('K') | « chrome/browser/extensions/api/feedback_private/log_source_access_manager.cc ('k') | chrome/browser/extensions/api/feedback_private/log_source_resource.h » ('j') | chrome/browser/extensions/api/feedback_private/single_log_source_factory.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698