autotest: Make params optional for site_linux_router deconfig/destroy
This lets us not pass empty dictionaries about in future tests.
TEST=Ran network_WiFiMatFunc/000Check11b which performs both operations.
BUG=None
Change-Id: Ic1bf328391bdaec34fc4eacd16ce160a517b1e7d
Reviewed-on: https://gerrit.chromium.org/gerrit/47810
Tested-by: Christopher Wiley <wiley@chromium.org>
Reviewed-by: Paul Stewart <pstew@chromium.org>
Commit-Queue: Christopher Wiley <wiley@chromium.org>
diff --git a/server/site_linux_router.py b/server/site_linux_router.py
index dc48fdd..71a5d65 100644
--- a/server/site_linux_router.py
+++ b/server/site_linux_router.py
@@ -126,7 +126,7 @@
}[device_type]
- def destroy(self, params):
+ def destroy(self, params={}):
""" Destroy a previously created device """
self.deconfig(params)
self.hostapd['conf'] = self.default_config.copy()
@@ -205,7 +205,7 @@
# remove non-hostapd config item from params
params.pop('multi_interface')
elif self.hostapd['configured'] or self.station['configured']:
- self.deconfig({})
+ self.deconfig()
local_server = params.pop('local_server', False)
@@ -378,7 +378,7 @@
raise NotImplementedError("non-ibss station")
if self.station['configured'] or self.hostapd['configured']:
- self.deconfig({})
+ self.deconfig()
local_server = params.pop('local_server', False)
mode = None
@@ -504,7 +504,7 @@
return parts[parts.index('link/ether') + 1]
- def deconfig(self, params):
+ def deconfig(self, params={}):
""" De-configure the AP (will also bring wlan down) """
if not self.hostapd['configured'] and not self.station['configured']: