Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 1 | // Copyright (c) 2010 The Chromium OS Authors. All rights reserved. |
| 2 | // Use of this source code is governed by a BSD-style license that can be |
| 3 | // found in the LICENSE file. |
| 4 | |
| 5 | #include <gtest/gtest.h> |
| 6 | |
Han Shen | 07428b5 | 2012-12-18 10:29:20 -0800 | [diff] [blame] | 7 | #include "update_engine/certificate_checker.h" |
| 8 | #include "update_engine/certificate_checker_mock.h" |
Jay Srinivasan | 08fce04 | 2012-06-07 16:31:01 -0700 | [diff] [blame] | 9 | #include "update_engine/mock_system_state.h" |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 10 | #include "update_engine/update_attempter_mock.h" |
| 11 | #include "update_engine/update_check_scheduler.h" |
| 12 | |
David Zeuthen | 639aa36 | 2014-02-03 16:23:44 -0800 | [diff] [blame] | 13 | using base::Time; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 14 | using std::string; |
| 15 | using testing::_; |
| 16 | using testing::AllOf; |
Darin Petkov | 2a0e633 | 2010-09-24 14:43:41 -0700 | [diff] [blame] | 17 | using testing::Assign; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 18 | using testing::Ge; |
| 19 | using testing::Le; |
| 20 | using testing::MockFunction; |
| 21 | using testing::Return; |
David Zeuthen | 639aa36 | 2014-02-03 16:23:44 -0800 | [diff] [blame] | 22 | using testing::SetArgumentPointee; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 23 | |
| 24 | namespace chromeos_update_engine { |
| 25 | |
| 26 | namespace { |
| 27 | void FuzzRange(int interval, int fuzz, int* interval_min, int* interval_max) { |
| 28 | *interval_min = interval - fuzz / 2; |
| 29 | *interval_max = interval + fuzz - fuzz / 2; |
| 30 | } |
| 31 | } // namespace {} |
| 32 | |
| 33 | // Test a subclass rather than the main class directly so that we can mock out |
| 34 | // GLib and utils in tests. There're explicit unit test for the wrapper methods. |
| 35 | class UpdateCheckSchedulerUnderTest : public UpdateCheckScheduler { |
| 36 | public: |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 37 | UpdateCheckSchedulerUnderTest(UpdateAttempter* update_attempter, |
| 38 | MockSystemState* mock_system_state) |
Gilad Arnold | bf7919b | 2013-01-08 13:07:37 -0800 | [diff] [blame] | 39 | : UpdateCheckScheduler(update_attempter, mock_system_state), |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 40 | mock_system_state_(mock_system_state) {} |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 41 | |
| 42 | MOCK_METHOD2(GTimeoutAddSeconds, guint(guint seconds, GSourceFunc function)); |
| 43 | MOCK_METHOD0(IsBootDeviceRemovable, bool()); |
| 44 | MOCK_METHOD0(IsOfficialBuild, bool()); |
Jay Srinivasan | 08fce04 | 2012-06-07 16:31:01 -0700 | [diff] [blame] | 45 | |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 46 | MockSystemState* mock_system_state_; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 47 | }; |
| 48 | |
| 49 | class UpdateCheckSchedulerTest : public ::testing::Test { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 50 | public: |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 51 | UpdateCheckSchedulerTest() |
| 52 | : attempter_(&mock_system_state_, &dbus_), |
| 53 | scheduler_(&attempter_, &mock_system_state_) {} |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 54 | |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 55 | protected: |
| 56 | virtual void SetUp() { |
| 57 | test_ = this; |
| 58 | loop_ = NULL; |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 59 | EXPECT_EQ(&attempter_, scheduler_.update_attempter_); |
| 60 | EXPECT_FALSE(scheduler_.enabled_); |
| 61 | EXPECT_FALSE(scheduler_.scheduled_); |
| 62 | EXPECT_EQ(0, scheduler_.last_interval_); |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 63 | EXPECT_EQ(0, scheduler_.poll_interval_); |
Han Shen | 07428b5 | 2012-12-18 10:29:20 -0800 | [diff] [blame] | 64 | // Make sure singleton CertificateChecker has its members properly setup. |
| 65 | CertificateChecker::set_system_state(&mock_system_state_); |
| 66 | CertificateChecker::set_openssl_wrapper(&openssl_wrapper_); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 67 | } |
| 68 | |
| 69 | virtual void TearDown() { |
| 70 | test_ = NULL; |
| 71 | loop_ = NULL; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 72 | } |
| 73 | |
| 74 | static gboolean SourceCallback(gpointer data) { |
| 75 | g_main_loop_quit(test_->loop_); |
| 76 | // Forwards the call to the function mock so that expectations can be set. |
| 77 | return test_->source_callback_.Call(data); |
| 78 | } |
| 79 | |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 80 | MockSystemState mock_system_state_; |
Andrew de los Reyes | 000d895 | 2011-03-02 15:21:14 -0800 | [diff] [blame] | 81 | MockDbusGlib dbus_; |
Han Shen | 07428b5 | 2012-12-18 10:29:20 -0800 | [diff] [blame] | 82 | OpenSSLWrapperMock openssl_wrapper_; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 83 | UpdateAttempterMock attempter_; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 84 | UpdateCheckSchedulerUnderTest scheduler_; |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 85 | MockFunction<gboolean(gpointer data)> source_callback_; |
| 86 | GMainLoop* loop_; |
| 87 | static UpdateCheckSchedulerTest* test_; |
| 88 | }; |
| 89 | |
| 90 | UpdateCheckSchedulerTest* UpdateCheckSchedulerTest::test_ = NULL; |
| 91 | |
| 92 | TEST_F(UpdateCheckSchedulerTest, CanScheduleTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 93 | EXPECT_FALSE(scheduler_.CanSchedule()); |
| 94 | scheduler_.enabled_ = true; |
| 95 | EXPECT_TRUE(scheduler_.CanSchedule()); |
| 96 | scheduler_.scheduled_ = true; |
| 97 | EXPECT_FALSE(scheduler_.CanSchedule()); |
| 98 | scheduler_.enabled_ = false; |
| 99 | EXPECT_FALSE(scheduler_.CanSchedule()); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 100 | } |
| 101 | |
| 102 | TEST_F(UpdateCheckSchedulerTest, ComputeNextIntervalAndFuzzBackoffTest) { |
| 103 | int interval, fuzz; |
| 104 | attempter_.set_http_response_code(500); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 105 | int last_interval = UpdateCheckScheduler::kTimeoutPeriodicInterval + 50; |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 106 | scheduler_.last_interval_ = last_interval; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 107 | scheduler_.ComputeNextIntervalAndFuzz(0, &interval, &fuzz); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 108 | EXPECT_EQ(2 * last_interval, interval); |
| 109 | EXPECT_EQ(2 * last_interval, fuzz); |
| 110 | |
| 111 | attempter_.set_http_response_code(503); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 112 | scheduler_.last_interval_ = |
| 113 | UpdateCheckScheduler::kTimeoutMaxBackoffInterval / 2 + 1; |
| 114 | scheduler_.ComputeNextIntervalAndFuzz(0, &interval, &fuzz); |
| 115 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutMaxBackoffInterval, interval); |
| 116 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutMaxBackoffInterval, fuzz); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 117 | } |
| 118 | |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 119 | TEST_F(UpdateCheckSchedulerTest, ComputeNextIntervalAndFuzzPollTest) { |
| 120 | int interval, fuzz; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 121 | int poll_interval = UpdateCheckScheduler::kTimeoutPeriodicInterval + 50; |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 122 | scheduler_.set_poll_interval(poll_interval); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 123 | scheduler_.ComputeNextIntervalAndFuzz(0, &interval, &fuzz); |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 124 | EXPECT_EQ(poll_interval, interval); |
| 125 | EXPECT_EQ(poll_interval, fuzz); |
| 126 | |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 127 | scheduler_.set_poll_interval( |
| 128 | UpdateCheckScheduler::kTimeoutMaxBackoffInterval + 1); |
| 129 | scheduler_.ComputeNextIntervalAndFuzz(0, &interval, &fuzz); |
| 130 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutMaxBackoffInterval, interval); |
| 131 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutMaxBackoffInterval, fuzz); |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 132 | |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 133 | scheduler_.set_poll_interval( |
| 134 | UpdateCheckScheduler::kTimeoutPeriodicInterval - 1); |
| 135 | scheduler_.ComputeNextIntervalAndFuzz(0, &interval, &fuzz); |
| 136 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutPeriodicInterval, interval); |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 137 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutRegularFuzz, fuzz); |
| 138 | } |
| 139 | |
| 140 | TEST_F(UpdateCheckSchedulerTest, ComputeNextIntervalAndFuzzPriorityTest) { |
| 141 | int interval, fuzz; |
| 142 | attempter_.set_http_response_code(500); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 143 | scheduler_.last_interval_ = |
| 144 | UpdateCheckScheduler::kTimeoutPeriodicInterval + 50; |
| 145 | int poll_interval = UpdateCheckScheduler::kTimeoutPeriodicInterval + 100; |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 146 | scheduler_.set_poll_interval(poll_interval); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 147 | scheduler_.ComputeNextIntervalAndFuzz(0, &interval, &fuzz); |
Darin Petkov | 85ced13 | 2010-09-01 10:20:56 -0700 | [diff] [blame] | 148 | EXPECT_EQ(poll_interval, interval); |
| 149 | EXPECT_EQ(poll_interval, fuzz); |
| 150 | } |
| 151 | |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 152 | TEST_F(UpdateCheckSchedulerTest, ComputeNextIntervalAndFuzzTest) { |
| 153 | int interval, fuzz; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 154 | scheduler_.ComputeNextIntervalAndFuzz(0, &interval, &fuzz); |
| 155 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutPeriodicInterval, interval); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 156 | EXPECT_EQ(UpdateCheckScheduler::kTimeoutRegularFuzz, fuzz); |
| 157 | } |
| 158 | |
| 159 | TEST_F(UpdateCheckSchedulerTest, GTimeoutAddSecondsTest) { |
| 160 | loop_ = g_main_loop_new(g_main_context_default(), FALSE); |
| 161 | // Invokes the actual GLib wrapper method rather than the subclass mock. |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 162 | scheduler_.UpdateCheckScheduler::GTimeoutAddSeconds(0, SourceCallback); |
| 163 | EXPECT_CALL(source_callback_, Call(&scheduler_)).Times(1); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 164 | g_main_loop_run(loop_); |
| 165 | g_main_loop_unref(loop_); |
| 166 | } |
| 167 | |
| 168 | TEST_F(UpdateCheckSchedulerTest, IsBootDeviceRemovableTest) { |
| 169 | // Invokes the actual utils wrapper method rather than the subclass mock. |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 170 | EXPECT_FALSE(scheduler_.UpdateCheckScheduler::IsBootDeviceRemovable()); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 171 | } |
| 172 | |
| 173 | TEST_F(UpdateCheckSchedulerTest, IsOfficialBuildTest) { |
| 174 | // Invokes the actual utils wrapper method rather than the subclass mock. |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 175 | EXPECT_TRUE(scheduler_.UpdateCheckScheduler::IsOfficialBuild()); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 176 | } |
| 177 | |
| 178 | TEST_F(UpdateCheckSchedulerTest, RunBootDeviceRemovableTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 179 | scheduler_.enabled_ = true; |
| 180 | EXPECT_CALL(scheduler_, IsOfficialBuild()).Times(1).WillOnce(Return(true)); |
| 181 | EXPECT_CALL(scheduler_, IsBootDeviceRemovable()) |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 182 | .Times(1) |
| 183 | .WillOnce(Return(true)); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 184 | scheduler_.Run(); |
| 185 | EXPECT_FALSE(scheduler_.enabled_); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 186 | EXPECT_EQ(NULL, attempter_.update_check_scheduler()); |
| 187 | } |
| 188 | |
| 189 | TEST_F(UpdateCheckSchedulerTest, RunNonOfficialBuildTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 190 | scheduler_.enabled_ = true; |
| 191 | EXPECT_CALL(scheduler_, IsOfficialBuild()).Times(1).WillOnce(Return(false)); |
| 192 | scheduler_.Run(); |
| 193 | EXPECT_FALSE(scheduler_.enabled_); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 194 | EXPECT_EQ(NULL, attempter_.update_check_scheduler()); |
| 195 | } |
| 196 | |
| 197 | TEST_F(UpdateCheckSchedulerTest, RunTest) { |
| 198 | int interval_min, interval_max; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 199 | FuzzRange(UpdateCheckScheduler::kTimeoutInitialInterval, |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 200 | UpdateCheckScheduler::kTimeoutRegularFuzz, |
| 201 | &interval_min, |
| 202 | &interval_max); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 203 | EXPECT_CALL(scheduler_, IsOfficialBuild()).Times(1).WillOnce(Return(true)); |
| 204 | EXPECT_CALL(scheduler_, IsBootDeviceRemovable()) |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 205 | .Times(1) |
| 206 | .WillOnce(Return(false)); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 207 | EXPECT_CALL(scheduler_, |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 208 | GTimeoutAddSeconds(AllOf(Ge(interval_min), Le(interval_max)), |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 209 | scheduler_.StaticCheck)).Times(1); |
| 210 | scheduler_.Run(); |
| 211 | EXPECT_TRUE(scheduler_.enabled_); |
| 212 | EXPECT_EQ(&scheduler_, attempter_.update_check_scheduler()); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 213 | } |
| 214 | |
| 215 | TEST_F(UpdateCheckSchedulerTest, ScheduleCheckDisabledTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 216 | EXPECT_CALL(scheduler_, GTimeoutAddSeconds(_, _)).Times(0); |
| 217 | scheduler_.ScheduleCheck(250, 30); |
| 218 | EXPECT_EQ(0, scheduler_.last_interval_); |
| 219 | EXPECT_FALSE(scheduler_.scheduled_); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 220 | } |
| 221 | |
| 222 | TEST_F(UpdateCheckSchedulerTest, ScheduleCheckEnabledTest) { |
| 223 | int interval_min, interval_max; |
| 224 | FuzzRange(100, 10, &interval_min,&interval_max); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 225 | EXPECT_CALL(scheduler_, |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 226 | GTimeoutAddSeconds(AllOf(Ge(interval_min), Le(interval_max)), |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 227 | scheduler_.StaticCheck)).Times(1); |
| 228 | scheduler_.enabled_ = true; |
| 229 | scheduler_.ScheduleCheck(100, 10); |
| 230 | EXPECT_EQ(100, scheduler_.last_interval_); |
| 231 | EXPECT_TRUE(scheduler_.scheduled_); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 232 | } |
| 233 | |
| 234 | TEST_F(UpdateCheckSchedulerTest, ScheduleCheckNegativeIntervalTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 235 | EXPECT_CALL(scheduler_, GTimeoutAddSeconds(0, scheduler_.StaticCheck)) |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 236 | .Times(1); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 237 | scheduler_.enabled_ = true; |
| 238 | scheduler_.ScheduleCheck(-50, 20); |
| 239 | EXPECT_TRUE(scheduler_.scheduled_); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 240 | } |
| 241 | |
| 242 | TEST_F(UpdateCheckSchedulerTest, ScheduleNextCheckDisabledTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 243 | EXPECT_CALL(scheduler_, GTimeoutAddSeconds(_, _)).Times(0); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 244 | scheduler_.ScheduleNextCheck(false); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 245 | } |
| 246 | |
| 247 | TEST_F(UpdateCheckSchedulerTest, ScheduleNextCheckEnabledTest) { |
| 248 | int interval_min, interval_max; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 249 | FuzzRange(UpdateCheckScheduler::kTimeoutPeriodicInterval, |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 250 | UpdateCheckScheduler::kTimeoutRegularFuzz, |
| 251 | &interval_min, |
| 252 | &interval_max); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 253 | EXPECT_CALL(scheduler_, |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 254 | GTimeoutAddSeconds(AllOf(Ge(interval_min), Le(interval_max)), |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 255 | scheduler_.StaticCheck)).Times(1); |
| 256 | scheduler_.enabled_ = true; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 257 | scheduler_.ScheduleNextCheck(false); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 258 | } |
| 259 | |
| 260 | TEST_F(UpdateCheckSchedulerTest, SetUpdateStatusIdleDisabledTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 261 | EXPECT_CALL(scheduler_, GTimeoutAddSeconds(_, _)).Times(0); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 262 | scheduler_.SetUpdateStatus(UPDATE_STATUS_IDLE, kUpdateNoticeUnspecified); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 263 | } |
| 264 | |
| 265 | TEST_F(UpdateCheckSchedulerTest, SetUpdateStatusIdleEnabledTest) { |
| 266 | int interval_min, interval_max; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 267 | FuzzRange(UpdateCheckScheduler::kTimeoutPeriodicInterval, |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 268 | UpdateCheckScheduler::kTimeoutRegularFuzz, |
| 269 | &interval_min, |
| 270 | &interval_max); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 271 | EXPECT_CALL(scheduler_, |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 272 | GTimeoutAddSeconds(AllOf(Ge(interval_min), Le(interval_max)), |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 273 | scheduler_.StaticCheck)).Times(1); |
| 274 | scheduler_.enabled_ = true; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 275 | scheduler_.SetUpdateStatus(UPDATE_STATUS_IDLE, kUpdateNoticeUnspecified); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 276 | } |
| 277 | |
| 278 | TEST_F(UpdateCheckSchedulerTest, SetUpdateStatusNonIdleTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 279 | EXPECT_CALL(scheduler_, GTimeoutAddSeconds(_, _)).Times(0); |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 280 | scheduler_.SetUpdateStatus(UPDATE_STATUS_DOWNLOADING, |
| 281 | kUpdateNoticeUnspecified); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 282 | scheduler_.enabled_ = true; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 283 | scheduler_.SetUpdateStatus(UPDATE_STATUS_DOWNLOADING, |
| 284 | kUpdateNoticeUnspecified); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 285 | } |
| 286 | |
Darin Petkov | 2a0e633 | 2010-09-24 14:43:41 -0700 | [diff] [blame] | 287 | TEST_F(UpdateCheckSchedulerTest, StaticCheckOOBECompleteTest) { |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 288 | scheduler_.scheduled_ = true; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 289 | EXPECT_TRUE(scheduler_.mock_system_state_ != NULL); |
| 290 | EXPECT_CALL(*scheduler_.mock_system_state_, |
David Zeuthen | 639aa36 | 2014-02-03 16:23:44 -0800 | [diff] [blame] | 291 | IsOOBEComplete(_)).Times(1) |
| 292 | .WillOnce(DoAll(SetArgumentPointee<0>(Time::UnixEpoch()), |
| 293 | Return(true))); |
Gilad Arnold | b92f0df | 2013-01-10 16:32:45 -0800 | [diff] [blame] | 294 | EXPECT_CALL(attempter_, Update("", "", false, false, false)) |
Darin Petkov | 2a0e633 | 2010-09-24 14:43:41 -0700 | [diff] [blame] | 295 | .Times(1) |
| 296 | .WillOnce(Assign(&scheduler_.scheduled_, true)); |
| 297 | scheduler_.enabled_ = true; |
| 298 | EXPECT_CALL(scheduler_, GTimeoutAddSeconds(_, _)).Times(0); |
| 299 | UpdateCheckSchedulerUnderTest::StaticCheck(&scheduler_); |
| 300 | } |
| 301 | |
| 302 | TEST_F(UpdateCheckSchedulerTest, StaticCheckOOBENotCompleteTest) { |
| 303 | scheduler_.scheduled_ = true; |
Jay Srinivasan | 6f6ea00 | 2012-12-14 11:26:28 -0800 | [diff] [blame] | 304 | EXPECT_CALL(*scheduler_.mock_system_state_, |
David Zeuthen | 639aa36 | 2014-02-03 16:23:44 -0800 | [diff] [blame] | 305 | IsOOBEComplete(_)).Times(1) |
| 306 | .WillOnce(DoAll(SetArgumentPointee<0>(Time::UnixEpoch()), |
| 307 | Return(false))); |
Gilad Arnold | b92f0df | 2013-01-10 16:32:45 -0800 | [diff] [blame] | 308 | EXPECT_CALL(attempter_, Update("", "", _, _, _)).Times(0); |
Darin Petkov | 2a0e633 | 2010-09-24 14:43:41 -0700 | [diff] [blame] | 309 | int interval_min, interval_max; |
Gilad Arnold | 1ebd813 | 2012-03-05 10:19:29 -0800 | [diff] [blame] | 310 | FuzzRange(UpdateCheckScheduler::kTimeoutInitialInterval, |
Darin Petkov | 2a0e633 | 2010-09-24 14:43:41 -0700 | [diff] [blame] | 311 | UpdateCheckScheduler::kTimeoutRegularFuzz, |
| 312 | &interval_min, |
| 313 | &interval_max); |
| 314 | scheduler_.enabled_ = true; |
| 315 | EXPECT_CALL(scheduler_, |
| 316 | GTimeoutAddSeconds(AllOf(Ge(interval_min), Le(interval_max)), |
| 317 | scheduler_.StaticCheck)).Times(1); |
Darin Petkov | f42cc1c | 2010-09-01 09:03:02 -0700 | [diff] [blame] | 318 | UpdateCheckSchedulerUnderTest::StaticCheck(&scheduler_); |
Darin Petkov | 1023a60 | 2010-08-30 13:47:51 -0700 | [diff] [blame] | 319 | } |
| 320 | |
| 321 | } // namespace chromeos_update_engine |