| Alex Deymo | aea4c1c | 2015-08-19 20:24:43 -0700 | [diff] [blame] | 1 | // | 
|  | 2 | // Copyright (C) 2014 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 | // | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 16 |  | 
| Alex Deymo | aab50e3 | 2014-11-10 19:55:35 -0800 | [diff] [blame] | 17 | #include "update_engine/update_manager/update_manager.h" | 
|  | 18 |  | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 19 | #include <unistd.h> | 
|  | 20 |  | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 21 | #include <algorithm> | 
| Ben Chan | 02f7c1d | 2014-10-18 15:18:02 -0700 | [diff] [blame] | 22 | #include <memory> | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 23 | #include <string> | 
| Gilad Arnold | dc4bb26 | 2014-07-23 10:45:19 -0700 | [diff] [blame] | 24 | #include <tuple> | 
| Alex Deymo | 94c0616 | 2014-03-21 20:34:46 -0700 | [diff] [blame] | 25 | #include <utility> | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 26 | #include <vector> | 
|  | 27 |  | 
|  | 28 | #include <base/bind.h> | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 29 | #include <base/test/simple_test_clock.h> | 
| Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 30 | #include <base/time/time.h> | 
| Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 31 | #include <brillo/message_loops/fake_message_loop.h> | 
|  | 32 | #include <brillo/message_loops/message_loop.h> | 
|  | 33 | #include <brillo/message_loops/message_loop_utils.h> | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 34 | #include <gmock/gmock.h> | 
| Alex Deymo | 1f01291 | 2014-04-24 19:08:04 -0700 | [diff] [blame] | 35 | #include <gtest/gtest.h> | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 36 |  | 
| Alex Deymo | 39910dc | 2015-11-09 17:04:30 -0800 | [diff] [blame] | 37 | #include "update_engine/common/fake_clock.h" | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 38 | #include "update_engine/update_manager/default_policy.h" | 
|  | 39 | #include "update_engine/update_manager/fake_state.h" | 
|  | 40 | #include "update_engine/update_manager/mock_policy.h" | 
|  | 41 | #include "update_engine/update_manager/umtest_utils.h" | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 42 |  | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 43 | using base::Bind; | 
|  | 44 | using base::Callback; | 
| Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 45 | using base::Time; | 
|  | 46 | using base::TimeDelta; | 
| Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 47 | using brillo::MessageLoop; | 
|  | 48 | using brillo::MessageLoopRunMaxIterations; | 
| Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 49 | using chromeos_update_engine::ErrorCode; | 
| Alex Deymo | 41a75a7 | 2014-04-15 15:36:22 -0700 | [diff] [blame] | 50 | using chromeos_update_engine::FakeClock; | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 51 | using std::pair; | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 52 | using std::string; | 
| Gilad Arnold | dc4bb26 | 2014-07-23 10:45:19 -0700 | [diff] [blame] | 53 | using std::tuple; | 
| Ben Chan | 02f7c1d | 2014-10-18 15:18:02 -0700 | [diff] [blame] | 54 | using std::unique_ptr; | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 55 | using std::vector; | 
| Gilad Arnold | 5ef9c48 | 2014-03-03 13:51:02 -0800 | [diff] [blame] | 56 |  | 
| Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 57 | namespace { | 
|  | 58 |  | 
|  | 59 | // Generates a fixed timestamp for use in faking the current time. | 
|  | 60 | Time FixedTime() { | 
|  | 61 | Time::Exploded now_exp; | 
|  | 62 | now_exp.year = 2014; | 
|  | 63 | now_exp.month = 3; | 
|  | 64 | now_exp.day_of_week = 2; | 
|  | 65 | now_exp.day_of_month = 18; | 
|  | 66 | now_exp.hour = 8; | 
|  | 67 | now_exp.minute = 5; | 
|  | 68 | now_exp.second = 33; | 
|  | 69 | now_exp.millisecond = 675; | 
|  | 70 | return Time::FromLocalExploded(now_exp); | 
|  | 71 | } | 
|  | 72 |  | 
|  | 73 | }  // namespace | 
|  | 74 |  | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 75 | namespace chromeos_update_manager { | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 76 |  | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 77 | class UmUpdateManagerTest : public ::testing::Test { | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 78 | protected: | 
| Alex Vakulenko | 157fe30 | 2014-08-11 15:59:58 -0700 | [diff] [blame] | 79 | void SetUp() override { | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 80 | loop_.SetAsCurrent(); | 
| Alex Deymo | 42c30c3 | 2014-04-24 18:41:18 -0700 | [diff] [blame] | 81 | fake_state_ = new FakeState(); | 
| Gilad Arnold | b227199 | 2014-06-19 12:35:24 -0700 | [diff] [blame] | 82 | umut_.reset(new UpdateManager(&fake_clock_, TimeDelta::FromSeconds(5), | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 83 | TimeDelta::FromSeconds(1), fake_state_)); | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 84 | } | 
|  | 85 |  | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 86 | void TearDown() override { | 
|  | 87 | EXPECT_FALSE(loop_.PendingTasks()); | 
|  | 88 | } | 
|  | 89 |  | 
|  | 90 | base::SimpleTestClock test_clock_; | 
| Alex Vakulenko | 3f39d5c | 2015-10-13 09:27:13 -0700 | [diff] [blame] | 91 | brillo::FakeMessageLoop loop_{&test_clock_}; | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 92 | FakeState* fake_state_;  // Owned by the umut_. | 
| Alex Deymo | 41a75a7 | 2014-04-15 15:36:22 -0700 | [diff] [blame] | 93 | FakeClock fake_clock_; | 
| Ben Chan | 02f7c1d | 2014-10-18 15:18:02 -0700 | [diff] [blame] | 94 | unique_ptr<UpdateManager> umut_; | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 95 | }; | 
|  | 96 |  | 
|  | 97 | // The FailingPolicy implements a single method and make it always fail. This | 
|  | 98 | // class extends the DefaultPolicy class to allow extensions of the Policy | 
|  | 99 | // class without extending nor changing this test. | 
|  | 100 | class FailingPolicy : public DefaultPolicy { | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 101 | public: | 
|  | 102 | explicit FailingPolicy(int* num_called_p) : num_called_p_(num_called_p) {} | 
|  | 103 | FailingPolicy() : FailingPolicy(nullptr) {} | 
| Alex Vakulenko | 157fe30 | 2014-08-11 15:59:58 -0700 | [diff] [blame] | 104 | EvalStatus UpdateCheckAllowed(EvaluationContext* ec, State* state, | 
|  | 105 | string* error, | 
|  | 106 | UpdateCheckParams* result) const override { | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 107 | if (num_called_p_) | 
|  | 108 | (*num_called_p_)++; | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 109 | *error = "FailingPolicy failed."; | 
| Alex Deymo | e636c3c | 2014-03-11 19:02:08 -0700 | [diff] [blame] | 110 | return EvalStatus::kFailed; | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 111 | } | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 112 |  | 
|  | 113 | protected: | 
| Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 114 | string PolicyName() const override { return "FailingPolicy"; } | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 115 |  | 
|  | 116 | private: | 
|  | 117 | int* num_called_p_; | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 118 | }; | 
|  | 119 |  | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 120 | // The LazyPolicy always returns EvalStatus::kAskMeAgainLater. | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 121 | class LazyPolicy : public DefaultPolicy { | 
| Alex Vakulenko | 157fe30 | 2014-08-11 15:59:58 -0700 | [diff] [blame] | 122 | EvalStatus UpdateCheckAllowed(EvaluationContext* ec, State* state, | 
|  | 123 | string* error, | 
|  | 124 | UpdateCheckParams* result) const override { | 
| Alex Deymo | e636c3c | 2014-03-11 19:02:08 -0700 | [diff] [blame] | 125 | return EvalStatus::kAskMeAgainLater; | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 126 | } | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 127 |  | 
|  | 128 | protected: | 
| Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 129 | string PolicyName() const override { return "LazyPolicy"; } | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 130 | }; | 
|  | 131 |  | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 132 | // A policy that sleeps for a predetermined amount of time, then checks for a | 
|  | 133 | // wallclock-based time threshold (if given) and returns | 
|  | 134 | // EvalStatus::kAskMeAgainLater if not passed; otherwise, returns | 
|  | 135 | // EvalStatus::kSucceeded. Increments a counter every time it is being queried, | 
|  | 136 | // if a pointer to it is provided. | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 137 | class DelayPolicy : public DefaultPolicy { | 
|  | 138 | public: | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 139 | DelayPolicy(int sleep_secs, Time time_threshold, int* num_called_p) | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 140 | : sleep_secs_(sleep_secs), time_threshold_(time_threshold), | 
|  | 141 | num_called_p_(num_called_p) {} | 
| Alex Vakulenko | 157fe30 | 2014-08-11 15:59:58 -0700 | [diff] [blame] | 142 | EvalStatus UpdateCheckAllowed(EvaluationContext* ec, State* state, | 
|  | 143 | string* error, | 
|  | 144 | UpdateCheckParams* result) const override { | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 145 | if (num_called_p_) | 
|  | 146 | (*num_called_p_)++; | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 147 |  | 
|  | 148 | // Sleep for a predetermined amount of time. | 
|  | 149 | if (sleep_secs_ > 0) | 
|  | 150 | sleep(sleep_secs_); | 
|  | 151 |  | 
|  | 152 | // Check for a time threshold. This can be used to ensure that the policy | 
|  | 153 | // has some non-constant dependency. | 
|  | 154 | if (time_threshold_ < Time::Max() && | 
|  | 155 | ec->IsWallclockTimeGreaterThan(time_threshold_)) | 
|  | 156 | return EvalStatus::kSucceeded; | 
|  | 157 |  | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 158 | return EvalStatus::kAskMeAgainLater; | 
|  | 159 | } | 
|  | 160 |  | 
|  | 161 | protected: | 
| Alex Deymo | f329b93 | 2014-10-30 01:37:48 -0700 | [diff] [blame] | 162 | string PolicyName() const override { return "DelayPolicy"; } | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 163 |  | 
|  | 164 | private: | 
|  | 165 | int sleep_secs_; | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 166 | Time time_threshold_; | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 167 | int* num_called_p_; | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 168 | }; | 
|  | 169 |  | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 170 | // AccumulateCallsCallback() adds to the passed |acc| accumulator vector pairs | 
|  | 171 | // of EvalStatus and T instances. This allows to create a callback that keeps | 
|  | 172 | // track of when it is called and the arguments passed to it, to be used with | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 173 | // the UpdateManager::AsyncPolicyRequest(). | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 174 | template<typename T> | 
|  | 175 | static void AccumulateCallsCallback(vector<pair<EvalStatus, T>>* acc, | 
|  | 176 | EvalStatus status, const T& result) { | 
|  | 177 | acc->push_back(std::make_pair(status, result)); | 
|  | 178 | } | 
|  | 179 |  | 
| Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 180 | // Tests that policy requests are completed successfully. It is important that | 
|  | 181 | // this tests cover all policy requests as defined in Policy. | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 182 | TEST_F(UmUpdateManagerTest, PolicyRequestCallUpdateCheckAllowed) { | 
| Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 183 | UpdateCheckParams result; | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 184 | EXPECT_EQ(EvalStatus::kSucceeded, umut_->PolicyRequest( | 
| Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 185 | &Policy::UpdateCheckAllowed, &result)); | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 186 | } | 
|  | 187 |  | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 188 | TEST_F(UmUpdateManagerTest, PolicyRequestCallUpdateCanStart) { | 
| Gilad Arnold | dc4bb26 | 2014-07-23 10:45:19 -0700 | [diff] [blame] | 189 | UpdateState update_state = UpdateState(); | 
|  | 190 | update_state.is_interactive = true; | 
|  | 191 | update_state.is_delta_payload = false; | 
|  | 192 | update_state.first_seen = FixedTime(); | 
|  | 193 | update_state.num_checks = 1; | 
|  | 194 | update_state.num_failures = 0; | 
|  | 195 | update_state.failures_last_updated = Time(); | 
|  | 196 | update_state.download_urls = vector<string>{"http://fake/url/"}; | 
|  | 197 | update_state.download_errors_max = 10; | 
| Gilad Arnold | 78ecbfc | 2014-10-22 14:38:25 -0700 | [diff] [blame] | 198 | update_state.p2p_downloading_disabled = false; | 
|  | 199 | update_state.p2p_sharing_disabled = false; | 
| Gilad Arnold | 349ac83 | 2014-10-06 14:20:28 -0700 | [diff] [blame] | 200 | update_state.p2p_num_attempts = 0; | 
|  | 201 | update_state.p2p_first_attempted = Time(); | 
| Gilad Arnold | dc4bb26 | 2014-07-23 10:45:19 -0700 | [diff] [blame] | 202 | update_state.last_download_url_idx = -1; | 
|  | 203 | update_state.last_download_url_num_errors = 0; | 
|  | 204 | update_state.download_errors = vector<tuple<int, ErrorCode, Time>>(); | 
|  | 205 | update_state.backoff_expiry = Time(); | 
|  | 206 | update_state.is_backoff_disabled = false; | 
|  | 207 | update_state.scatter_wait_period = TimeDelta::FromSeconds(15); | 
|  | 208 | update_state.scatter_check_threshold = 4; | 
|  | 209 | update_state.scatter_wait_period_max = TimeDelta::FromSeconds(60); | 
|  | 210 | update_state.scatter_check_threshold_min = 2; | 
|  | 211 | update_state.scatter_check_threshold_max = 8; | 
|  | 212 |  | 
| Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 213 | UpdateDownloadParams result; | 
| Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 214 | EXPECT_EQ(EvalStatus::kSucceeded, | 
| Gilad Arnold | dc4bb26 | 2014-07-23 10:45:19 -0700 | [diff] [blame] | 215 | umut_->PolicyRequest(&Policy::UpdateCanStart, &result, | 
| Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 216 | update_state)); | 
|  | 217 | } | 
|  | 218 |  | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 219 | TEST_F(UmUpdateManagerTest, PolicyRequestCallsDefaultOnError) { | 
|  | 220 | umut_->set_policy(new FailingPolicy()); | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 221 |  | 
|  | 222 | // Tests that the DefaultPolicy instance is called when the method fails, | 
|  | 223 | // which will set this as true. | 
| Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 224 | UpdateCheckParams result; | 
|  | 225 | result.updates_enabled = false; | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 226 | EvalStatus status = umut_->PolicyRequest( | 
| Alex Deymo | 680d022 | 2014-04-24 21:00:08 -0700 | [diff] [blame] | 227 | &Policy::UpdateCheckAllowed, &result); | 
| Gilad Arnold | af2f6ae | 2014-04-28 14:14:52 -0700 | [diff] [blame] | 228 | EXPECT_EQ(EvalStatus::kSucceeded, status); | 
| Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 229 | EXPECT_TRUE(result.updates_enabled); | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 230 | } | 
|  | 231 |  | 
| Gilad Arnold | 897b5e5 | 2014-05-21 09:37:18 -0700 | [diff] [blame] | 232 | // This test only applies to debug builds where DCHECK is enabled. | 
|  | 233 | #if DCHECK_IS_ON | 
|  | 234 | TEST_F(UmUpdateManagerTest, PolicyRequestDoesntBlockDeathTest) { | 
|  | 235 | // The update manager should die (DCHECK) if a policy called synchronously | 
|  | 236 | // returns a kAskMeAgainLater value. | 
| Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 237 | UpdateCheckParams result; | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 238 | umut_->set_policy(new LazyPolicy()); | 
| Gilad Arnold | 897b5e5 | 2014-05-21 09:37:18 -0700 | [diff] [blame] | 239 | EXPECT_DEATH(umut_->PolicyRequest(&Policy::UpdateCheckAllowed, &result), ""); | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 240 | } | 
| Gilad Arnold | 897b5e5 | 2014-05-21 09:37:18 -0700 | [diff] [blame] | 241 | #endif  // DCHECK_IS_ON | 
| Alex Deymo | c705cc8 | 2014-02-19 11:15:00 -0800 | [diff] [blame] | 242 |  | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 243 | TEST_F(UmUpdateManagerTest, AsyncPolicyRequestDelaysEvaluation) { | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 244 | // To avoid differences in code execution order between an AsyncPolicyRequest | 
|  | 245 | // call on a policy that returns AskMeAgainLater the first time and one that | 
|  | 246 | // succeeds the first time, we ensure that the passed callback is called from | 
|  | 247 | // the main loop in both cases even when we could evaluate it right now. | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 248 | umut_->set_policy(new FailingPolicy()); | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 249 |  | 
| Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 250 | vector<pair<EvalStatus, UpdateCheckParams>> calls; | 
| Gilad Arnold | 44dc3bf | 2014-07-18 23:39:38 -0700 | [diff] [blame] | 251 | Callback<void(EvalStatus, const UpdateCheckParams&)> callback = Bind( | 
|  | 252 | AccumulateCallsCallback<UpdateCheckParams>, &calls); | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 253 |  | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 254 | umut_->AsyncPolicyRequest(callback, &Policy::UpdateCheckAllowed); | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 255 | // The callback should wait until we run the main loop for it to be executed. | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 256 | EXPECT_EQ(0U, calls.size()); | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 257 | MessageLoopRunMaxIterations(MessageLoop::current(), 100); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 258 | EXPECT_EQ(1U, calls.size()); | 
| Alex Deymo | 7b948f0 | 2014-03-10 17:01:10 -0700 | [diff] [blame] | 259 | } | 
|  | 260 |  | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 261 | TEST_F(UmUpdateManagerTest, AsyncPolicyRequestTimeoutDoesNotFire) { | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 262 | // Set up an async policy call to return immediately, then wait a little and | 
|  | 263 | // ensure that the timeout event does not fire. | 
|  | 264 | int num_called = 0; | 
|  | 265 | umut_->set_policy(new FailingPolicy(&num_called)); | 
|  | 266 |  | 
|  | 267 | vector<pair<EvalStatus, UpdateCheckParams>> calls; | 
|  | 268 | Callback<void(EvalStatus, const UpdateCheckParams&)> callback = | 
|  | 269 | Bind(AccumulateCallsCallback<UpdateCheckParams>, &calls); | 
|  | 270 |  | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 271 | umut_->AsyncPolicyRequest(callback, &Policy::UpdateCheckAllowed); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 272 | // Run the main loop, ensure that policy was attempted once before deferring | 
|  | 273 | // to the default. | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 274 | MessageLoopRunMaxIterations(MessageLoop::current(), 100); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 275 | EXPECT_EQ(1, num_called); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 276 | ASSERT_EQ(1U, calls.size()); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 277 | EXPECT_EQ(EvalStatus::kSucceeded, calls[0].first); | 
|  | 278 | // Wait for the timeout to expire, run the main loop again, ensure that | 
|  | 279 | // nothing happened. | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 280 | test_clock_.Advance(TimeDelta::FromSeconds(2)); | 
|  | 281 | MessageLoopRunMaxIterations(MessageLoop::current(), 10); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 282 | EXPECT_EQ(1, num_called); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 283 | EXPECT_EQ(1U, calls.size()); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 284 | } | 
|  | 285 |  | 
|  | 286 | TEST_F(UmUpdateManagerTest, AsyncPolicyRequestTimesOut) { | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 287 | // Set up an async policy call to exceed its expiration timeout, make sure | 
|  | 288 | // that the default policy was not used (no callback) and that evaluation is | 
|  | 289 | // reattempted. | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 290 | int num_called = 0; | 
|  | 291 | umut_->set_policy(new DelayPolicy( | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 292 | 0, fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds(3), | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 293 | &num_called)); | 
|  | 294 |  | 
|  | 295 | vector<pair<EvalStatus, UpdateCheckParams>> calls; | 
|  | 296 | Callback<void(EvalStatus, const UpdateCheckParams&)> callback = | 
|  | 297 | Bind(AccumulateCallsCallback<UpdateCheckParams>, &calls); | 
|  | 298 |  | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 299 | umut_->AsyncPolicyRequest(callback, &Policy::UpdateCheckAllowed); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 300 | // Run the main loop, ensure that policy was attempted once but the callback | 
|  | 301 | // was not invoked. | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 302 | MessageLoopRunMaxIterations(MessageLoop::current(), 100); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 303 | EXPECT_EQ(1, num_called); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 304 | EXPECT_EQ(0U, calls.size()); | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 305 | // Wait for the expiration timeout to expire, run the main loop again, | 
|  | 306 | // ensure that reevaluation occurred but callback was not invoked (i.e. | 
|  | 307 | // default policy was not consulted). | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 308 | test_clock_.Advance(TimeDelta::FromSeconds(2)); | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 309 | fake_clock_.SetWallclockTime(fake_clock_.GetWallclockTime() + | 
|  | 310 | TimeDelta::FromSeconds(2)); | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 311 | MessageLoopRunMaxIterations(MessageLoop::current(), 10); | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 312 | EXPECT_EQ(2, num_called); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 313 | EXPECT_EQ(0U, calls.size()); | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 314 | // Wait for reevaluation due to delay to happen, ensure that it occurs and | 
|  | 315 | // that the callback is invoked. | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 316 | test_clock_.Advance(TimeDelta::FromSeconds(2)); | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 317 | fake_clock_.SetWallclockTime(fake_clock_.GetWallclockTime() + | 
|  | 318 | TimeDelta::FromSeconds(2)); | 
| Alex Deymo | 509dd53 | 2015-06-10 14:11:05 -0700 | [diff] [blame] | 319 | MessageLoopRunMaxIterations(MessageLoop::current(), 10); | 
| Gilad Arnold | fd45a73 | 2014-08-07 15:53:46 -0700 | [diff] [blame] | 320 | EXPECT_EQ(3, num_called); | 
| Alex Deymo | 80f70ff | 2016-02-10 16:08:11 -0800 | [diff] [blame] | 321 | ASSERT_EQ(1U, calls.size()); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 322 | EXPECT_EQ(EvalStatus::kSucceeded, calls[0].first); | 
| Gilad Arnold | f9f85d6 | 2014-06-19 18:07:01 -0700 | [diff] [blame] | 323 | } | 
|  | 324 |  | 
| Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 325 | }  // namespace chromeos_update_manager |