bsimonnet@chromium.org | 064107e | 2014-05-02 00:59:06 | [diff] [blame] | 1 | // Copyright 2014 The Chromium Authors. All rights reserved. |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
bsimonnet@chromium.org | 064107e | 2014-05-02 00:59:06 | [diff] [blame] | 5 | #include "components/metrics/metrics_log_manager.h" |
asvitkine@chromium.org | cfee9aa5 | 2013-10-19 17:53:05 | [diff] [blame] | 6 | |
isherman@chromium.org | fe58acc2 | 2012-02-29 01:29:58 | [diff] [blame] | 7 | #include <string> |
| 8 | #include <utility> |
| 9 | #include <vector> |
| 10 | |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 11 | #include "base/prefs/pref_registry_simple.h" |
| 12 | #include "base/prefs/testing_pref_service.h" |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 13 | #include "components/metrics/metrics_log.h" |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 14 | #include "components/metrics/metrics_pref_names.h" |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 15 | #include "components/metrics/test_metrics_service_client.h" |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 16 | #include "testing/gtest/include/gtest/gtest.h" |
| 17 | |
bsimonnet@chromium.org | 064107e | 2014-05-02 00:59:06 | [diff] [blame] | 18 | namespace metrics { |
| 19 | |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 20 | namespace { |
isherman@chromium.org | fe58acc2 | 2012-02-29 01:29:58 | [diff] [blame] | 21 | |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 22 | // Dummy serializer that just stores logs in memory. |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 23 | class TestLogPrefService : public TestingPrefServiceSimple { |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 24 | public: |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 25 | TestLogPrefService() { |
| 26 | registry()->RegisterListPref(prefs::kMetricsInitialLogs); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 27 | registry()->RegisterListPref(prefs::kMetricsInitialLogsOld); |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 28 | registry()->RegisterListPref(prefs::kMetricsOngoingLogs); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 29 | registry()->RegisterListPref(prefs::kMetricsOngoingLogsOld); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 30 | } |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 31 | |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 32 | // Returns the number of logs of the given type. |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 33 | size_t TypeCount(MetricsLog::LogType log_type) { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 34 | int list_length = 0; |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 35 | if (log_type == MetricsLog::INITIAL_STABILITY_LOG) |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 36 | list_length = GetList(prefs::kMetricsInitialLogs)->GetSize(); |
| 37 | else |
| 38 | list_length = GetList(prefs::kMetricsOngoingLogs)->GetSize(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 39 | return list_length / 2; |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 40 | } |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 41 | }; |
isherman@chromium.org | fe58acc2 | 2012-02-29 01:29:58 | [diff] [blame] | 42 | |
| 43 | } // namespace |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 44 | |
| 45 | TEST(MetricsLogManagerTest, StandardFlow) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 46 | TestMetricsServiceClient client; |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 47 | TestLogPrefService pref_service; |
| 48 | MetricsLogManager log_manager(&pref_service, 0); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 49 | |
| 50 | // Make sure a new manager has a clean slate. |
| 51 | EXPECT_EQ(NULL, log_manager.current_log()); |
| 52 | EXPECT_FALSE(log_manager.has_staged_log()); |
| 53 | EXPECT_FALSE(log_manager.has_unsent_logs()); |
| 54 | |
| 55 | // Check that the normal flow works. |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 56 | MetricsLog* initial_log = new MetricsLog( |
| 57 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service); |
isherman@chromium.org | 09dee82d | 2014-05-22 14:00:53 | [diff] [blame] | 58 | log_manager.BeginLoggingWithLog(make_scoped_ptr(initial_log)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 59 | EXPECT_EQ(initial_log, log_manager.current_log()); |
| 60 | EXPECT_FALSE(log_manager.has_staged_log()); |
| 61 | |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 62 | log_manager.FinishCurrentLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 63 | EXPECT_EQ(NULL, log_manager.current_log()); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 64 | EXPECT_TRUE(log_manager.has_unsent_logs()); |
| 65 | EXPECT_FALSE(log_manager.has_staged_log()); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 66 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 67 | MetricsLog* second_log = |
| 68 | new MetricsLog("id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service); |
isherman@chromium.org | 09dee82d | 2014-05-22 14:00:53 | [diff] [blame] | 69 | log_manager.BeginLoggingWithLog(make_scoped_ptr(second_log)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 70 | EXPECT_EQ(second_log, log_manager.current_log()); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 71 | |
| 72 | log_manager.StageNextLogForUpload(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 73 | EXPECT_TRUE(log_manager.has_staged_log()); |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 74 | EXPECT_FALSE(log_manager.staged_log().empty()); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 75 | |
isherman@chromium.org | 5f3e164 | 2013-05-05 03:37:34 | [diff] [blame] | 76 | log_manager.DiscardStagedLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 77 | EXPECT_EQ(second_log, log_manager.current_log()); |
| 78 | EXPECT_FALSE(log_manager.has_staged_log()); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 79 | EXPECT_FALSE(log_manager.has_unsent_logs()); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 80 | |
| 81 | EXPECT_FALSE(log_manager.has_unsent_logs()); |
| 82 | } |
| 83 | |
| 84 | TEST(MetricsLogManagerTest, AbandonedLog) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 85 | TestMetricsServiceClient client; |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 86 | TestLogPrefService pref_service; |
| 87 | MetricsLogManager log_manager(&pref_service, 0); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 88 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 89 | MetricsLog* dummy_log = new MetricsLog( |
| 90 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service); |
isherman@chromium.org | 09dee82d | 2014-05-22 14:00:53 | [diff] [blame] | 91 | log_manager.BeginLoggingWithLog(make_scoped_ptr(dummy_log)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 92 | EXPECT_EQ(dummy_log, log_manager.current_log()); |
| 93 | |
| 94 | log_manager.DiscardCurrentLog(); |
| 95 | EXPECT_EQ(NULL, log_manager.current_log()); |
| 96 | EXPECT_FALSE(log_manager.has_staged_log()); |
| 97 | } |
| 98 | |
| 99 | TEST(MetricsLogManagerTest, InterjectedLog) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 100 | TestMetricsServiceClient client; |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 101 | TestLogPrefService pref_service; |
| 102 | MetricsLogManager log_manager(&pref_service, 0); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 103 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 104 | MetricsLog* ongoing_log = |
| 105 | new MetricsLog("id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service); |
| 106 | MetricsLog* temp_log = new MetricsLog( |
| 107 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 108 | |
isherman@chromium.org | 09dee82d | 2014-05-22 14:00:53 | [diff] [blame] | 109 | log_manager.BeginLoggingWithLog(make_scoped_ptr(ongoing_log)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 110 | EXPECT_EQ(ongoing_log, log_manager.current_log()); |
| 111 | |
| 112 | log_manager.PauseCurrentLog(); |
| 113 | EXPECT_EQ(NULL, log_manager.current_log()); |
| 114 | |
isherman@chromium.org | 09dee82d | 2014-05-22 14:00:53 | [diff] [blame] | 115 | log_manager.BeginLoggingWithLog(make_scoped_ptr(temp_log)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 116 | EXPECT_EQ(temp_log, log_manager.current_log()); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 117 | log_manager.FinishCurrentLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 118 | EXPECT_EQ(NULL, log_manager.current_log()); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 119 | |
| 120 | log_manager.ResumePausedLog(); |
| 121 | EXPECT_EQ(ongoing_log, log_manager.current_log()); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 122 | |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 123 | EXPECT_FALSE(log_manager.has_staged_log()); |
| 124 | log_manager.StageNextLogForUpload(); |
isherman@chromium.org | 5f3e164 | 2013-05-05 03:37:34 | [diff] [blame] | 125 | log_manager.DiscardStagedLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 126 | EXPECT_FALSE(log_manager.has_unsent_logs()); |
| 127 | } |
| 128 | |
stuartmorgan@chromium.org | aa752c3f | 2012-04-27 22:31:51 | [diff] [blame] | 129 | TEST(MetricsLogManagerTest, InterjectedLogPreservesType) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 130 | TestMetricsServiceClient client; |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 131 | TestLogPrefService pref_service; |
| 132 | MetricsLogManager log_manager(&pref_service, 0); |
asvitkine@chromium.org | 80a8f31 | 2013-12-16 18:00:30 | [diff] [blame] | 133 | log_manager.LoadPersistedUnsentLogs(); |
stuartmorgan@chromium.org | aa752c3f | 2012-04-27 22:31:51 | [diff] [blame] | 134 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 135 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 136 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | aa752c3f | 2012-04-27 22:31:51 | [diff] [blame] | 137 | log_manager.PauseCurrentLog(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 138 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 139 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | aa752c3f | 2012-04-27 22:31:51 | [diff] [blame] | 140 | log_manager.FinishCurrentLog(); |
| 141 | log_manager.ResumePausedLog(); |
| 142 | log_manager.StageNextLogForUpload(); |
isherman@chromium.org | 5f3e164 | 2013-05-05 03:37:34 | [diff] [blame] | 143 | log_manager.DiscardStagedLog(); |
stuartmorgan@chromium.org | aa752c3f | 2012-04-27 22:31:51 | [diff] [blame] | 144 | |
| 145 | // Verify that the remaining log (which is the original ongoing log) still |
| 146 | // has the right type. |
stuartmorgan@chromium.org | aa752c3f | 2012-04-27 22:31:51 | [diff] [blame] | 147 | log_manager.FinishCurrentLog(); |
| 148 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 149 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 150 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | aa752c3f | 2012-04-27 22:31:51 | [diff] [blame] | 151 | } |
| 152 | |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 153 | TEST(MetricsLogManagerTest, StoreAndLoad) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 154 | TestMetricsServiceClient client; |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 155 | TestLogPrefService pref_service; |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 156 | // Set up some in-progress logging in a scoped log manager simulating the |
| 157 | // leadup to quitting, then persist as would be done on quit. |
| 158 | { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 159 | MetricsLogManager log_manager(&pref_service, 0); |
asvitkine@chromium.org | 80a8f31 | 2013-12-16 18:00:30 | [diff] [blame] | 160 | log_manager.LoadPersistedUnsentLogs(); |
| 161 | |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 162 | // Simulate a log having already been unsent from a previous session. |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 163 | { |
| 164 | std::string log("proto"); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 165 | PersistedLogs ongoing_logs(&pref_service, prefs::kMetricsOngoingLogs, |
| 166 | prefs::kMetricsOngoingLogsOld, 1, 1, 0); |
| 167 | ongoing_logs.StoreLog(log); |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 168 | ongoing_logs.SerializeLogs(); |
| 169 | } |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 170 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 171 | EXPECT_FALSE(log_manager.has_unsent_logs()); |
| 172 | log_manager.LoadPersistedUnsentLogs(); |
| 173 | EXPECT_TRUE(log_manager.has_unsent_logs()); |
| 174 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 175 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 176 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 177 | log_manager.FinishCurrentLog(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 178 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 179 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 180 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 181 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::NORMAL_STORE); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 182 | log_manager.FinishCurrentLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 183 | |
| 184 | // Nothing should be written out until PersistUnsentLogs is called. |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 185 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 186 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 187 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 188 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 189 | EXPECT_EQ(2U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 190 | } |
| 191 | |
| 192 | // Now simulate the relaunch, ensure that the log manager restores |
| 193 | // everything correctly, and verify that once the are handled they are not |
| 194 | // re-persisted. |
| 195 | { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 196 | MetricsLogManager log_manager(&pref_service, 0); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 197 | log_manager.LoadPersistedUnsentLogs(); |
| 198 | EXPECT_TRUE(log_manager.has_unsent_logs()); |
| 199 | |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 200 | log_manager.StageNextLogForUpload(); |
isherman@chromium.org | 5f3e164 | 2013-05-05 03:37:34 | [diff] [blame] | 201 | log_manager.DiscardStagedLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 202 | // The initial log should be sent first; update the persisted storage to |
| 203 | // verify. |
| 204 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 205 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 206 | EXPECT_EQ(2U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 207 | |
| 208 | // Handle the first ongoing log. |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 209 | log_manager.StageNextLogForUpload(); |
isherman@chromium.org | 5f3e164 | 2013-05-05 03:37:34 | [diff] [blame] | 210 | log_manager.DiscardStagedLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 211 | EXPECT_TRUE(log_manager.has_unsent_logs()); |
| 212 | |
| 213 | // Handle the last log. |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 214 | log_manager.StageNextLogForUpload(); |
isherman@chromium.org | 5f3e164 | 2013-05-05 03:37:34 | [diff] [blame] | 215 | log_manager.DiscardStagedLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 216 | EXPECT_FALSE(log_manager.has_unsent_logs()); |
| 217 | |
| 218 | // Nothing should have changed "on disk" since PersistUnsentLogs hasn't been |
| 219 | // called again. |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 220 | EXPECT_EQ(2U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 221 | // Persist, and make sure nothing is left. |
| 222 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 223 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 224 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 225 | } |
| 226 | } |
| 227 | |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 228 | TEST(MetricsLogManagerTest, StoreStagedLogTypes) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 229 | TestMetricsServiceClient client; |
| 230 | |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 231 | // Ensure that types are preserved when storing staged logs. |
| 232 | { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 233 | TestLogPrefService pref_service; |
| 234 | MetricsLogManager log_manager(&pref_service, 0); |
asvitkine@chromium.org | 80a8f31 | 2013-12-16 18:00:30 | [diff] [blame] | 235 | log_manager.LoadPersistedUnsentLogs(); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 236 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 237 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 238 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 239 | log_manager.FinishCurrentLog(); |
| 240 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 241 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::NORMAL_STORE); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 242 | log_manager.PersistUnsentLogs(); |
| 243 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 244 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 245 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 246 | } |
| 247 | |
| 248 | { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 249 | TestLogPrefService pref_service; |
| 250 | MetricsLogManager log_manager(&pref_service, 0); |
asvitkine@chromium.org | 80a8f31 | 2013-12-16 18:00:30 | [diff] [blame] | 251 | log_manager.LoadPersistedUnsentLogs(); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 252 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 253 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 254 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 255 | log_manager.FinishCurrentLog(); |
| 256 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 257 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::NORMAL_STORE); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 258 | log_manager.PersistUnsentLogs(); |
| 259 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 260 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 261 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | f829cc36 | 2012-03-10 10:09:16 | [diff] [blame] | 262 | } |
| 263 | } |
| 264 | |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 265 | TEST(MetricsLogManagerTest, LargeLogDiscarding) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 266 | TestMetricsServiceClient client; |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 267 | TestLogPrefService pref_service; |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 268 | // Set the size threshold very low, to verify that it's honored. |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 269 | MetricsLogManager log_manager(&pref_service, 1); |
| 270 | log_manager.LoadPersistedUnsentLogs(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 271 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 272 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 273 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 274 | log_manager.FinishCurrentLog(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 275 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 276 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | 2994826 | 2012-03-01 12:15:08 | [diff] [blame] | 277 | log_manager.FinishCurrentLog(); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 278 | |
| 279 | // Only the ongoing log should be written out, due to the threshold. |
| 280 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 281 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 282 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | cac267c | 2011-09-29 15:18:10 | [diff] [blame] | 283 | } |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 284 | |
| 285 | TEST(MetricsLogManagerTest, ProvisionalStoreStandardFlow) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 286 | TestMetricsServiceClient client; |
| 287 | |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 288 | // Ensure that provisional store works, and discards the correct log. |
| 289 | { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 290 | TestLogPrefService pref_service; |
| 291 | MetricsLogManager log_manager(&pref_service, 0); |
asvitkine@chromium.org | 80a8f31 | 2013-12-16 18:00:30 | [diff] [blame] | 292 | log_manager.LoadPersistedUnsentLogs(); |
| 293 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 294 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 295 | "id", 0, MetricsLog::INITIAL_STABILITY_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 296 | log_manager.FinishCurrentLog(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 297 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 298 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 299 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 300 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::PROVISIONAL_STORE); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 301 | log_manager.FinishCurrentLog(); |
| 302 | log_manager.DiscardLastProvisionalStore(); |
| 303 | |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 304 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 305 | EXPECT_EQ(0U, pref_service.TypeCount(MetricsLog::INITIAL_STABILITY_LOG)); |
| 306 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 307 | } |
| 308 | } |
| 309 | |
| 310 | TEST(MetricsLogManagerTest, ProvisionalStoreNoop) { |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 311 | TestMetricsServiceClient client; |
| 312 | |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 313 | // Ensure that trying to drop a sent log is a no-op, even if another log has |
| 314 | // since been staged. |
| 315 | { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 316 | TestLogPrefService pref_service; |
| 317 | MetricsLogManager log_manager(&pref_service, 0); |
asvitkine@chromium.org | 80a8f31 | 2013-12-16 18:00:30 | [diff] [blame] | 318 | log_manager.LoadPersistedUnsentLogs(); |
| 319 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 320 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 321 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 322 | log_manager.FinishCurrentLog(); |
| 323 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 324 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::PROVISIONAL_STORE); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 325 | log_manager.StageNextLogForUpload(); |
isherman@chromium.org | 5f3e164 | 2013-05-05 03:37:34 | [diff] [blame] | 326 | log_manager.DiscardStagedLog(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 327 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 328 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 329 | log_manager.FinishCurrentLog(); |
| 330 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 331 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::NORMAL_STORE); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 332 | log_manager.DiscardLastProvisionalStore(); |
| 333 | |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 334 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 335 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 336 | } |
| 337 | |
| 338 | // Ensure that trying to drop more than once is a no-op |
| 339 | { |
holte@chromium.org | 7f07db6 | 2014-05-15 01:12:45 | [diff] [blame] | 340 | TestLogPrefService pref_service; |
| 341 | MetricsLogManager log_manager(&pref_service, 0); |
asvitkine@chromium.org | 80a8f31 | 2013-12-16 18:00:30 | [diff] [blame] | 342 | log_manager.LoadPersistedUnsentLogs(); |
| 343 | |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 344 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 345 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 346 | log_manager.FinishCurrentLog(); |
| 347 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 348 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::NORMAL_STORE); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 349 | log_manager.BeginLoggingWithLog(make_scoped_ptr(new MetricsLog( |
| 350 | "id", 0, MetricsLog::ONGOING_LOG, &client, &pref_service))); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 351 | log_manager.FinishCurrentLog(); |
| 352 | log_manager.StageNextLogForUpload(); |
asvitkine@chromium.org | 9706e1b | 2014-06-11 16:31:24 | [diff] [blame^] | 353 | log_manager.StoreStagedLogAsUnsent(PersistedLogs::PROVISIONAL_STORE); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 354 | log_manager.DiscardLastProvisionalStore(); |
| 355 | log_manager.DiscardLastProvisionalStore(); |
| 356 | |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 357 | log_manager.PersistUnsentLogs(); |
asvitkine@chromium.org | bfb77b5 | 2014-06-07 01:54:01 | [diff] [blame] | 358 | EXPECT_EQ(1U, pref_service.TypeCount(MetricsLog::ONGOING_LOG)); |
stuartmorgan@chromium.org | e7508d8 | 2012-05-03 15:59:53 | [diff] [blame] | 359 | } |
| 360 | } |
asvitkine@chromium.org | cfee9aa5 | 2013-10-19 17:53:05 | [diff] [blame] | 361 | |
bsimonnet@chromium.org | 064107e | 2014-05-02 00:59:06 | [diff] [blame] | 362 | } // namespace metrics |