Chromium Code Reviews| 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/chromeos/network_settings/onc_validator.h" | 5 #include "chrome/browser/chromeos/network_settings/onc_validator.h" |
| 6 | 6 |
| 7 #include <string> | 7 #include <string> |
| 8 | 8 |
| 9 #include "base/logging.h" | 9 #include "base/logging.h" |
| 10 #include "base/memory/scoped_ptr.h" | 10 #include "base/memory/scoped_ptr.h" |
| (...skipping 98 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 109 TEST_P(ONCValidatorValidTest, LiberalValidationValid) { | 109 TEST_P(ONCValidatorValidTest, LiberalValidationValid) { |
| 110 OncParams onc = GetParam(); | 110 OncParams onc = GetParam(); |
| 111 Validate(false, test_utils::ReadTestDictionary(onc.location), onc.signature, | 111 Validate(false, test_utils::ReadTestDictionary(onc.location), onc.signature, |
| 112 onc.is_managed); | 112 onc.is_managed); |
| 113 ExpectValid(); | 113 ExpectValid(); |
| 114 } | 114 } |
| 115 | 115 |
| 116 INSTANTIATE_TEST_CASE_P( | 116 INSTANTIATE_TEST_CASE_P( |
| 117 ONCValidatorValidTest, | 117 ONCValidatorValidTest, |
| 118 ONCValidatorValidTest, | 118 ONCValidatorValidTest, |
| 119 ::testing::Values(OncParams("managed_toplevel.onc", | 119 ::testing::Values(OncParams("managed_toplevel1.onc", |
| 120 &kToplevelConfigurationSignature, | |
| 121 true), | |
| 122 OncParams("managed_toplevel2.onc", | |
| 123 &kToplevelConfigurationSignature, | |
| 124 true), | |
| 125 // Test a configuration generated by CPanel. | |
| 126 OncParams("managed_toplevel_cpanel.onc", | |
| 120 &kToplevelConfigurationSignature, | 127 &kToplevelConfigurationSignature, |
| 121 true), | 128 true), |
| 122 OncParams("encrypted.onc", | 129 OncParams("encrypted.onc", |
| 123 &kToplevelConfigurationSignature, | 130 &kToplevelConfigurationSignature, |
| 124 true), | 131 true), |
| 125 OncParams("managed_vpn.onc", | 132 OncParams("managed_vpn.onc", |
| 126 &kNetworkConfigurationSignature, | 133 &kNetworkConfigurationSignature, |
| 127 true), | 134 true), |
| 128 OncParams("managed_ethernet.onc", | 135 OncParams("managed_ethernet.onc", |
| 129 &kNetworkConfigurationSignature, | 136 &kNetworkConfigurationSignature, |
| (...skipping 101 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 231 &kToplevelConfigurationSignature, | 238 &kToplevelConfigurationSignature, |
| 232 true), | 239 true), |
| 233 RepairParams("toplevel-repaired", | 240 RepairParams("toplevel-repaired", |
| 234 "toplevel-repaired")), | 241 "toplevel-repaired")), |
| 235 // Ignore recommended arrays in unmanaged ONC. | 242 // Ignore recommended arrays in unmanaged ONC. |
| 236 std::make_pair(OncParams("network-with-illegal-recommended", | 243 std::make_pair(OncParams("network-with-illegal-recommended", |
| 237 &kNetworkConfigurationSignature, | 244 &kNetworkConfigurationSignature, |
| 238 false), | 245 false), |
| 239 RepairParams("network-repaired", "network-repaired")))); | 246 RepairParams("network-repaired", "network-repaired")))); |
| 240 | 247 |
| 248 // Strict and liberal validator repair but differently. | |
|
Mattias Nissler (ping if slow)
2012/12/13 09:51:46
... both repair, but with different results.
pneubeck (no reviews)
2012/12/13 14:33:02
Done.
| |
| 249 INSTANTIATE_TEST_CASE_P( | |
| 250 StrictAndLiberalRepairDifferently, | |
| 251 ONCValidatorTestRepairable, | |
| 252 ::testing::Values( | |
| 253 std::make_pair(OncParams("toplevel-with-nested-warning", | |
| 254 &kToplevelConfigurationSignature, | |
| 255 false), | |
| 256 RepairParams("toplevel-empty", "toplevel-repaired")))); | |
| 257 | |
| 241 // Strict and liberal validator return both INVALID. | 258 // Strict and liberal validator return both INVALID. |
| 242 INSTANTIATE_TEST_CASE_P( | 259 INSTANTIATE_TEST_CASE_P( |
| 243 StrictAndLiberalInvalid, | 260 StrictAndLiberalInvalid, |
| 244 ONCValidatorTestRepairable, | 261 ONCValidatorTestRepairable, |
| 245 ::testing::Values( | 262 ::testing::Values( |
| 246 std::make_pair(OncParams("network-unknown-value", | 263 std::make_pair(OncParams("network-unknown-value", |
| 247 &kNetworkConfigurationSignature, false), | 264 &kNetworkConfigurationSignature, false), |
| 248 RepairParams("", "")), | 265 RepairParams("", "")), |
| 249 std::make_pair(OncParams("managed-network-unknown-value", | 266 std::make_pair(OncParams("managed-network-unknown-value", |
| 250 &kNetworkConfigurationSignature, true), | 267 &kNetworkConfigurationSignature, true), |
| 251 RepairParams("", "")), | 268 RepairParams("", "")), |
| 252 std::make_pair(OncParams("network-value-out-of-range", | 269 std::make_pair(OncParams("network-value-out-of-range", |
| 253 &kNetworkConfigurationSignature, false), | 270 &kNetworkConfigurationSignature, false), |
| 254 RepairParams("", "")), | 271 RepairParams("", "")), |
| 255 std::make_pair(OncParams("managed-network-value-out-of-range", | 272 std::make_pair(OncParams("managed-network-value-out-of-range", |
| 256 &kNetworkConfigurationSignature, true), | 273 &kNetworkConfigurationSignature, true), |
| 257 RepairParams("", "")), | 274 RepairParams("", "")), |
| 258 std::make_pair(OncParams("network-wrong-type", | 275 std::make_pair(OncParams("network-wrong-type", |
| 259 &kNetworkConfigurationSignature, false), | 276 &kNetworkConfigurationSignature, false), |
| 260 RepairParams("", "")), | 277 RepairParams("", "")), |
| 261 std::make_pair(OncParams("managed-network-wrong-type", | 278 std::make_pair(OncParams("managed-network-wrong-type", |
| 262 &kNetworkConfigurationSignature, true), | 279 &kNetworkConfigurationSignature, true), |
| 263 RepairParams("", "")))); | 280 RepairParams("", "")))); |
| 264 | 281 |
| 265 } // namespace onc | 282 } // namespace onc |
| 266 } // namespace chromeos | 283 } // namespace chromeos |
| OLD | NEW |