Bertrand SIMONNET | 52e5b99 | 2015-08-10 15:18:00 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2015 The Android Open Source Project |
| 3 | * |
| 4 | * Licensed under the Apache License, Version 2.0 (the "License"); |
| 5 | * you may not use this file except in compliance with the License. |
| 6 | * You may obtain a copy of the License at |
| 7 | * |
| 8 | * http://www.apache.org/licenses/LICENSE-2.0 |
| 9 | * |
| 10 | * Unless required by applicable law or agreed to in writing, software |
| 11 | * distributed under the License is distributed on an "AS IS" BASIS, |
| 12 | * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 13 | * See the License for the specific language governing permissions and |
| 14 | * limitations under the License. |
| 15 | */ |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 16 | |
Bertrand SIMONNET | 6c9fbb9 | 2015-12-21 14:56:40 -0800 | [diff] [blame] | 17 | #include <memory> |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 18 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 19 | #include <base/at_exit.h> |
| 20 | #include <base/files/file_util.h> |
| 21 | #include <base/files/scoped_temp_dir.h> |
| 22 | #include <base/logging.h> |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 23 | #include <base/metrics/sparse_histogram.h> |
| 24 | #include <base/metrics/statistics_recorder.h> |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 25 | #include <base/sys_info.h> |
Bertrand SIMONNET | 6c9fbb9 | 2015-12-21 14:56:40 -0800 | [diff] [blame] | 26 | #include <gtest/gtest.h> |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 27 | |
| 28 | #include "constants.h" |
Bertrand SIMONNET | 1df10c4 | 2015-09-09 10:39:51 -0700 | [diff] [blame] | 29 | #include "persistent_integer.h" |
Bertrand SIMONNET | 4b915ae | 2015-07-28 15:38:14 -0700 | [diff] [blame] | 30 | #include "uploader/metrics_log.h" |
| 31 | #include "uploader/mock/mock_system_profile_setter.h" |
| 32 | #include "uploader/mock/sender_mock.h" |
| 33 | #include "uploader/proto/chrome_user_metrics_extension.pb.h" |
| 34 | #include "uploader/proto/histogram_event.pb.h" |
| 35 | #include "uploader/proto/system_profile.pb.h" |
| 36 | #include "uploader/system_profile_cache.h" |
| 37 | #include "uploader/upload_service.h" |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 38 | |
| 39 | class UploadServiceTest : public testing::Test { |
| 40 | protected: |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 41 | virtual void SetUp() { |
| 42 | CHECK(dir_.CreateUniqueTempDir()); |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 43 | // Make sure the statistics recorder is inactive (contains no metrics) then |
| 44 | // initialize it. |
| 45 | ASSERT_FALSE(base::StatisticsRecorder::IsActive()); |
| 46 | base::StatisticsRecorder::Initialize(); |
Bertrand SIMONNET | 9d3a4ae | 2015-11-25 13:49:12 -0800 | [diff] [blame] | 47 | |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 48 | private_dir_ = dir_.path().Append("private"); |
| 49 | shared_dir_ = dir_.path().Append("shared"); |
Bertrand SIMONNET | 9d3a4ae | 2015-11-25 13:49:12 -0800 | [diff] [blame] | 50 | |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 51 | EXPECT_TRUE(base::CreateDirectory(private_dir_)); |
| 52 | EXPECT_TRUE(base::CreateDirectory(shared_dir_)); |
Bertrand SIMONNET | 9d3a4ae | 2015-11-25 13:49:12 -0800 | [diff] [blame] | 53 | |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 54 | ASSERT_EQ(0, base::WriteFile(shared_dir_.Append(metrics::kConsentFileName), |
Bertrand SIMONNET | 9d3a4ae | 2015-11-25 13:49:12 -0800 | [diff] [blame] | 55 | "", 0)); |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 56 | |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 57 | upload_service_.reset(new UploadService( |
| 58 | "", base::TimeDelta(), base::TimeDelta(), private_dir_, shared_dir_)); |
Bertrand SIMONNET | b6c77af | 2015-12-08 17:46:00 -0800 | [diff] [blame] | 59 | counters_ = upload_service_->counters_; |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 60 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 61 | upload_service_->sender_.reset(new SenderMock); |
Bertrand SIMONNET | 608e428 | 2015-11-12 17:52:17 -0800 | [diff] [blame] | 62 | upload_service_->InitForTest(new MockSystemProfileSetter); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 63 | upload_service_->GatherHistograms(); |
| 64 | upload_service_->Reset(); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 65 | } |
| 66 | |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 67 | void SendSparseHistogram(const std::string& name, int sample) { |
| 68 | base::HistogramBase* histogram = base::SparseHistogram::FactoryGet( |
| 69 | name, base::Histogram::kUmaTargetedHistogramFlag); |
| 70 | histogram->Add(sample); |
| 71 | } |
| 72 | |
| 73 | void SendHistogram( |
| 74 | const std::string& name, int sample, int min, int max, int nbuckets) { |
| 75 | base::HistogramBase* histogram = base::Histogram::FactoryGet( |
| 76 | name, min, max, nbuckets, base::Histogram::kUmaTargetedHistogramFlag); |
| 77 | histogram->Add(sample); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 78 | } |
| 79 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 80 | void SetTestingProperty(const std::string& name, const std::string& value) { |
Bertrand SIMONNET | 608e428 | 2015-11-12 17:52:17 -0800 | [diff] [blame] | 81 | base::FilePath filepath = |
| 82 | dir_.path().Append("etc/os-release.d").Append(name); |
Bertrand SIMONNET | eb697ab | 2015-10-14 13:26:42 -0700 | [diff] [blame] | 83 | ASSERT_TRUE(base::CreateDirectory(filepath.DirName())); |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 84 | ASSERT_EQ(value.size(), |
| 85 | base::WriteFile(filepath, value.data(), value.size())); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 86 | } |
| 87 | |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 88 | const metrics::SystemProfileProto_Stability GetCurrentStability() { |
Bertrand SIMONNET | 6c9fbb9 | 2015-12-21 14:56:40 -0800 | [diff] [blame] | 89 | EXPECT_TRUE(upload_service_->current_log_.get()); |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 90 | |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 91 | return upload_service_->current_log_->uma_proto() |
| 92 | ->system_profile() |
| 93 | .stability(); |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 94 | } |
| 95 | |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 96 | base::ScopedTempDir dir_; |
Bertrand SIMONNET | 6c9fbb9 | 2015-12-21 14:56:40 -0800 | [diff] [blame] | 97 | std::unique_ptr<UploadService> upload_service_; |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 98 | |
Bertrand SIMONNET | 6c9fbb9 | 2015-12-21 14:56:40 -0800 | [diff] [blame] | 99 | std::unique_ptr<base::AtExitManager> exit_manager_; |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 100 | std::shared_ptr<CrashCounters> counters_; |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 101 | base::FilePath private_dir_; |
| 102 | base::FilePath shared_dir_; |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 103 | }; |
| 104 | |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 105 | TEST_F(UploadServiceTest, FailedSendAreRetried) { |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 106 | SenderMock* sender = new SenderMock(); |
| 107 | upload_service_->sender_.reset(sender); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 108 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 109 | sender->set_should_succeed(false); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 110 | |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 111 | SendSparseHistogram("hello", 1); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 112 | upload_service_->UploadEvent(); |
| 113 | EXPECT_EQ(1, sender->send_call_count()); |
| 114 | std::string sent_string = sender->last_message(); |
| 115 | |
| 116 | upload_service_->UploadEvent(); |
| 117 | EXPECT_EQ(2, sender->send_call_count()); |
| 118 | EXPECT_EQ(sent_string, sender->last_message()); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 119 | } |
| 120 | |
| 121 | TEST_F(UploadServiceTest, DiscardLogsAfterTooManyFailedUpload) { |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 122 | SenderMock* sender = new SenderMock(); |
| 123 | upload_service_->sender_.reset(sender); |
| 124 | |
| 125 | sender->set_should_succeed(false); |
| 126 | |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 127 | SendSparseHistogram("hello", 1); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 128 | |
| 129 | for (int i = 0; i < UploadService::kMaxFailedUpload; i++) { |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 130 | upload_service_->UploadEvent(); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 131 | } |
| 132 | |
Bertrand SIMONNET | 1df10c4 | 2015-09-09 10:39:51 -0700 | [diff] [blame] | 133 | EXPECT_TRUE(upload_service_->HasStagedLog()); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 134 | upload_service_->UploadEvent(); |
Bertrand SIMONNET | 1df10c4 | 2015-09-09 10:39:51 -0700 | [diff] [blame] | 135 | EXPECT_FALSE(upload_service_->HasStagedLog()); |
| 136 | |
| 137 | // Log a new sample. The failed upload counter should be reset. |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 138 | SendSparseHistogram("hello", 1); |
Bertrand SIMONNET | 1df10c4 | 2015-09-09 10:39:51 -0700 | [diff] [blame] | 139 | for (int i = 0; i < UploadService::kMaxFailedUpload; i++) { |
| 140 | upload_service_->UploadEvent(); |
| 141 | } |
| 142 | // The log is not discarded after multiple failed uploads. |
| 143 | EXPECT_TRUE(upload_service_->HasStagedLog()); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 144 | } |
| 145 | |
| 146 | TEST_F(UploadServiceTest, EmptyLogsAreNotSent) { |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 147 | SenderMock* sender = new SenderMock(); |
| 148 | upload_service_->sender_.reset(sender); |
| 149 | upload_service_->UploadEvent(); |
| 150 | EXPECT_FALSE(upload_service_->current_log_); |
| 151 | EXPECT_EQ(0, sender->send_call_count()); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 152 | } |
| 153 | |
| 154 | TEST_F(UploadServiceTest, LogEmptyByDefault) { |
Bertrand SIMONNET | b6c77af | 2015-12-08 17:46:00 -0800 | [diff] [blame] | 155 | // current_log_ should be initialized later as it needs AtExitManager to exist |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 156 | // in order to gather system information from SysInfo. |
Bertrand SIMONNET | b6c77af | 2015-12-08 17:46:00 -0800 | [diff] [blame] | 157 | EXPECT_FALSE(upload_service_->current_log_); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 158 | } |
| 159 | |
| 160 | TEST_F(UploadServiceTest, CanSendMultipleTimes) { |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 161 | SenderMock* sender = new SenderMock(); |
| 162 | upload_service_->sender_.reset(sender); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 163 | |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 164 | SendSparseHistogram("hello", 1); |
| 165 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 166 | upload_service_->UploadEvent(); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 167 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 168 | std::string first_message = sender->last_message(); |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 169 | SendSparseHistogram("hello", 2); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 170 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 171 | upload_service_->UploadEvent(); |
| 172 | |
| 173 | EXPECT_NE(first_message, sender->last_message()); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 174 | } |
| 175 | |
| 176 | TEST_F(UploadServiceTest, LogEmptyAfterUpload) { |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 177 | SendSparseHistogram("hello", 2); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 178 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 179 | upload_service_->UploadEvent(); |
| 180 | EXPECT_FALSE(upload_service_->current_log_); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 181 | } |
| 182 | |
| 183 | TEST_F(UploadServiceTest, LogContainsAggregatedValues) { |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 184 | SendHistogram("foo", 11, 0, 42, 10); |
| 185 | SendHistogram("foo", 12, 0, 42, 10); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 186 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 187 | upload_service_->GatherHistograms(); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 188 | metrics::ChromeUserMetricsExtension* proto = |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 189 | upload_service_->current_log_->uma_proto(); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 190 | EXPECT_EQ(1, proto->histogram_event().size()); |
| 191 | } |
| 192 | |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 193 | TEST_F(UploadServiceTest, LogContainsCrashCounts) { |
| 194 | // By default, there is no current log. |
| 195 | upload_service_->GatherHistograms(); |
| 196 | EXPECT_FALSE(upload_service_->current_log_); |
| 197 | |
| 198 | // If the user crash counter is incremented, we add the count to the current |
| 199 | // log. |
| 200 | counters_->IncrementUserCrashCount(); |
| 201 | upload_service_->GatherHistograms(); |
| 202 | EXPECT_EQ(1, GetCurrentStability().other_user_crash_count()); |
| 203 | |
| 204 | // If the kernel crash counter is incremented, we add the count to the current |
| 205 | // log. |
| 206 | counters_->IncrementKernelCrashCount(); |
| 207 | upload_service_->GatherHistograms(); |
| 208 | EXPECT_EQ(1, GetCurrentStability().kernel_crash_count()); |
| 209 | |
| 210 | // If the kernel crash counter is incremented, we add the count to the current |
| 211 | // log. |
| 212 | counters_->IncrementUncleanShutdownCount(); |
| 213 | counters_->IncrementUncleanShutdownCount(); |
| 214 | upload_service_->GatherHistograms(); |
| 215 | EXPECT_EQ(2, GetCurrentStability().unclean_system_shutdown_count()); |
| 216 | |
| 217 | // If no counter is incremented, the reported numbers don't change. |
| 218 | upload_service_->GatherHistograms(); |
| 219 | EXPECT_EQ(1, GetCurrentStability().other_user_crash_count()); |
| 220 | EXPECT_EQ(1, GetCurrentStability().kernel_crash_count()); |
| 221 | EXPECT_EQ(2, GetCurrentStability().unclean_system_shutdown_count()); |
| 222 | } |
| 223 | |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 224 | TEST_F(UploadServiceTest, ExtractChannelFromString) { |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 225 | EXPECT_EQ(SystemProfileCache::ProtoChannelFromString("developer-build"), |
| 226 | metrics::SystemProfileProto::CHANNEL_UNKNOWN); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 227 | |
| 228 | EXPECT_EQ(metrics::SystemProfileProto::CHANNEL_DEV, |
Bertrand SIMONNET | dc225c8 | 2015-11-12 14:04:03 -0800 | [diff] [blame] | 229 | SystemProfileCache::ProtoChannelFromString("dev-channel")); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 230 | |
| 231 | EXPECT_EQ(metrics::SystemProfileProto::CHANNEL_STABLE, |
Bertrand SIMONNET | dc225c8 | 2015-11-12 14:04:03 -0800 | [diff] [blame] | 232 | SystemProfileCache::ProtoChannelFromString("stable-channel")); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 233 | |
| 234 | EXPECT_EQ(metrics::SystemProfileProto::CHANNEL_UNKNOWN, |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 235 | SystemProfileCache::ProtoChannelFromString("this is a test")); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 236 | } |
| 237 | |
| 238 | TEST_F(UploadServiceTest, ValuesInConfigFileAreSent) { |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 239 | SenderMock* sender = new SenderMock(); |
| 240 | upload_service_->sender_.reset(sender); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 241 | |
Bertrand SIMONNET | eb697ab | 2015-10-14 13:26:42 -0700 | [diff] [blame] | 242 | SetTestingProperty(metrics::kProductId, "hello"); |
| 243 | SetTestingProperty(metrics::kProductVersion, "1.2.3.4"); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 244 | |
Bertrand SIMONNET | 6b8629a | 2015-11-18 13:46:33 -0800 | [diff] [blame] | 245 | SendSparseHistogram("hello", 1); |
| 246 | |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 247 | // Reset to create the new log with the profile setter. |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 248 | upload_service_->system_profile_setter_.reset( |
| 249 | new SystemProfileCache(true, dir_.path())); |
| 250 | upload_service_->Reset(); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 251 | upload_service_->UploadEvent(); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 252 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 253 | EXPECT_EQ(1, sender->send_call_count()); |
| 254 | EXPECT_TRUE(sender->is_good_proto()); |
| 255 | EXPECT_EQ(1, sender->last_message_proto().histogram_event().size()); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 256 | |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 257 | EXPECT_NE(0, sender->last_message_proto().client_id()); |
| 258 | EXPECT_NE(0, sender->last_message_proto().system_profile().build_timestamp()); |
| 259 | EXPECT_NE(0, sender->last_message_proto().session_id()); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 260 | } |
| 261 | |
| 262 | TEST_F(UploadServiceTest, PersistentGUID) { |
| 263 | std::string tmp_file = dir_.path().Append("tmpfile").value(); |
| 264 | |
| 265 | std::string first_guid = SystemProfileCache::GetPersistentGUID(tmp_file); |
| 266 | std::string second_guid = SystemProfileCache::GetPersistentGUID(tmp_file); |
| 267 | |
| 268 | // The GUID are cached. |
| 269 | EXPECT_EQ(first_guid, second_guid); |
| 270 | |
| 271 | base::DeleteFile(base::FilePath(tmp_file), false); |
| 272 | |
| 273 | first_guid = SystemProfileCache::GetPersistentGUID(tmp_file); |
| 274 | base::DeleteFile(base::FilePath(tmp_file), false); |
| 275 | second_guid = SystemProfileCache::GetPersistentGUID(tmp_file); |
| 276 | |
| 277 | // Random GUIDs are generated (not all the same). |
| 278 | EXPECT_NE(first_guid, second_guid); |
| 279 | } |
| 280 | |
| 281 | TEST_F(UploadServiceTest, SessionIdIncrementedAtInitialization) { |
Bertrand SIMONNET | eb697ab | 2015-10-14 13:26:42 -0700 | [diff] [blame] | 282 | SetTestingProperty(metrics::kProductId, "hello"); |
Bertrand SIMONNET | 1253186 | 2015-08-31 11:11:57 -0700 | [diff] [blame] | 283 | SystemProfileCache cache(true, dir_.path()); |
| 284 | cache.Initialize(); |
| 285 | int session_id = cache.profile_.session_id; |
| 286 | cache.initialized_ = false; |
| 287 | cache.Initialize(); |
| 288 | EXPECT_EQ(cache.profile_.session_id, session_id + 1); |
Bertrand SIMONNET | 46b49da | 2014-06-25 14:38:07 -0700 | [diff] [blame] | 289 | } |
Bertrand SIMONNET | 4c8a8ad | 2015-09-09 13:27:33 -0700 | [diff] [blame] | 290 | |
Bertrand SIMONNET | 1d15d46 | 2015-11-17 13:20:06 -0800 | [diff] [blame] | 291 | // The product id must be set for metrics to be uploaded. |
| 292 | // If it is not set, the system profile cache should fail to initialize. |
| 293 | TEST_F(UploadServiceTest, ProductIdMandatory) { |
| 294 | SystemProfileCache cache(true, dir_.path()); |
| 295 | ASSERT_FALSE(cache.Initialize()); |
| 296 | SetTestingProperty(metrics::kProductId, ""); |
| 297 | ASSERT_FALSE(cache.Initialize()); |
| 298 | SetTestingProperty(metrics::kProductId, "hello"); |
| 299 | ASSERT_TRUE(cache.Initialize()); |
| 300 | } |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 301 | |
| 302 | TEST_F(UploadServiceTest, CurrentLogSavedAndResumed) { |
| 303 | SendHistogram("hello", 10, 0, 100, 10); |
| 304 | upload_service_->PersistToDisk(); |
| 305 | EXPECT_EQ( |
| 306 | 1, upload_service_->current_log_->uma_proto()->histogram_event().size()); |
Alex Deymo | de4497e | 2016-02-10 11:55:06 -0800 | [diff] [blame] | 307 | // Destroy the old service before creating a new one. |
| 308 | upload_service_.reset(); |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 309 | upload_service_.reset(new UploadService( |
| 310 | "", base::TimeDelta(), base::TimeDelta(), private_dir_, shared_dir_)); |
| 311 | upload_service_->InitForTest(nullptr); |
| 312 | |
| 313 | SendHistogram("hello", 10, 0, 100, 10); |
| 314 | upload_service_->GatherHistograms(); |
| 315 | EXPECT_EQ(2, upload_service_->GetOrCreateCurrentLog() |
| 316 | ->uma_proto() |
| 317 | ->histogram_event() |
| 318 | .size()); |
| 319 | } |
| 320 | |
| 321 | TEST_F(UploadServiceTest, PersistEmptyLog) { |
| 322 | upload_service_->PersistToDisk(); |
| 323 | EXPECT_FALSE(base::PathExists(upload_service_->saved_log_path_)); |
| 324 | } |
| 325 | |
| 326 | TEST_F(UploadServiceTest, CorruptedSavedLog) { |
| 327 | // Write a bogus saved log. |
| 328 | EXPECT_EQ(5, base::WriteFile(upload_service_->saved_log_path_, "hello", 5)); |
| 329 | |
Alex Deymo | de4497e | 2016-02-10 11:55:06 -0800 | [diff] [blame] | 330 | // Destroy the old service before creating a new one. |
| 331 | upload_service_.reset(); |
Bertrand SIMONNET | 0586504 | 2015-12-15 12:33:01 -0800 | [diff] [blame] | 332 | upload_service_.reset(new UploadService( |
| 333 | "", base::TimeDelta(), base::TimeDelta(), private_dir_, shared_dir_)); |
| 334 | |
| 335 | upload_service_->InitForTest(nullptr); |
| 336 | // If the log is unreadable, we drop it and continue execution. |
| 337 | ASSERT_NE(nullptr, upload_service_->GetOrCreateCurrentLog()); |
| 338 | ASSERT_FALSE(base::PathExists(upload_service_->saved_log_path_)); |
| 339 | } |