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

Unified Diff: server/site_wlan_connect.py

Issue 6905170: Add test to select between identical SSID/different Auth (Closed) Base URL: http://git.chromium.org/git/autotest.git@master
Patch Set: Created 9 years, 8 months 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « server/site_tests/network_WiFiManager/040MutiAuth ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: server/site_wlan_connect.py
diff --git a/server/site_wlan_connect.py b/server/site_wlan_connect.py
index 4bcbec475cbaa716fdedb2a77b94608a9bc294c5..50c639f510c852886d697d2f86c61678bb2895d3 100644
--- a/server/site_wlan_connect.py
+++ b/server/site_wlan_connect.py
@@ -63,21 +63,24 @@ class ConnectStateHandler(StateHandler):
for svc in FindObjects('Service', 'SSID', self.service_name,
path_list=path_list):
props = svc.GetProperties()
+ set_props = {}
for key, val in self.connection_settings.items():
prop_val = convert_dbus_value(props.get(key))
if key != 'SSID' and prop_val != val:
if key in ['Passphrase', 'SaveCredentials'] or key.startswith('EAP.'):
- try:
- svc.SetProperty(key, val)
- except dbus.exceptions.DBusException, e:
- self.failure = ('SetProperty: DBus exception %s for set of %s' %
- (e, key))
- return None
+ set_props[key] = val
else:
self.Debug('Service key mismatch: %s %s != %s' %
(key, val, str(prop_val)))
break
else:
+ for key, val in set_props.iteritems():
+ try:
+ svc.SetProperty(key, val)
+ except dbus.exceptions.DBusException, e:
+ self.failure = ('SetProperty: DBus exception %s for set of %s' %
+ (e, key))
+ return None
service = svc
if self.scan_timeout is not None:
gobject.source_remove(self.scan_timeout)
« no previous file with comments | « server/site_tests/network_WiFiManager/040MutiAuth ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698