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

Unified Diff: components/offline_pages/background/request_coordinator_event_logger_unittest.cc

Issue 2489443002: Move all components/offline_pages/ files into component/offline_pages/core (Closed)
Patch Set: more rebase Created 4 years, 1 month 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: components/offline_pages/background/request_coordinator_event_logger_unittest.cc
diff --git a/components/offline_pages/background/request_coordinator_event_logger_unittest.cc b/components/offline_pages/background/request_coordinator_event_logger_unittest.cc
deleted file mode 100644
index eed1f45b3259850c71995aafa3a3fddb6894051b..0000000000000000000000000000000000000000
--- a/components/offline_pages/background/request_coordinator_event_logger_unittest.cc
+++ /dev/null
@@ -1,72 +0,0 @@
-// 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 "components/offline_pages/background/request_coordinator_event_logger.h"
-
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace offline_pages {
-
-namespace {
-
-const char kNamespace[] = "last_n";
-const Offliner::RequestStatus kOfflinerStatus = Offliner::SAVED;
-const RequestNotifier::BackgroundSavePageResult kDroppedResult =
- RequestNotifier::BackgroundSavePageResult::START_COUNT_EXCEEDED;
-const int64_t kId = 1234;
-const UpdateRequestResult kQueueUpdateResult =
- UpdateRequestResult::STORE_FAILURE;
-
-const char kOfflinerStatusLogString[] =
- "Background save attempt for last_n:1234 - SAVED";
-const char kDroppedResultLogString[] =
- "Background save request removed last_n:1234 - START_COUNT_EXCEEDED";
-const char kQueueUpdateResultLogString[] =
- "Updating queued request for last_n failed - STORE_FAILURE";
-const int kTimeLength = 21;
-
-} // namespace
-
-TEST(RequestCoordinatorEventLoggerTest, RecordsWhenLoggingIsOn) {
- RequestCoordinatorEventLogger logger;
- std::vector<std::string> log;
-
- logger.SetIsLogging(true);
- logger.RecordOfflinerResult(kNamespace, kOfflinerStatus, kId);
- logger.RecordDroppedSavePageRequest(kNamespace, kDroppedResult, kId);
- logger.RecordUpdateRequestFailed(kNamespace, kQueueUpdateResult);
- logger.GetLogs(&log);
-
- EXPECT_EQ(3u, log.size());
- EXPECT_EQ(std::string(kQueueUpdateResultLogString),
- log[0].substr(kTimeLength));
- EXPECT_EQ(std::string(kDroppedResultLogString), log[1].substr(kTimeLength));
- EXPECT_EQ(std::string(kOfflinerStatusLogString), log[2].substr(kTimeLength));
-}
-
-TEST(RequestCoordinatorEventLoggerTest, RecordsWhenLoggingIsOff) {
- RequestCoordinatorEventLogger logger;
- std::vector<std::string> log;
-
- logger.SetIsLogging(false);
- logger.RecordOfflinerResult(kNamespace, kOfflinerStatus, kId);
- logger.GetLogs(&log);
-
- EXPECT_EQ(0u, log.size());
-}
-
-TEST(RequestCoordinatorEventLoggerTest, DoesNotExceedMaxSize) {
- RequestCoordinatorEventLogger logger;
- std::vector<std::string> log;
-
- logger.SetIsLogging(true);
- for (size_t i = 0; i < kMaxLogCount + 1; ++i) {
- logger.RecordOfflinerResult(kNamespace, kOfflinerStatus, kId);
- }
- logger.GetLogs(&log);
-
- EXPECT_EQ(kMaxLogCount, log.size());
-}
-
-} // namespace offline_pages

Powered by Google App Engine
This is Rietveld 408576698