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

Unified Diff: server/site_wifitest.py

Issue 1736001: Rolled up change to wireless test suite for OpenWRT. (Closed)
Patch Set: Consistent state Created 10 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_linux_router.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: server/site_wifitest.py
diff --git a/server/site_wifitest.py b/server/site_wifitest.py
index 0fb080a89d0b9aa5bea8b68a2a07e374a5800a5c..49712bb8e6cb4dd872ab22bf234a5c885d0e3b98 100644
--- a/server/site_wifitest.py
+++ b/server/site_wifitest.py
@@ -66,8 +66,8 @@ class WiFiTest(object):
def __init__(self, name, steps, config):
self.name = name
self.steps = steps
-
router = config['router']
+
self.router = hosts.create_host(router['addr'])
# NB: truncate SSID to 32 characters
self.defssid = self.__get_defssid(router['addr'])[0:32]
@@ -76,7 +76,7 @@ class WiFiTest(object):
# auto-detect router type
if site_linux_router.isLinuxRouter(self.router):
router['type'] = 'linux'
- if site_bsd_router.isBSDRouter(self.router):
+ elif site_bsd_router.isBSDRouter(self.router):
router['type'] = 'bsd'
else:
raise Exception('Unable to autodetect router type')
@@ -170,13 +170,15 @@ class WiFiTest(object):
def __get_connect_script(self, params):
+ # Something is off in our current setup so we'll set the assoc_timeout
+ # to 60. When all is fixed, it should go back to 15.
return '''
import dbus, dbus.mainloop.glib, gobject, logging, re, sys, time
ssid = "''' + params['ssid'] + '''"
security = "''' + params['security'] + '''"
psk = "''' + params.get('psk', "") + '''"
-assoc_timeout = ''' + params.get('assoc_timeout', "15") + '''
+assoc_timeout = ''' + params.get('assoc_timeout', "60") + '''
config_timeout = ''' + params.get('config_timeout', "15") + '''
bus_loop = dbus.mainloop.glib.DBusGMainLoop(set_as_default=True)
@@ -439,9 +441,9 @@ sys.exit(0)'''
""" Ping the server from the client """
ping_ip = params.get('ping_ip', self.server_wifi_ip)
count = params.get('count', 10)
- # set timeout for 3s / ping packet
+ # set timeout for 5s / ping packet
result = self.client.run("ping %s %s" % \
- (self.__ping_args(params), ping_ip), timeout=3*int(count))
+ (self.__ping_args(params), ping_ip), timeout=5*int(count))
self.__print_pingstats("client_ping ",
self.__get_pingstats(result.stdout))
@@ -469,9 +471,9 @@ sys.exit(0)'''
return
ping_ip = params.get('ping_ip', self.client_wifi_ip)
count = params.get('count', 10)
- # set timeout for 3s / ping packet
+ # set timeout for 5s / ping packet
result = self.server.run("ping %s %s" % \
- (self.__ping_args(params), ping_ip), timeout=3*int(count))
+ (self.__ping_args(params), ping_ip), timeout=5*int(count))
self.__print_pingstats("server_ping ",
self.__get_pingstats(result.stdout))
« no previous file with comments | « server/site_linux_router.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698