Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(298)

Side by Side Diff: chrome/browser/chromeos/cros/onc_network_parser_unittest.cc

Issue 8883046: Show parse errors in the UI when loading ONC files. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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/cros/onc_network_parser.h" 5 #include "chrome/browser/chromeos/cros/onc_network_parser.h"
6 6
7 #include <cert.h> 7 #include <cert.h>
8 #include <keyhi.h> 8 #include <keyhi.h>
9 #include <pk11pub.h> 9 #include <pk11pub.h>
10 10
(...skipping 245 matching lines...) Expand 10 before | Expand all | Expand 10 after
256 " \"Security\": \"WEP-PSK\"," 256 " \"Security\": \"WEP-PSK\","
257 " \"SSID\": \"ssid\"," 257 " \"SSID\": \"ssid\","
258 " \"Passphrase\": \"pass\"," 258 " \"Passphrase\": \"pass\","
259 " }" 259 " }"
260 " }]" 260 " }]"
261 "}"); 261 "}");
262 OncNetworkParser parser(test_blob); 262 OncNetworkParser parser(test_blob);
263 263
264 EXPECT_EQ(1, parser.GetNetworkConfigsSize()); 264 EXPECT_EQ(1, parser.GetNetworkConfigsSize());
265 EXPECT_EQ(0, parser.GetCertificatesSize()); 265 EXPECT_EQ(0, parser.GetCertificatesSize());
266 EXPECT_FALSE(parser.ParseNetwork(1));
267 scoped_ptr<Network> network(parser.ParseNetwork(0)); 266 scoped_ptr<Network> network(parser.ParseNetwork(0));
268 ASSERT_TRUE(network.get()); 267 ASSERT_TRUE(network.get());
269 268
270 EXPECT_EQ(network->type(), chromeos::TYPE_WIFI); 269 EXPECT_EQ(network->type(), chromeos::TYPE_WIFI);
271 WifiNetwork* wifi = static_cast<WifiNetwork*>(network.get()); 270 WifiNetwork* wifi = static_cast<WifiNetwork*>(network.get());
272 EXPECT_EQ(wifi->encryption(), chromeos::SECURITY_WEP); 271 EXPECT_EQ(wifi->encryption(), chromeos::SECURITY_WEP);
273 CheckStringProperty(wifi, PROPERTY_INDEX_SECURITY, flimflam::kSecurityWep); 272 CheckStringProperty(wifi, PROPERTY_INDEX_SECURITY, flimflam::kSecurityWep);
274 EXPECT_EQ(wifi->name(), "ssid"); 273 EXPECT_EQ(wifi->name(), "ssid");
275 CheckStringProperty(wifi, PROPERTY_INDEX_SSID, "ssid"); 274 CheckStringProperty(wifi, PROPERTY_INDEX_SSID, "ssid");
276 EXPECT_EQ(wifi->auto_connect(), false); 275 EXPECT_EQ(wifi->auto_connect(), false);
(...skipping 15 matching lines...) Expand all
292 " \"Outer\": \"PEAP\"," 291 " \"Outer\": \"PEAP\","
293 " \"UseSystemCAs\": false," 292 " \"UseSystemCAs\": false,"
294 " }" 293 " }"
295 " }" 294 " }"
296 " }]" 295 " }]"
297 "}"); 296 "}");
298 OncNetworkParser parser(test_blob); 297 OncNetworkParser parser(test_blob);
299 298
300 EXPECT_EQ(1, parser.GetNetworkConfigsSize()); 299 EXPECT_EQ(1, parser.GetNetworkConfigsSize());
301 EXPECT_EQ(0, parser.GetCertificatesSize()); 300 EXPECT_EQ(0, parser.GetCertificatesSize());
302 EXPECT_FALSE(parser.ParseNetwork(1));
303 scoped_ptr<Network> network(parser.ParseNetwork(0)); 301 scoped_ptr<Network> network(parser.ParseNetwork(0));
304 ASSERT_TRUE(network.get()); 302 ASSERT_TRUE(network.get());
305 303
306 EXPECT_EQ(network->type(), chromeos::TYPE_WIFI); 304 EXPECT_EQ(network->type(), chromeos::TYPE_WIFI);
307 WifiNetwork* wifi = static_cast<WifiNetwork*>(network.get()); 305 WifiNetwork* wifi = static_cast<WifiNetwork*>(network.get());
308 EXPECT_EQ(wifi->encryption(), chromeos::SECURITY_8021X); 306 EXPECT_EQ(wifi->encryption(), chromeos::SECURITY_8021X);
309 CheckStringProperty(wifi, PROPERTY_INDEX_SECURITY, flimflam::kSecurity8021x); 307 CheckStringProperty(wifi, PROPERTY_INDEX_SECURITY, flimflam::kSecurity8021x);
310 EXPECT_EQ(wifi->name(), "ssid"); 308 EXPECT_EQ(wifi->name(), "ssid");
311 EXPECT_EQ(wifi->auto_connect(), true); 309 EXPECT_EQ(wifi->auto_connect(), true);
312 CheckBooleanProperty(wifi, PROPERTY_INDEX_AUTO_CONNECT, true); 310 CheckBooleanProperty(wifi, PROPERTY_INDEX_AUTO_CONNECT, true);
(...skipping 19 matching lines...) Expand all
332 " \"Password\": \"pass\"," 330 " \"Password\": \"pass\","
333 " \"AnonymousIdentity\": \"anon\"," 331 " \"AnonymousIdentity\": \"anon\","
334 " }" 332 " }"
335 " }" 333 " }"
336 " }]" 334 " }]"
337 "}"); 335 "}");
338 OncNetworkParser parser(test_blob); 336 OncNetworkParser parser(test_blob);
339 337
340 EXPECT_EQ(1, parser.GetNetworkConfigsSize()); 338 EXPECT_EQ(1, parser.GetNetworkConfigsSize());
341 EXPECT_EQ(0, parser.GetCertificatesSize()); 339 EXPECT_EQ(0, parser.GetCertificatesSize());
342 EXPECT_FALSE(parser.ParseNetwork(1));
343 scoped_ptr<Network> network(parser.ParseNetwork(0)); 340 scoped_ptr<Network> network(parser.ParseNetwork(0));
344 ASSERT_TRUE(network.get()); 341 ASSERT_TRUE(network.get());
345 342
346 EXPECT_EQ(network->type(), chromeos::TYPE_WIFI); 343 EXPECT_EQ(network->type(), chromeos::TYPE_WIFI);
347 WifiNetwork* wifi = static_cast<WifiNetwork*>(network.get()); 344 WifiNetwork* wifi = static_cast<WifiNetwork*>(network.get());
348 EXPECT_EQ(wifi->encryption(), chromeos::SECURITY_8021X); 345 EXPECT_EQ(wifi->encryption(), chromeos::SECURITY_8021X);
349 EXPECT_EQ(wifi->name(), "ssid"); 346 EXPECT_EQ(wifi->name(), "ssid");
350 EXPECT_EQ(wifi->auto_connect(), false); 347 EXPECT_EQ(wifi->auto_connect(), false);
351 EXPECT_EQ(wifi->eap_method(), EAP_METHOD_LEAP); 348 EXPECT_EQ(wifi->eap_method(), EAP_METHOD_LEAP);
352 EXPECT_EQ(wifi->eap_use_system_cas(), true); 349 EXPECT_EQ(wifi->eap_use_system_cas(), true);
(...skipping 298 matching lines...) Expand 10 before | Expand all | Expand 10 after
651 648
652 EXPECT_EQ(1, parser.GetNetworkConfigsSize()); 649 EXPECT_EQ(1, parser.GetNetworkConfigsSize());
653 scoped_ptr<Network> network(parser.ParseNetwork(0)); 650 scoped_ptr<Network> network(parser.ParseNetwork(0));
654 ASSERT_TRUE(network.get() != NULL); 651 ASSERT_TRUE(network.get() != NULL);
655 EXPECT_EQ(network->type(), chromeos::TYPE_VPN); 652 EXPECT_EQ(network->type(), chromeos::TYPE_VPN);
656 VirtualNetwork* vpn = static_cast<VirtualNetwork*>(network.get()); 653 VirtualNetwork* vpn = static_cast<VirtualNetwork*>(network.get());
657 EXPECT_EQ(PROVIDER_TYPE_OPEN_VPN, vpn->provider_type()); 654 EXPECT_EQ(PROVIDER_TYPE_OPEN_VPN, vpn->provider_type());
658 } 655 }
659 656
660 } // namespace chromeos 657 } // namespace chromeos
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/cros/onc_network_parser.cc ('k') | chrome/browser/policy/network_configuration_updater.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698