diff options
| author | Roshan Pius <rpius@google.com> | 2018-05-15 15:38:27 -0700 |
|---|---|---|
| committer | Roshan Pius <rpius@google.com> | 2018-05-15 15:46:14 -0700 |
| commit | 044ebed17342b5234f41f45d6a79971a8137a4e7 (patch) | |
| tree | cf870a62175da047887c9c6cc0e60232e656ca33 | |
| parent | 35d297eb67831cde25e62b51499852887edbe938 (diff) | |
| download | platform_tools_test_connectivity-044ebed17342b5234f41f45d6a79971a8137a4e7.tar.gz platform_tools_test_connectivity-044ebed17342b5234f41f45d6a79971a8137a4e7.tar.bz2 platform_tools_test_connectivity-044ebed17342b5234f41f45d6a79971a8137a4e7.zip | |
WifiStaApConcurrencyTest: Fix a bug in setup
Need to move the supported models check after unpacking of user params.
These changes were present locally, but missed in the previous commit.
Bug: 72639832
Test: `act.py -c softap_cross_onhub.config -tb dut-name -tc
Change-Id: I31b82622600b19155f4d625dbf19cb69f433909b
WifiStaApConcurrencyTest:test_wifi_connection_5G_DFS_softap_5G`
| -rwxr-xr-x | acts/tests/google/wifi/WifiStaApConcurrencyTest.py | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/acts/tests/google/wifi/WifiStaApConcurrencyTest.py b/acts/tests/google/wifi/WifiStaApConcurrencyTest.py index ff55819215..6f0beba430 100755 --- a/acts/tests/google/wifi/WifiStaApConcurrencyTest.py +++ b/acts/tests/google/wifi/WifiStaApConcurrencyTest.py @@ -53,10 +53,6 @@ class WifiStaApConcurrencyTest(WifiBaseTest): self.dut_client = self.android_devices[1] wutils.wifi_test_device_init(self.dut) wutils.wifi_test_device_init(self.dut_client) - if self.dut.model not in self.dbs_supported_models: - asserts.skip( - ("Device %s does not support dual interfaces.") - % self.dut.model) # Do a simple version of init - mainly just sync the time and enable # verbose logging. This test will fail if the DUT has a sim and cell # data is disabled. We would also like to test with phones in less @@ -76,11 +72,16 @@ class WifiStaApConcurrencyTest(WifiBaseTest): asserts.assert_equal(self.dut_client.droid.wifiGetVerboseLoggingLevel(), 1, "Failed to enable WiFi verbose logging on the client dut.") - req_params = ["AccessPoint"] + req_params = ["AccessPoint", "dbs_supported_models"] opt_param = ["iperf_server_address"] self.unpack_userparams( req_param_names=req_params, opt_param_names=opt_param) + if self.dut.model not in self.dbs_supported_models: + asserts.skip( + ("Device %s does not support dual interfaces.") + % self.dut.model) + if "iperf_server_address" in self.user_params: self.iperf_server = self.iperf_servers[0] if hasattr(self, 'iperf_server'): |