Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 1 | // Copyright (c) 2014 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 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 5 | #include "update_engine/update_manager/chromeos_policy.h" |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 6 | |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 7 | #include <set> |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 8 | #include <string> |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 9 | #include <vector> |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 10 | |
| 11 | #include <base/time/time.h> |
| 12 | #include <gtest/gtest.h> |
| 13 | |
| 14 | #include "update_engine/fake_clock.h" |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 15 | #include "update_engine/update_manager/evaluation_context.h" |
| 16 | #include "update_engine/update_manager/fake_state.h" |
| 17 | #include "update_engine/update_manager/umtest_utils.h" |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 18 | |
| 19 | using base::Time; |
| 20 | using base::TimeDelta; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 21 | using chromeos_update_engine::ErrorCode; |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 22 | using chromeos_update_engine::FakeClock; |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 23 | using std::set; |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 24 | using std::string; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 25 | using std::vector; |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 26 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 27 | namespace chromeos_update_manager { |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 28 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 29 | class UmChromeOSPolicyTest : public ::testing::Test { |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 30 | protected: |
| 31 | virtual void SetUp() { |
| 32 | SetUpDefaultClock(); |
Gilad Arnold | b227199 | 2014-06-19 12:35:24 -0700 | [diff] [blame] | 33 | eval_ctx_ = new EvaluationContext(&fake_clock_, TimeDelta::FromSeconds(5)); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 34 | SetUpDefaultState(); |
| 35 | SetUpDefaultDevicePolicy(); |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 36 | } |
| 37 | |
| 38 | // Sets the clock to fixed values. |
| 39 | void SetUpDefaultClock() { |
| 40 | fake_clock_.SetMonotonicTime(Time::FromInternalValue(12345678L)); |
| 41 | fake_clock_.SetWallclockTime(Time::FromInternalValue(12345678901234L)); |
| 42 | } |
| 43 | |
| 44 | void SetUpDefaultState() { |
| 45 | fake_state_.updater_provider()->var_updater_started_time()->reset( |
| 46 | new Time(fake_clock_.GetWallclockTime())); |
| 47 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 48 | new Time(fake_clock_.GetWallclockTime())); |
| 49 | fake_state_.updater_provider()->var_consecutive_failed_update_checks()-> |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 50 | reset(new unsigned int(0)); // NOLINT(readability/casting) |
Gilad Arnold | a0258a5 | 2014-07-10 16:21:19 -0700 | [diff] [blame] | 51 | fake_state_.updater_provider()->var_server_dictated_poll_interval()-> |
| 52 | reset(new unsigned int(0)); // NOLINT(readability/casting) |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 53 | |
| 54 | fake_state_.random_provider()->var_seed()->reset( |
| 55 | new uint64_t(4)); // chosen by fair dice roll. |
| 56 | // guaranteed to be random. |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 57 | |
| 58 | // No device policy loaded by default. |
| 59 | fake_state_.device_policy_provider()->var_device_policy_is_loaded()->reset( |
| 60 | new bool(false)); |
| 61 | |
Gilad Arnold | a1eabcd | 2014-07-09 15:42:40 -0700 | [diff] [blame] | 62 | // OOBE is enabled by default. |
| 63 | fake_state_.config_provider()->var_is_oobe_enabled()->reset( |
| 64 | new bool(true)); |
| 65 | |
Gilad Arnold | 76a11f6 | 2014-05-20 09:02:12 -0700 | [diff] [blame] | 66 | // For the purpose of the tests, this is an official build and OOBE was |
| 67 | // completed. |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 68 | fake_state_.system_provider()->var_is_official_build()->reset( |
| 69 | new bool(true)); |
Gilad Arnold | 76a11f6 | 2014-05-20 09:02:12 -0700 | [diff] [blame] | 70 | fake_state_.system_provider()->var_is_oobe_complete()->reset( |
| 71 | new bool(true)); |
Gilad Arnold | bfc44f7 | 2014-07-09 14:41:39 -0700 | [diff] [blame] | 72 | fake_state_.system_provider()->var_is_boot_device_removable()->reset( |
| 73 | new bool(false)); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 74 | |
| 75 | // Connection is wifi, untethered. |
| 76 | fake_state_.shill_provider()->var_conn_type()-> |
| 77 | reset(new ConnectionType(ConnectionType::kWifi)); |
| 78 | fake_state_.shill_provider()->var_conn_tethering()-> |
| 79 | reset(new ConnectionTethering(ConnectionTethering::kNotDetected)); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 80 | } |
| 81 | |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 82 | // Sets up a default device policy that does not impose any restrictions |
| 83 | // (HTTP) nor enables any features (P2P). |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 84 | void SetUpDefaultDevicePolicy() { |
| 85 | fake_state_.device_policy_provider()->var_device_policy_is_loaded()->reset( |
| 86 | new bool(true)); |
| 87 | fake_state_.device_policy_provider()->var_update_disabled()->reset( |
| 88 | new bool(false)); |
| 89 | fake_state_.device_policy_provider()-> |
| 90 | var_allowed_connection_types_for_update()->reset(nullptr); |
| 91 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 92 | new TimeDelta()); |
| 93 | fake_state_.device_policy_provider()->var_http_downloads_enabled()->reset( |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 94 | new bool(true)); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 95 | fake_state_.device_policy_provider()->var_au_p2p_enabled()->reset( |
| 96 | new bool(false)); |
| 97 | fake_state_.device_policy_provider()->var_release_channel_delegated()-> |
| 98 | reset(new bool(true)); |
| 99 | } |
| 100 | |
| 101 | // Configures the UpdateCheckAllowed policy to return a desired value by |
| 102 | // faking the current wall clock time as needed. Restores the default state. |
| 103 | // This is used when testing policies that depend on this one. |
| 104 | void SetUpdateCheckAllowed(bool allow_check) { |
| 105 | Time next_update_check; |
| 106 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 107 | &ChromeOSPolicy::NextUpdateCheckTime, |
| 108 | &next_update_check); |
| 109 | SetUpDefaultState(); |
| 110 | SetUpDefaultDevicePolicy(); |
| 111 | Time curr_time = next_update_check; |
| 112 | if (allow_check) |
| 113 | curr_time += TimeDelta::FromSeconds(1); |
| 114 | else |
| 115 | curr_time -= TimeDelta::FromSeconds(1); |
| 116 | fake_clock_.SetWallclockTime(curr_time); |
| 117 | } |
| 118 | |
| 119 | // Returns a default UpdateState structure: first seen time is calculated |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 120 | // backward from the current wall clock time, update was seen just once; |
| 121 | // there's a single HTTP download URL with a maximum of 10 allowed failures; |
| 122 | // there is no scattering wait period and the max allowed is 7 days, there is |
| 123 | // no check threshold and none is allowed. |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 124 | UpdateState GetDefaultUpdateState(TimeDelta update_first_seen_period) { |
| 125 | UpdateState update_state = { |
| 126 | fake_clock_.GetWallclockTime() - update_first_seen_period, 1, |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 127 | vector<string>(1, "http://fake/url/"), 10, 0, 0, vector<ErrorCode>(), |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 128 | TimeDelta(), TimeDelta::FromDays(7), 0, 0, 0 |
| 129 | }; |
| 130 | return update_state; |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 131 | } |
| 132 | |
| 133 | // Runs the passed |policy_method| policy and expects it to return the |
| 134 | // |expected| return value. |
| 135 | template<typename T, typename R, typename... Args> |
| 136 | void ExpectPolicyStatus( |
| 137 | EvalStatus expected, |
| 138 | T policy_method, |
| 139 | R* result, Args... args) { |
| 140 | string error = "<None>"; |
| 141 | eval_ctx_->ResetEvaluation(); |
| 142 | EXPECT_EQ(expected, |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 143 | (policy_.*policy_method)(eval_ctx_, &fake_state_, &error, result, |
| 144 | args...)) |
Alex Deymo | c850b4f | 2014-05-19 11:06:41 -0700 | [diff] [blame] | 145 | << "Returned error: " << error |
| 146 | << "\nEvaluation context: " << eval_ctx_->DumpContext(); |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 147 | } |
| 148 | |
| 149 | FakeClock fake_clock_; |
| 150 | FakeState fake_state_; |
| 151 | scoped_refptr<EvaluationContext> eval_ctx_; |
| 152 | ChromeOSPolicy policy_; // ChromeOSPolicy under test. |
| 153 | }; |
| 154 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 155 | TEST_F(UmChromeOSPolicyTest, FirstCheckIsAtMostInitialIntervalAfterStart) { |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 156 | Time next_update_check; |
| 157 | |
Gilad Arnold | 38b1402 | 2014-07-09 12:45:56 -0700 | [diff] [blame] | 158 | // Set the last update time so it'll appear as if this is a first update check |
| 159 | // in the lifetime of the current updater. |
| 160 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 161 | new Time(fake_clock_.GetWallclockTime() - TimeDelta::FromMinutes(10))); |
| 162 | |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 163 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 164 | &ChromeOSPolicy::NextUpdateCheckTime, &next_update_check); |
| 165 | |
| 166 | EXPECT_LE(fake_clock_.GetWallclockTime(), next_update_check); |
Gilad Arnold | 38b1402 | 2014-07-09 12:45:56 -0700 | [diff] [blame] | 167 | EXPECT_GE( |
| 168 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 169 | ChromeOSPolicy::kTimeoutInitialInterval + |
| 170 | ChromeOSPolicy::kTimeoutRegularFuzz / 2), |
| 171 | next_update_check); |
| 172 | } |
| 173 | |
| 174 | TEST_F(UmChromeOSPolicyTest, RecurringCheckBaseIntervalAndFuzz) { |
| 175 | // Ensure that we're using the correct interval (kPeriodicInterval) and fuzz |
| 176 | // (kTimeoutRegularFuzz) as base values for period updates. |
| 177 | Time next_update_check; |
| 178 | |
| 179 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 180 | &ChromeOSPolicy::NextUpdateCheckTime, &next_update_check); |
| 181 | |
| 182 | EXPECT_LE( |
| 183 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 184 | ChromeOSPolicy::kTimeoutPeriodicInterval - |
| 185 | ChromeOSPolicy::kTimeoutRegularFuzz / 2), |
| 186 | next_update_check); |
| 187 | EXPECT_GE( |
| 188 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 189 | ChromeOSPolicy::kTimeoutPeriodicInterval + |
| 190 | ChromeOSPolicy::kTimeoutRegularFuzz / 2), |
| 191 | next_update_check); |
| 192 | } |
| 193 | |
| 194 | TEST_F(UmChromeOSPolicyTest, RecurringCheckBackoffIntervalAndFuzz) { |
| 195 | // Ensure that we're properly backing off and fuzzing in the presence of |
| 196 | // failed updates attempts. |
| 197 | Time next_update_check; |
| 198 | |
| 199 | fake_state_.updater_provider()->var_consecutive_failed_update_checks()-> |
| 200 | reset(new unsigned int(2)); // NOLINT(readability/casting) |
| 201 | |
| 202 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 203 | &ChromeOSPolicy::NextUpdateCheckTime, &next_update_check); |
| 204 | |
| 205 | int expected_interval = ChromeOSPolicy::kTimeoutPeriodicInterval * 4; |
| 206 | EXPECT_LE( |
| 207 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 208 | expected_interval - expected_interval / 2), |
| 209 | next_update_check); |
| 210 | EXPECT_GE( |
| 211 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 212 | expected_interval + expected_interval / 2), |
| 213 | next_update_check); |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 214 | } |
| 215 | |
Gilad Arnold | a0258a5 | 2014-07-10 16:21:19 -0700 | [diff] [blame] | 216 | TEST_F(UmChromeOSPolicyTest, RecurringCheckServerDictatedPollInterval) { |
| 217 | // Policy honors the server provided check poll interval. |
| 218 | Time next_update_check; |
| 219 | |
| 220 | const unsigned int kInterval = ChromeOSPolicy::kTimeoutPeriodicInterval * 4; |
| 221 | fake_state_.updater_provider()->var_server_dictated_poll_interval()-> |
| 222 | reset(new unsigned int(kInterval)); // NOLINT(readability/casting) |
| 223 | // We should not be backing off in this case. |
| 224 | fake_state_.updater_provider()->var_consecutive_failed_update_checks()-> |
| 225 | reset(new unsigned int(2)); // NOLINT(readability/casting) |
| 226 | |
| 227 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 228 | &ChromeOSPolicy::NextUpdateCheckTime, &next_update_check); |
| 229 | |
| 230 | EXPECT_LE( |
| 231 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 232 | kInterval - kInterval / 2), |
| 233 | next_update_check); |
| 234 | EXPECT_GE( |
| 235 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 236 | kInterval + kInterval / 2), |
| 237 | next_update_check); |
| 238 | } |
| 239 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 240 | TEST_F(UmChromeOSPolicyTest, ExponentialBackoffIsCapped) { |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 241 | Time next_update_check; |
| 242 | |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 243 | fake_state_.updater_provider()->var_consecutive_failed_update_checks()-> |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 244 | reset(new unsigned int(100)); // NOLINT(readability/casting) |
Gilad Arnold | 38b1402 | 2014-07-09 12:45:56 -0700 | [diff] [blame] | 245 | |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 246 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 247 | &ChromeOSPolicy::NextUpdateCheckTime, &next_update_check); |
| 248 | |
Gilad Arnold | 38b1402 | 2014-07-09 12:45:56 -0700 | [diff] [blame] | 249 | EXPECT_LE( |
| 250 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 251 | ChromeOSPolicy::kTimeoutMaxBackoffInterval - |
| 252 | ChromeOSPolicy::kTimeoutMaxBackoffInterval / 2), |
| 253 | next_update_check); |
| 254 | EXPECT_GE( |
| 255 | fake_clock_.GetWallclockTime() + TimeDelta::FromSeconds( |
| 256 | ChromeOSPolicy::kTimeoutMaxBackoffInterval + |
| 257 | ChromeOSPolicy::kTimeoutMaxBackoffInterval /2), |
| 258 | next_update_check); |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 259 | } |
| 260 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 261 | TEST_F(UmChromeOSPolicyTest, UpdateCheckAllowedWaitsForTheTimeout) { |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 262 | // We get the next update_check timestamp from the policy's private method |
| 263 | // and then we check the public method respects that value on the normal |
| 264 | // case. |
| 265 | Time next_update_check; |
| 266 | Time last_checked_time = |
| 267 | fake_clock_.GetWallclockTime() + TimeDelta::FromMinutes(1234); |
| 268 | |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 269 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 270 | new Time(last_checked_time)); |
| 271 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 272 | &ChromeOSPolicy::NextUpdateCheckTime, &next_update_check); |
| 273 | |
| 274 | UpdateCheckParams result; |
| 275 | |
| 276 | // Check that the policy blocks until the next_update_check is reached. |
| 277 | SetUpDefaultClock(); |
| 278 | SetUpDefaultState(); |
| 279 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 280 | new Time(last_checked_time)); |
| 281 | fake_clock_.SetWallclockTime(next_update_check - TimeDelta::FromSeconds(1)); |
| 282 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, |
| 283 | &Policy::UpdateCheckAllowed, &result); |
| 284 | |
| 285 | SetUpDefaultClock(); |
| 286 | SetUpDefaultState(); |
| 287 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 288 | new Time(last_checked_time)); |
| 289 | fake_clock_.SetWallclockTime(next_update_check + TimeDelta::FromSeconds(1)); |
| 290 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 291 | &Policy::UpdateCheckAllowed, &result); |
Gilad Arnold | a1eabcd | 2014-07-09 15:42:40 -0700 | [diff] [blame] | 292 | EXPECT_TRUE(result.updates_enabled); |
| 293 | } |
| 294 | |
| 295 | TEST_F(UmChromeOSPolicyTest, UpdateCheckAllowedWaitsForOOBE) { |
Alex Vakulenko | 072359c | 2014-07-18 11:41:07 -0700 | [diff] [blame^] | 296 | // Update checks are deferred until OOBE is completed. |
Gilad Arnold | a1eabcd | 2014-07-09 15:42:40 -0700 | [diff] [blame] | 297 | |
| 298 | // Ensure that update is not allowed even if wait period is satisfied. |
| 299 | Time next_update_check; |
| 300 | Time last_checked_time = |
| 301 | fake_clock_.GetWallclockTime() + TimeDelta::FromMinutes(1234); |
| 302 | |
| 303 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 304 | new Time(last_checked_time)); |
| 305 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 306 | &ChromeOSPolicy::NextUpdateCheckTime, &next_update_check); |
| 307 | |
| 308 | SetUpDefaultClock(); |
| 309 | SetUpDefaultState(); |
| 310 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 311 | new Time(last_checked_time)); |
| 312 | fake_clock_.SetWallclockTime(next_update_check + TimeDelta::FromSeconds(1)); |
| 313 | fake_state_.system_provider()->var_is_oobe_complete()->reset( |
| 314 | new bool(false)); |
| 315 | |
| 316 | UpdateCheckParams result; |
| 317 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, |
| 318 | &Policy::UpdateCheckAllowed, &result); |
| 319 | |
| 320 | // Now check that it is allowed if OOBE is completed. |
| 321 | SetUpDefaultClock(); |
| 322 | SetUpDefaultState(); |
| 323 | fake_state_.updater_provider()->var_last_checked_time()->reset( |
| 324 | new Time(last_checked_time)); |
| 325 | fake_clock_.SetWallclockTime(next_update_check + TimeDelta::FromSeconds(1)); |
| 326 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 327 | &Policy::UpdateCheckAllowed, &result); |
| 328 | EXPECT_TRUE(result.updates_enabled); |
Alex Deymo | 0d11c60 | 2014-04-23 20:12:20 -0700 | [diff] [blame] | 329 | } |
| 330 | |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 331 | TEST_F(UmChromeOSPolicyTest, UpdateCheckAllowedWithAttributes) { |
Alex Vakulenko | 072359c | 2014-07-18 11:41:07 -0700 | [diff] [blame^] | 332 | // Update check is allowed, response includes attributes for use in the |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 333 | // request. |
| 334 | SetUpdateCheckAllowed(true); |
| 335 | |
| 336 | // Override specific device policy attributes. |
| 337 | fake_state_.device_policy_provider()->var_release_channel_delegated()-> |
| 338 | reset(new bool(false)); |
| 339 | fake_state_.device_policy_provider()->var_release_channel()-> |
| 340 | reset(new string("foo-channel")); |
| 341 | |
| 342 | UpdateCheckParams result; |
| 343 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 344 | &Policy::UpdateCheckAllowed, &result); |
| 345 | EXPECT_TRUE(result.updates_enabled); |
| 346 | EXPECT_EQ("foo-channel", result.target_channel); |
| 347 | } |
| 348 | |
Gilad Arnold | fe12a0f | 2014-07-09 14:26:57 -0700 | [diff] [blame] | 349 | TEST_F(UmChromeOSPolicyTest, |
| 350 | UpdateCheckAllowedUpdatesDisabledForUnofficialBuilds) { |
| 351 | // UpdateCheckAllowed should return false (kSucceeded) if this is an |
| 352 | // unofficial build; we don't want periodic update checks on developer images. |
| 353 | |
| 354 | fake_state_.system_provider()->var_is_official_build()->reset( |
| 355 | new bool(false)); |
| 356 | |
| 357 | UpdateCheckParams result; |
| 358 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 359 | &Policy::UpdateCheckAllowed, &result); |
| 360 | EXPECT_FALSE(result.updates_enabled); |
| 361 | } |
| 362 | |
Gilad Arnold | bfc44f7 | 2014-07-09 14:41:39 -0700 | [diff] [blame] | 363 | TEST_F(UmChromeOSPolicyTest, |
| 364 | UpdateCheckAllowedUpdatesDisabledForRemovableBootDevice) { |
| 365 | // UpdateCheckAllowed should return false (kSucceeded) if the image booted |
| 366 | // from a removable device. |
| 367 | |
| 368 | fake_state_.system_provider()->var_is_boot_device_removable()->reset( |
| 369 | new bool(true)); |
| 370 | |
| 371 | UpdateCheckParams result; |
| 372 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 373 | &Policy::UpdateCheckAllowed, &result); |
| 374 | EXPECT_FALSE(result.updates_enabled); |
| 375 | } |
| 376 | |
Gilad Arnold | fe12a0f | 2014-07-09 14:26:57 -0700 | [diff] [blame] | 377 | TEST_F(UmChromeOSPolicyTest, UpdateCheckAllowedUpdatesDisabledByPolicy) { |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 378 | // UpdateCheckAllowed should return kAskMeAgainLater because a device policy |
| 379 | // is loaded and prohibits updates. |
| 380 | |
| 381 | SetUpdateCheckAllowed(false); |
| 382 | fake_state_.device_policy_provider()->var_update_disabled()->reset( |
| 383 | new bool(true)); |
| 384 | |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 385 | UpdateCheckParams result; |
| 386 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, |
| 387 | &Policy::UpdateCheckAllowed, &result); |
| 388 | } |
| 389 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 390 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartFailsCheckAllowedError) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 391 | // The UpdateCanStart policy fails, not being able to query |
| 392 | // UpdateCheckAllowed. |
| 393 | |
| 394 | // Configure the UpdateCheckAllowed policy to fail. |
| 395 | fake_state_.updater_provider()->var_updater_started_time()->reset(nullptr); |
| 396 | |
| 397 | // Check that the UpdateCanStart fails. |
| 398 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 399 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 400 | ExpectPolicyStatus(EvalStatus::kFailed, |
| 401 | &Policy::UpdateCanStart, &result, false, update_state); |
| 402 | } |
| 403 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 404 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartNotAllowedCheckDue) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 405 | // The UpdateCanStart policy returns false because we are due for another |
| 406 | // update check. |
| 407 | |
| 408 | SetUpdateCheckAllowed(true); |
| 409 | |
| 410 | // Check that the UpdateCanStart returns false. |
| 411 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 412 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 413 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 414 | &Policy::UpdateCanStart, &result, false, update_state); |
| 415 | EXPECT_FALSE(result.update_can_start); |
| 416 | EXPECT_EQ(UpdateCannotStartReason::kCheckDue, result.cannot_start_reason); |
| 417 | } |
| 418 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 419 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedNoDevicePolicy) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 420 | // The UpdateCanStart policy returns true; no device policy is loaded. |
| 421 | |
| 422 | SetUpdateCheckAllowed(false); |
| 423 | fake_state_.device_policy_provider()->var_device_policy_is_loaded()->reset( |
| 424 | new bool(false)); |
| 425 | |
| 426 | // Check that the UpdateCanStart returns true with no further attributes. |
| 427 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 428 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 429 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 430 | &Policy::UpdateCanStart, &result, false, update_state); |
| 431 | EXPECT_TRUE(result.update_can_start); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 432 | EXPECT_FALSE(result.p2p_allowed); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 433 | EXPECT_EQ(0, result.download_url_idx); |
| 434 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 435 | } |
| 436 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 437 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedBlankPolicy) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 438 | // The UpdateCanStart policy returns true; device policy is loaded but imposes |
| 439 | // no restrictions on updating. |
| 440 | |
| 441 | SetUpdateCheckAllowed(false); |
| 442 | |
| 443 | // Check that the UpdateCanStart returns true. |
| 444 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 445 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 446 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
| 447 | &Policy::UpdateCanStart, &result, false, update_state); |
| 448 | EXPECT_TRUE(result.update_can_start); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 449 | EXPECT_FALSE(result.p2p_allowed); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 450 | EXPECT_EQ(0, result.download_url_idx); |
| 451 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 452 | } |
| 453 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 454 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartFailsScatteringFailed) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 455 | // The UpdateCanStart policy fails because the UpdateScattering policy it |
| 456 | // depends on fails (unset variable). |
| 457 | |
| 458 | SetUpdateCheckAllowed(false); |
| 459 | |
| 460 | // Override the default seed variable with a null value so that the policy |
| 461 | // request would fail. |
| 462 | fake_state_.random_provider()->var_seed()->reset(nullptr); |
| 463 | |
| 464 | // Check that the UpdateCanStart fails. |
| 465 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(1)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 466 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 467 | ExpectPolicyStatus(EvalStatus::kFailed, |
| 468 | &Policy::UpdateCanStart, &result, false, update_state); |
| 469 | } |
| 470 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 471 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 472 | UpdateCanStartNotAllowedScatteringNewWaitPeriodApplies) { |
| 473 | // The UpdateCanStart policy returns false; device policy is loaded and |
| 474 | // scattering applies due to an unsatisfied wait period, which was newly |
| 475 | // generated. |
| 476 | |
| 477 | SetUpdateCheckAllowed(false); |
| 478 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 479 | new TimeDelta(TimeDelta::FromMinutes(2))); |
| 480 | |
| 481 | |
| 482 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(1)); |
| 483 | |
| 484 | // Check that the UpdateCanStart returns false and a new wait period |
| 485 | // generated. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 486 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 487 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 488 | false, update_state); |
| 489 | EXPECT_FALSE(result.update_can_start); |
| 490 | EXPECT_EQ(UpdateCannotStartReason::kScattering, result.cannot_start_reason); |
| 491 | EXPECT_LT(TimeDelta(), result.scatter_wait_period); |
| 492 | EXPECT_EQ(0, result.scatter_check_threshold); |
| 493 | } |
| 494 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 495 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 496 | UpdateCanStartNotAllowedScatteringPrevWaitPeriodStillApplies) { |
| 497 | // The UpdateCanStart policy returns false w/ kAskMeAgainLater; device policy |
| 498 | // is loaded and a previously generated scattering period still applies, none |
| 499 | // of the scattering values has changed. |
| 500 | |
| 501 | SetUpdateCheckAllowed(false); |
| 502 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 503 | new TimeDelta(TimeDelta::FromMinutes(2))); |
| 504 | |
| 505 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(1)); |
| 506 | update_state.scatter_wait_period = TimeDelta::FromSeconds(35); |
| 507 | |
| 508 | // Check that the UpdateCanStart returns false and a new wait period |
| 509 | // generated. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 510 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 511 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, &Policy::UpdateCanStart, |
| 512 | &result, false, update_state); |
| 513 | EXPECT_FALSE(result.update_can_start); |
| 514 | EXPECT_EQ(UpdateCannotStartReason::kScattering, result.cannot_start_reason); |
| 515 | EXPECT_EQ(TimeDelta::FromSeconds(35), result.scatter_wait_period); |
| 516 | EXPECT_EQ(0, result.scatter_check_threshold); |
| 517 | } |
| 518 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 519 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 520 | UpdateCanStartNotAllowedScatteringNewCountThresholdApplies) { |
| 521 | // The UpdateCanStart policy returns false; device policy is loaded and |
| 522 | // scattering applies due to an unsatisfied update check count threshold. |
| 523 | // |
| 524 | // This ensures a non-zero check threshold, which may or may not be combined |
| 525 | // with a non-zero wait period (for which we cannot reliably control). |
| 526 | |
| 527 | SetUpdateCheckAllowed(false); |
| 528 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 529 | new TimeDelta(TimeDelta::FromSeconds(1))); |
| 530 | |
| 531 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(1)); |
| 532 | update_state.scatter_check_threshold_min = 2; |
| 533 | update_state.scatter_check_threshold_max = 5; |
| 534 | |
| 535 | // Check that the UpdateCanStart returns false. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 536 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 537 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 538 | false, update_state); |
| 539 | EXPECT_FALSE(result.update_can_start); |
| 540 | EXPECT_EQ(UpdateCannotStartReason::kScattering, result.cannot_start_reason); |
| 541 | EXPECT_LE(2, result.scatter_check_threshold); |
| 542 | EXPECT_GE(5, result.scatter_check_threshold); |
| 543 | } |
| 544 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 545 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 546 | UpdateCanStartNotAllowedScatteringPrevCountThresholdStillApplies) { |
| 547 | // The UpdateCanStart policy returns false; device policy is loaded and |
| 548 | // scattering due to a previously generated count threshold still applies. |
| 549 | |
| 550 | SetUpdateCheckAllowed(false); |
| 551 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 552 | new TimeDelta(TimeDelta::FromSeconds(1))); |
| 553 | |
| 554 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(1)); |
| 555 | update_state.scatter_check_threshold = 3; |
| 556 | update_state.scatter_check_threshold_min = 2; |
| 557 | update_state.scatter_check_threshold_max = 5; |
| 558 | |
| 559 | // Check that the UpdateCanStart returns false. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 560 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 561 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 562 | false, update_state); |
| 563 | EXPECT_FALSE(result.update_can_start); |
| 564 | EXPECT_EQ(UpdateCannotStartReason::kScattering, result.cannot_start_reason); |
| 565 | EXPECT_EQ(3, result.scatter_check_threshold); |
| 566 | } |
| 567 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 568 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedScatteringSatisfied) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 569 | // The UpdateCanStart policy returns true; device policy is loaded and |
| 570 | // scattering is enabled, but both wait period and check threshold are |
| 571 | // satisfied. |
| 572 | |
| 573 | SetUpdateCheckAllowed(false); |
| 574 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 575 | new TimeDelta(TimeDelta::FromSeconds(120))); |
| 576 | |
| 577 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(75)); |
| 578 | update_state.num_checks = 4; |
| 579 | update_state.scatter_wait_period = TimeDelta::FromSeconds(60); |
| 580 | update_state.scatter_check_threshold = 3; |
| 581 | update_state.scatter_check_threshold_min = 2; |
| 582 | update_state.scatter_check_threshold_max = 5; |
| 583 | |
| 584 | // Check that the UpdateCanStart returns true. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 585 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 586 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 587 | false, update_state); |
| 588 | EXPECT_TRUE(result.update_can_start); |
| 589 | EXPECT_EQ(TimeDelta(), result.scatter_wait_period); |
| 590 | EXPECT_EQ(0, result.scatter_check_threshold); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 591 | EXPECT_EQ(0, result.download_url_idx); |
| 592 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 593 | } |
| 594 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 595 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 596 | UpdateCanStartAllowedInteractivePreventsScattering) { |
| 597 | // The UpdateCanStart policy returns true; device policy is loaded and |
| 598 | // scattering would have applied, except that the update check is interactive |
| 599 | // and so it is suppressed. |
| 600 | |
| 601 | SetUpdateCheckAllowed(false); |
| 602 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 603 | new TimeDelta(TimeDelta::FromSeconds(1))); |
| 604 | |
| 605 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(1)); |
| 606 | update_state.scatter_check_threshold = 0; |
| 607 | update_state.scatter_check_threshold_min = 2; |
| 608 | update_state.scatter_check_threshold_max = 5; |
| 609 | |
| 610 | // Check that the UpdateCanStart returns true. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 611 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 612 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 613 | true, update_state); |
| 614 | EXPECT_TRUE(result.update_can_start); |
| 615 | EXPECT_EQ(TimeDelta(), result.scatter_wait_period); |
| 616 | EXPECT_EQ(0, result.scatter_check_threshold); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 617 | EXPECT_EQ(0, result.download_url_idx); |
| 618 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 619 | } |
| 620 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 621 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | 76a11f6 | 2014-05-20 09:02:12 -0700 | [diff] [blame] | 622 | UpdateCanStartAllowedOobePreventsScattering) { |
| 623 | // The UpdateCanStart policy returns true; device policy is loaded and |
| 624 | // scattering would have applied, except that OOBE was not completed and so it |
| 625 | // is suppressed. |
| 626 | |
| 627 | SetUpdateCheckAllowed(false); |
| 628 | fake_state_.device_policy_provider()->var_scatter_factor()->reset( |
| 629 | new TimeDelta(TimeDelta::FromSeconds(1))); |
| 630 | fake_state_.system_provider()->var_is_oobe_complete()->reset(new bool(false)); |
| 631 | |
| 632 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromSeconds(1)); |
| 633 | update_state.scatter_check_threshold = 0; |
| 634 | update_state.scatter_check_threshold_min = 2; |
| 635 | update_state.scatter_check_threshold_max = 5; |
| 636 | |
| 637 | // Check that the UpdateCanStart returns true. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 638 | UpdateDownloadParams result; |
Gilad Arnold | 76a11f6 | 2014-05-20 09:02:12 -0700 | [diff] [blame] | 639 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 640 | true, update_state); |
| 641 | EXPECT_TRUE(result.update_can_start); |
| 642 | EXPECT_EQ(TimeDelta(), result.scatter_wait_period); |
| 643 | EXPECT_EQ(0, result.scatter_check_threshold); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 644 | EXPECT_EQ(0, result.download_url_idx); |
| 645 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | 76a11f6 | 2014-05-20 09:02:12 -0700 | [diff] [blame] | 646 | } |
| 647 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 648 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedWithAttributes) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 649 | // The UpdateCanStart policy returns true; device policy permits both HTTP and |
| 650 | // P2P updates, as well as a non-empty target channel string. |
| 651 | |
| 652 | SetUpdateCheckAllowed(false); |
| 653 | |
| 654 | // Override specific device policy attributes. |
| 655 | fake_state_.device_policy_provider()->var_http_downloads_enabled()->reset( |
| 656 | new bool(true)); |
| 657 | fake_state_.device_policy_provider()->var_au_p2p_enabled()->reset( |
| 658 | new bool(true)); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 659 | |
| 660 | // Check that the UpdateCanStart returns true. |
| 661 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 662 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 663 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 664 | false, update_state); |
| 665 | EXPECT_TRUE(result.update_can_start); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 666 | EXPECT_TRUE(result.p2p_allowed); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 667 | EXPECT_EQ(0, result.download_url_idx); |
| 668 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 669 | } |
| 670 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 671 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedWithP2PFromUpdater) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 672 | // The UpdateCanStart policy returns true; device policy forbids both HTTP and |
| 673 | // P2P updates, but the updater is configured to allow P2P and overrules the |
| 674 | // setting. |
| 675 | |
| 676 | SetUpdateCheckAllowed(false); |
| 677 | |
| 678 | // Override specific device policy attributes. |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 679 | fake_state_.updater_provider()->var_p2p_enabled()->reset(new bool(true)); |
| 680 | |
| 681 | // Check that the UpdateCanStart returns true. |
| 682 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 683 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 684 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 685 | false, update_state); |
| 686 | EXPECT_TRUE(result.update_can_start); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 687 | EXPECT_TRUE(result.p2p_allowed); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 688 | EXPECT_EQ(0, result.download_url_idx); |
| 689 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 690 | } |
| 691 | |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 692 | TEST_F(UmChromeOSPolicyTest, |
| 693 | UpdateCanStartAllowedWithHttpUrlForUnofficialBuild) { |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 694 | // The UpdateCanStart policy returns true; device policy forbids both HTTP and |
| 695 | // P2P updates, but marking this an unofficial build overrules the HTTP |
| 696 | // setting. |
| 697 | |
| 698 | SetUpdateCheckAllowed(false); |
| 699 | |
| 700 | // Override specific device policy attributes. |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 701 | fake_state_.device_policy_provider()->var_http_downloads_enabled()->reset( |
| 702 | new bool(false)); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 703 | fake_state_.system_provider()->var_is_official_build()-> |
| 704 | reset(new bool(false)); |
| 705 | |
| 706 | // Check that the UpdateCanStart returns true. |
| 707 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 708 | UpdateDownloadParams result; |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 709 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 710 | false, update_state); |
| 711 | EXPECT_TRUE(result.update_can_start); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 712 | EXPECT_FALSE(result.p2p_allowed); |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 713 | EXPECT_EQ(0, result.download_url_idx); |
| 714 | EXPECT_EQ(0, result.download_url_num_failures); |
| 715 | } |
| 716 | |
| 717 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedWithHttpsUrl) { |
| 718 | // The UpdateCanStart policy returns true; device policy forbids both HTTP and |
| 719 | // P2P updates, but an HTTPS URL is provided and selected for download. |
| 720 | |
| 721 | SetUpdateCheckAllowed(false); |
| 722 | |
| 723 | // Override specific device policy attributes. |
| 724 | fake_state_.device_policy_provider()->var_http_downloads_enabled()->reset( |
| 725 | new bool(false)); |
| 726 | |
| 727 | // Add an HTTPS URL. |
| 728 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
| 729 | update_state.download_urls.push_back("https://secure/url/"); |
| 730 | |
| 731 | // Check that the UpdateCanStart returns true. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 732 | UpdateDownloadParams result; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 733 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 734 | false, update_state); |
| 735 | EXPECT_TRUE(result.update_can_start); |
| 736 | EXPECT_FALSE(result.p2p_allowed); |
| 737 | EXPECT_EQ(1, result.download_url_idx); |
| 738 | EXPECT_EQ(0, result.download_url_num_failures); |
| 739 | } |
| 740 | |
| 741 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedWithSecondUrlMaxExceeded) { |
| 742 | // The UpdateCanStart policy returns true; the first URL exceeded the maximum |
| 743 | // allowed number of failures, but a second URL is available. |
| 744 | |
| 745 | SetUpdateCheckAllowed(false); |
| 746 | |
| 747 | // Add a second URL; update with this URL attempted and failed enough times to |
| 748 | // disqualify the current (first) URL. This tests both the previously |
| 749 | // accounted failures (download_url_num_failures) as well as those occurring |
| 750 | // since the last call (download_url_error_codes). |
| 751 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
| 752 | update_state.num_checks = 10; |
| 753 | update_state.download_urls.push_back("http://another/fake/url/"); |
| 754 | update_state.download_url_num_failures = 9; |
| 755 | update_state.download_url_error_codes.push_back( |
| 756 | ErrorCode::kDownloadTransferError); |
| 757 | update_state.download_url_error_codes.push_back( |
| 758 | ErrorCode::kDownloadWriteError); |
| 759 | |
| 760 | // Check that the UpdateCanStart returns true. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 761 | UpdateDownloadParams result; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 762 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 763 | false, update_state); |
| 764 | EXPECT_TRUE(result.update_can_start); |
| 765 | EXPECT_FALSE(result.p2p_allowed); |
| 766 | EXPECT_EQ(1, result.download_url_idx); |
| 767 | EXPECT_EQ(0, result.download_url_num_failures); |
| 768 | } |
| 769 | |
| 770 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedWithSecondUrlHardError) { |
| 771 | // The UpdateCanStart policy returns true; the first URL fails with a hard |
| 772 | // error, but a second URL is available. |
| 773 | |
| 774 | SetUpdateCheckAllowed(false); |
| 775 | |
| 776 | // Add a second URL; update with this URL attempted and failed in a way that |
| 777 | // causes it to switch directly to the next URL. |
| 778 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
| 779 | update_state.num_checks = 10; |
| 780 | update_state.download_urls.push_back("http://another/fake/url/"); |
| 781 | update_state.download_url_error_codes.push_back( |
| 782 | ErrorCode::kPayloadHashMismatchError); |
| 783 | |
| 784 | // Check that the UpdateCanStart returns true. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 785 | UpdateDownloadParams result; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 786 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 787 | false, update_state); |
| 788 | EXPECT_TRUE(result.update_can_start); |
| 789 | EXPECT_FALSE(result.p2p_allowed); |
| 790 | EXPECT_EQ(1, result.download_url_idx); |
| 791 | EXPECT_EQ(0, result.download_url_num_failures); |
| 792 | } |
| 793 | |
| 794 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedUrlWrapsAround) { |
| 795 | // The UpdateCanStart policy returns true; URL search properly wraps around |
| 796 | // the last one on the list. |
| 797 | |
| 798 | SetUpdateCheckAllowed(false); |
| 799 | |
| 800 | // Add a second URL; update with this URL attempted and failed in a way that |
| 801 | // causes it to switch directly to the next URL. |
| 802 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
| 803 | update_state.num_checks = 10; |
| 804 | update_state.download_urls.push_back("http://another/fake/url/"); |
| 805 | update_state.download_url_idx = 1; |
| 806 | update_state.download_url_error_codes.push_back( |
| 807 | ErrorCode::kPayloadHashMismatchError); |
| 808 | |
| 809 | // Check that the UpdateCanStart returns true. |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 810 | UpdateDownloadParams result; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 811 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 812 | false, update_state); |
| 813 | EXPECT_TRUE(result.update_can_start); |
| 814 | EXPECT_FALSE(result.p2p_allowed); |
| 815 | EXPECT_EQ(0, result.download_url_idx); |
| 816 | EXPECT_EQ(0, result.download_url_num_failures); |
| 817 | } |
| 818 | |
| 819 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartNotAllowedNoUsableUrls) { |
| 820 | // The UpdateCanStart policy returns false; there's a single HTTP URL but its |
| 821 | // use is forbidden by policy. |
| 822 | |
| 823 | SetUpdateCheckAllowed(false); |
| 824 | |
| 825 | // Override specific device policy attributes. |
| 826 | fake_state_.device_policy_provider()->var_http_downloads_enabled()->reset( |
| 827 | new bool(false)); |
| 828 | |
| 829 | // Check that the UpdateCanStart returns false. |
| 830 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 831 | UpdateDownloadParams result; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 832 | ExpectPolicyStatus(EvalStatus::kFailed, &Policy::UpdateCanStart, &result, |
| 833 | false, update_state); |
| 834 | } |
| 835 | |
| 836 | TEST_F(UmChromeOSPolicyTest, UpdateCanStartAllowedNoUsableUrlsButP2PEnabled) { |
| 837 | // The UpdateCanStart policy returns true; there's a single HTTP URL but its |
| 838 | // use is forbidden by policy, however P2P is enabled. The result indicates |
| 839 | // that no URL can be used. |
| 840 | |
| 841 | SetUpdateCheckAllowed(false); |
| 842 | |
| 843 | // Override specific device policy attributes. |
| 844 | fake_state_.device_policy_provider()->var_au_p2p_enabled()->reset( |
| 845 | new bool(true)); |
| 846 | fake_state_.device_policy_provider()->var_http_downloads_enabled()->reset( |
| 847 | new bool(false)); |
| 848 | |
| 849 | // Check that the UpdateCanStart returns true. |
| 850 | UpdateState update_state = GetDefaultUpdateState(TimeDelta::FromMinutes(10)); |
Gilad Arnold | 42f253b | 2014-06-25 12:39:17 -0700 | [diff] [blame] | 851 | UpdateDownloadParams result; |
Gilad Arnold | b3b0544 | 2014-05-30 14:25:05 -0700 | [diff] [blame] | 852 | ExpectPolicyStatus(EvalStatus::kSucceeded, &Policy::UpdateCanStart, &result, |
| 853 | false, update_state); |
| 854 | EXPECT_TRUE(result.update_can_start); |
| 855 | EXPECT_TRUE(result.p2p_allowed); |
| 856 | EXPECT_GT(0, result.download_url_idx); |
| 857 | EXPECT_EQ(0, result.download_url_num_failures); |
Gilad Arnold | f62a4b8 | 2014-05-01 07:41:07 -0700 | [diff] [blame] | 858 | } |
| 859 | |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 860 | TEST_F(UmChromeOSPolicyTest, UpdateDownloadAllowedEthernetDefault) { |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 861 | // Ethernet is always allowed. |
| 862 | |
| 863 | fake_state_.shill_provider()->var_conn_type()-> |
| 864 | reset(new ConnectionType(ConnectionType::kEthernet)); |
| 865 | |
| 866 | bool result; |
| 867 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 868 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 869 | EXPECT_TRUE(result); |
| 870 | } |
| 871 | |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 872 | TEST_F(UmChromeOSPolicyTest, UpdateDownloadAllowedWifiDefault) { |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 873 | // Wifi is allowed if not tethered. |
| 874 | |
| 875 | fake_state_.shill_provider()->var_conn_type()-> |
| 876 | reset(new ConnectionType(ConnectionType::kWifi)); |
| 877 | |
| 878 | bool result; |
| 879 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 880 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 881 | EXPECT_TRUE(result); |
| 882 | } |
| 883 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 884 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 885 | UpdateCurrentConnectionNotAllowedWifiTetheredDefault) { |
| 886 | // Tethered wifi is not allowed by default. |
| 887 | |
| 888 | fake_state_.shill_provider()->var_conn_type()-> |
| 889 | reset(new ConnectionType(ConnectionType::kWifi)); |
| 890 | fake_state_.shill_provider()->var_conn_tethering()-> |
| 891 | reset(new ConnectionTethering(ConnectionTethering::kConfirmed)); |
| 892 | |
| 893 | bool result; |
Gilad Arnold | 28d6be6 | 2014-06-30 14:04:04 -0700 | [diff] [blame] | 894 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 895 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 896 | } |
| 897 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 898 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 899 | UpdateDownloadAllowedWifiTetheredPolicyOverride) { |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 900 | // Tethered wifi can be allowed by policy. |
| 901 | |
| 902 | fake_state_.shill_provider()->var_conn_type()-> |
| 903 | reset(new ConnectionType(ConnectionType::kWifi)); |
| 904 | fake_state_.shill_provider()->var_conn_tethering()-> |
| 905 | reset(new ConnectionTethering(ConnectionTethering::kConfirmed)); |
| 906 | set<ConnectionType> allowed_connections; |
| 907 | allowed_connections.insert(ConnectionType::kCellular); |
| 908 | fake_state_.device_policy_provider()-> |
| 909 | var_allowed_connection_types_for_update()-> |
| 910 | reset(new set<ConnectionType>(allowed_connections)); |
| 911 | |
| 912 | bool result; |
| 913 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 914 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 915 | EXPECT_TRUE(result); |
| 916 | } |
| 917 | |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 918 | TEST_F(UmChromeOSPolicyTest, UpdateDownloadAllowedWimaxDefault) { |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 919 | // Wimax is always allowed. |
| 920 | |
| 921 | fake_state_.shill_provider()->var_conn_type()-> |
| 922 | reset(new ConnectionType(ConnectionType::kWifi)); |
| 923 | |
| 924 | bool result; |
| 925 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 926 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 927 | EXPECT_TRUE(result); |
| 928 | } |
| 929 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 930 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 931 | UpdateCurrentConnectionNotAllowedBluetoothDefault) { |
| 932 | // Bluetooth is never allowed. |
| 933 | |
| 934 | fake_state_.shill_provider()->var_conn_type()-> |
| 935 | reset(new ConnectionType(ConnectionType::kBluetooth)); |
| 936 | |
| 937 | bool result; |
Gilad Arnold | 28d6be6 | 2014-06-30 14:04:04 -0700 | [diff] [blame] | 938 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 939 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 940 | } |
| 941 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 942 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 943 | UpdateCurrentConnectionNotAllowedBluetoothPolicyCannotOverride) { |
| 944 | // Bluetooth cannot be allowed even by policy. |
| 945 | |
| 946 | fake_state_.shill_provider()->var_conn_type()-> |
| 947 | reset(new ConnectionType(ConnectionType::kBluetooth)); |
| 948 | set<ConnectionType> allowed_connections; |
| 949 | allowed_connections.insert(ConnectionType::kBluetooth); |
| 950 | fake_state_.device_policy_provider()-> |
| 951 | var_allowed_connection_types_for_update()-> |
| 952 | reset(new set<ConnectionType>(allowed_connections)); |
| 953 | |
| 954 | bool result; |
Gilad Arnold | 28d6be6 | 2014-06-30 14:04:04 -0700 | [diff] [blame] | 955 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 956 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 957 | } |
| 958 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 959 | TEST_F(UmChromeOSPolicyTest, UpdateCurrentConnectionNotAllowedCellularDefault) { |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 960 | // Cellular is not allowed by default. |
| 961 | |
| 962 | fake_state_.shill_provider()->var_conn_type()-> |
| 963 | reset(new ConnectionType(ConnectionType::kCellular)); |
| 964 | |
| 965 | bool result; |
Gilad Arnold | 28d6be6 | 2014-06-30 14:04:04 -0700 | [diff] [blame] | 966 | ExpectPolicyStatus(EvalStatus::kAskMeAgainLater, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 967 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 968 | } |
| 969 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 970 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 971 | UpdateDownloadAllowedCellularPolicyOverride) { |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 972 | // Update over cellular can be enabled by policy. |
| 973 | |
| 974 | fake_state_.shill_provider()->var_conn_type()-> |
| 975 | reset(new ConnectionType(ConnectionType::kCellular)); |
| 976 | set<ConnectionType> allowed_connections; |
| 977 | allowed_connections.insert(ConnectionType::kCellular); |
| 978 | fake_state_.device_policy_provider()-> |
| 979 | var_allowed_connection_types_for_update()-> |
| 980 | reset(new set<ConnectionType>(allowed_connections)); |
| 981 | |
| 982 | bool result; |
| 983 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 984 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 985 | EXPECT_TRUE(result); |
| 986 | } |
| 987 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 988 | TEST_F(UmChromeOSPolicyTest, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 989 | UpdateDownloadAllowedCellularUserOverride) { |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 990 | // Update over cellular can be enabled by user settings, but only if policy |
| 991 | // is present and does not determine allowed connections. |
| 992 | |
| 993 | fake_state_.shill_provider()->var_conn_type()-> |
| 994 | reset(new ConnectionType(ConnectionType::kCellular)); |
| 995 | set<ConnectionType> allowed_connections; |
| 996 | allowed_connections.insert(ConnectionType::kCellular); |
| 997 | fake_state_.updater_provider()->var_cellular_enabled()-> |
| 998 | reset(new bool(true)); |
| 999 | |
| 1000 | bool result; |
| 1001 | ExpectPolicyStatus(EvalStatus::kSucceeded, |
Gilad Arnold | 684219d | 2014-07-07 14:54:57 -0700 | [diff] [blame] | 1002 | &Policy::UpdateDownloadAllowed, &result); |
Gilad Arnold | 0adbc94 | 2014-05-12 10:35:43 -0700 | [diff] [blame] | 1003 | EXPECT_TRUE(result); |
| 1004 | } |
| 1005 | |
Alex Deymo | 63784a5 | 2014-05-28 10:46:14 -0700 | [diff] [blame] | 1006 | } // namespace chromeos_update_manager |