commit 5cd6d3c8fabc02477695a548a2e26a6d24826ae7
author Ang Li <angli@google.com> Mon Feb 01 11:29:14 2016 -0800
committer Ang Li <angli@google.com> Mon Feb 01 13:27:05 2016 -0800
tree 12d7e48fcfa4bdf66d6a5fbe886067c5d8d5dc57
parent 930fa0e2e0b27000879723616f1b9a7ba4f2c26e

Standardize unpack_userparam.

Bug=26472080

Raise exceptions instead of return True/False.
Update all related scripts to use the new convention.

Change-Id: I7d8486bfcbc2a304c300e01c248550b5d6bf1e4a

acts/tests/google/wifi/WifiEnterpriseTest.py

diff --git a/acts/tests/google/wifi/WifiEnterpriseTest.py b/acts/tests/google/wifi/WifiEnterpriseTest.py
index fbdd02f..3f35725 100755
--- a/acts/tests/google/wifi/WifiEnterpriseTest.py
+++ b/acts/tests/google/wifi/WifiEnterpriseTest.py
@@ -70,7 +70,7 @@
             "roaming_consortium_ids",
             "plmn"
         )
-        assert self.unpack_userparams(required_userparam_names,
+        self.unpack_userparams(required_userparam_names,
                     opt_param_names = optional_userparam_names)
         # Default configs for EAP networks.
         self.config_peap = {
@@ -125,7 +125,6 @@
         del self.config_passpoint_ttls[WifiEnums.SSID_KEY]
         # Set screen lock password so ConfigStore is unlocked.
         self.droid.setDevicePassword(self.device_password)
-        return True
 
     def teardown_class(self):
         wutils.reset_wifi(self.dut)
@@ -138,7 +137,6 @@
         self.droid.wakeUpNow()
         wutils.reset_wifi(self.dut)
         self.ed.clear_all_events()
-        return True
 
     def teardown_test(self):
         self.droid.wakeLockRelease()