commit d89b528e0d664f8bdadbbfe669c6629abfe0f6c0
author Christopher Wiley <wiley@chromium.org> Wed Apr 10 15:21:26 2013 -0700
committer ChromeBot <chrome-bot@google.com> Wed Apr 10 20:28:09 2013 -0700
tree 8c87e89081fbbe1b754340ae2dd679b9fe51ae39
parent b42f9b5a8a16405b53cbdcf4f4b40235c1de3ec6

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>

server/site_linux_router.py

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']: