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