OLD | NEW |
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
4 | 4 |
5 #include "chrome/browser/policy/network_configuration_updater.h" | 5 #include "chrome/browser/policy/network_configuration_updater.h" |
6 | 6 |
7 #include "chrome/browser/chromeos/cros/mock_network_library.h" | 7 #include "chrome/browser/chromeos/cros/mock_network_library.h" |
8 #include "chrome/browser/policy/mock_configuration_policy_provider.h" | 8 #include "chrome/browser/policy/mock_configuration_policy_provider.h" |
9 #include "chrome/browser/policy/policy_map.h" | 9 #include "chrome/browser/policy/policy_map.h" |
10 #include "policy/policy_constants.h" | 10 #include "policy/policy_constants.h" |
11 #include "testing/gmock/include/gmock/gmock.h" | 11 #include "testing/gmock/include/gmock/gmock.h" |
12 #include "testing/gtest/include/gtest/gtest.h" | 12 #include "testing/gtest/include/gtest/gtest.h" |
13 | 13 |
14 using testing::Mock; | 14 using testing::Mock; |
15 using testing::Return; | 15 using testing::Return; |
16 using testing::_; | 16 using testing::_; |
17 | 17 |
18 namespace policy { | 18 namespace policy { |
19 | 19 |
20 static const char kFakeONC[] = "{ \"GUID\": \"1234\" }"; | 20 static const char kFakeONC[] = "{ \"GUID\": \"1234\" }"; |
21 | 21 |
22 class NetworkConfigurationUpdaterTest | 22 class NetworkConfigurationUpdaterTest |
23 : public testing::TestWithParam<const char*> { | 23 : public testing::TestWithParam<const char*> { |
24 protected: | 24 protected: |
25 virtual void SetUp() OVERRIDE { | 25 virtual void SetUp() OVERRIDE { |
26 EXPECT_CALL(provider_, ProvideInternal(_)) | |
27 .WillRepeatedly(CopyPolicyMap(&policy_)); | |
28 EXPECT_CALL(network_library_, LoadOncNetworks(_, "", _, _)) | 26 EXPECT_CALL(network_library_, LoadOncNetworks(_, "", _, _)) |
29 .WillRepeatedly(Return(true)); | 27 .WillRepeatedly(Return(true)); |
30 } | 28 } |
31 | 29 |
32 // Maps configuration policy name to corresponding ONC source. | 30 // Maps configuration policy name to corresponding ONC source. |
33 static chromeos::NetworkUIData::ONCSource NameToONCSource( | 31 static chromeos::NetworkUIData::ONCSource NameToONCSource( |
34 const std::string& name) { | 32 const std::string& name) { |
35 if (name == key::kDeviceOpenNetworkConfiguration) | 33 if (name == key::kDeviceOpenNetworkConfiguration) |
36 return chromeos::NetworkUIData::ONC_SOURCE_DEVICE_POLICY; | 34 return chromeos::NetworkUIData::ONC_SOURCE_DEVICE_POLICY; |
37 if (name == key::kOpenNetworkConfiguration) | 35 if (name == key::kOpenNetworkConfiguration) |
38 return chromeos::NetworkUIData::ONC_SOURCE_USER_POLICY; | 36 return chromeos::NetworkUIData::ONC_SOURCE_USER_POLICY; |
39 return chromeos::NetworkUIData::ONC_SOURCE_NONE; | 37 return chromeos::NetworkUIData::ONC_SOURCE_NONE; |
40 } | 38 } |
41 | 39 |
42 chromeos::MockNetworkLibrary network_library_; | 40 chromeos::MockNetworkLibrary network_library_; |
43 PolicyMap policy_; | |
44 MockConfigurationPolicyProvider provider_; | 41 MockConfigurationPolicyProvider provider_; |
45 }; | 42 }; |
46 | 43 |
47 TEST_P(NetworkConfigurationUpdaterTest, InitialUpdate) { | 44 TEST_P(NetworkConfigurationUpdaterTest, InitialUpdate) { |
48 policy_.Set(GetParam(), POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER, | 45 PolicyMap policy; |
49 Value::CreateStringValue(kFakeONC)); | 46 policy.Set(GetParam(), POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER, |
| 47 Value::CreateStringValue(kFakeONC)); |
| 48 provider_.UpdateChromePolicy(policy); |
50 | 49 |
51 EXPECT_CALL(network_library_, | 50 EXPECT_CALL(network_library_, |
52 LoadOncNetworks(kFakeONC, "", NameToONCSource(GetParam()), _)) | 51 LoadOncNetworks(kFakeONC, "", NameToONCSource(GetParam()), _)) |
53 .WillOnce(Return(true)); | 52 .WillOnce(Return(true)); |
54 | 53 |
55 NetworkConfigurationUpdater updater(&provider_, &network_library_); | 54 NetworkConfigurationUpdater updater(&provider_, &network_library_); |
56 Mock::VerifyAndClearExpectations(&network_library_); | 55 Mock::VerifyAndClearExpectations(&network_library_); |
57 } | 56 } |
58 | 57 |
59 TEST_P(NetworkConfigurationUpdaterTest, PolicyChange) { | 58 TEST_P(NetworkConfigurationUpdaterTest, PolicyChange) { |
60 NetworkConfigurationUpdater updater(&provider_, &network_library_); | 59 NetworkConfigurationUpdater updater(&provider_, &network_library_); |
61 | 60 |
62 // We should update if policy changes. | 61 // We should update if policy changes. |
63 EXPECT_CALL(network_library_, | 62 EXPECT_CALL(network_library_, |
64 LoadOncNetworks(kFakeONC, "", NameToONCSource(GetParam()), _)) | 63 LoadOncNetworks(kFakeONC, "", NameToONCSource(GetParam()), _)) |
65 .WillOnce(Return(true)); | 64 .WillOnce(Return(true)); |
66 policy_.Set(GetParam(), POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER, | 65 PolicyMap policy; |
67 Value::CreateStringValue(kFakeONC)); | 66 policy.Set(GetParam(), POLICY_LEVEL_MANDATORY, POLICY_SCOPE_USER, |
68 provider_.NotifyPolicyUpdated(); | 67 Value::CreateStringValue(kFakeONC)); |
| 68 provider_.UpdateChromePolicy(policy); |
69 Mock::VerifyAndClearExpectations(&network_library_); | 69 Mock::VerifyAndClearExpectations(&network_library_); |
70 | 70 |
71 // No update if the set the same value again. | 71 // No update if the set the same value again. |
72 EXPECT_CALL(network_library_, | 72 EXPECT_CALL(network_library_, |
73 LoadOncNetworks(kFakeONC, "", NameToONCSource(GetParam()), _)) | 73 LoadOncNetworks(kFakeONC, "", NameToONCSource(GetParam()), _)) |
74 .Times(0); | 74 .Times(0); |
75 provider_.NotifyPolicyUpdated(); | 75 provider_.UpdateChromePolicy(policy); |
76 Mock::VerifyAndClearExpectations(&network_library_); | 76 Mock::VerifyAndClearExpectations(&network_library_); |
77 | 77 |
78 // Another update is expected if the policy goes away. | 78 // Another update is expected if the policy goes away. |
79 EXPECT_CALL(network_library_, | 79 EXPECT_CALL(network_library_, |
80 LoadOncNetworks(NetworkConfigurationUpdater::kEmptyConfiguration, | 80 LoadOncNetworks(NetworkConfigurationUpdater::kEmptyConfiguration, |
81 "", NameToONCSource(GetParam()), _)) | 81 "", NameToONCSource(GetParam()), _)) |
82 .WillOnce(Return(true)); | 82 .WillOnce(Return(true)); |
83 policy_.Erase(GetParam()); | 83 policy.Erase(GetParam()); |
84 provider_.NotifyPolicyUpdated(); | 84 provider_.UpdateChromePolicy(policy); |
85 Mock::VerifyAndClearExpectations(&network_library_); | 85 Mock::VerifyAndClearExpectations(&network_library_); |
86 } | 86 } |
87 | 87 |
88 INSTANTIATE_TEST_CASE_P( | 88 INSTANTIATE_TEST_CASE_P( |
89 NetworkConfigurationUpdaterTestInstance, | 89 NetworkConfigurationUpdaterTestInstance, |
90 NetworkConfigurationUpdaterTest, | 90 NetworkConfigurationUpdaterTest, |
91 testing::Values(key::kDeviceOpenNetworkConfiguration, | 91 testing::Values(key::kDeviceOpenNetworkConfiguration, |
92 key::kOpenNetworkConfiguration)); | 92 key::kOpenNetworkConfiguration)); |
93 | 93 |
94 } // namespace policy | 94 } // namespace policy |
OLD | NEW |