tests: Add a helper function for clearing scan cache

This makes it more convenient and consistent to clear the cached scan
results from cfg80211 and wpa_supplicant.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>

diff --git a/tests/hwsim/test_ap_hs20.py b/tests/hwsim/test_ap_hs20.py
index 7a44be3306557a1224f487ef208f0c9503f1c12d..fdb267d1f131c55efdbccbe8040888c242b8ff85 100644 (file)
--- a/tests/hwsim/test_ap_hs20.py
+++ b/tests/hwsim/test_ap_hs20.py
@@ -476,6 +476,7 @@ def test_ap_hs20_connect_api(dev, apdev):
     wpas = WpaSupplicant(global_iface='/tmp/wpas-wlan5')
     wpas.interface_add("wlan5", drv_params="force_connect_cmd=1")
     wpas.hs20_enable()
+    wpas.flush_scan_cache()
     id = wpas.add_cred_values({ 'realm': "example.com",
                                   'username': "hs20-test",
                                   'password': "password",
@@ -2114,7 +2115,7 @@ def test_ap_hs20_random_mac_addr(dev, apdev):
     wpas.request("SET preassoc_mac_addr 1")
     wpas.request("SET rand_addr_lifetime 60")
     wpas.hs20_enable()
-    wpas.scan(freq="2412", only_new=True)
+    wpas.flush_scan_cache()
     id = wpas.add_cred_values({ 'realm': "example.com",
                                   'username': "hs20-test",
                                   'password': "password",

diff --git a/tests/hwsim/test_ap_wps.py b/tests/hwsim/test_ap_wps.py
index 156d804d7a6436f24a4083a55db098635e1ebdfe..51a445b6228c8a6883014549dc6ad744b31105e4 100644 (file)
--- a/tests/hwsim/test_ap_wps.py
+++ b/tests/hwsim/test_ap_wps.py
@@ -556,11 +556,7 @@ def test_ap_wps_reg_config(dev, apdev):
 
     logger.info("Re-configure back to open")
     dev[0].request("REMOVE_NETWORK all")
-    dev[0].request("BSS_FLUSH 0")
-    dev[0].request("SCAN freq=2412 only_new=1")
-    ev = dev[0].wait_event(["CTRL-EVENT-SCAN-RESULTS"], 15)
-    if ev is None:
-        raise Exception("Scan timed out")
+    dev[0].flush_scan_cache()
     dev[0].dump_monitor()
     dev[0].wps_reg(apdev[0]['bssid'], appin, "wps-open", "OPEN", "NONE", "")
     status = dev[0].get_status()

diff --git a/tests/hwsim/test_p2p_channel.py b/tests/hwsim/test_p2p_channel.py
index 41e60d46cd68f9fc9146fc8d8e11b1607d24dcbf..600071ed80437cf8d29e1e94d02b5cb7a9d15a6f 100644 (file)
--- a/tests/hwsim/test_p2p_channel.py
+++ b/tests/hwsim/test_p2p_channel.py
@@ -207,10 +207,8 @@ def test_autogo_following_bss(dev, apdev):
 def test_go_neg_with_bss_connected(dev, apdev):
     """P2P channel selection: GO negotiation when station interface is connected"""
 
-    dev[0].request("BSS_FLUSH 0")
-    dev[0].request("SCAN freq=2412 only_new=1")
-    dev[1].request("BSS_FLUSH 0")
-    dev[1].request("SCAN freq=2412 only_new=1")
+    dev[0].flush_scan_cache()
+    dev[1].flush_scan_cache()
     dev[0].request("SET p2p_no_group_iface 0")
 
     hapd = hostapd.add_ap(apdev[0]['ifname'],
@@ -281,10 +279,8 @@ def test_go_neg_with_bss_on_disallowed_chan(dev, apdev):
             hapd = hostapd.add_ap(apdev[0]['ifname'],
                                   { "ssid": 'bss-2.4ghz', "channel": '1' })
             # make sure PBC overlap from old test cases is not maintained
-            dev[0].request("BSS_FLUSH 0")
-            dev[0].request("SCAN freq=2412 only_new=1")
-            dev[1].request("BSS_FLUSH 0")
-            dev[1].request("SCAN freq=2412 only_new=1")
+            dev[0].flush_scan_cache()
+            dev[1].flush_scan_cache()
             wpas.connect("bss-2.4ghz", key_mgmt="NONE", scan_freq="2412")
             wpas.request("P2P_SET disallow_freq 2412")
 
@@ -343,8 +339,7 @@ def test_go_neg_forced_freq_diff_than_bss_freq(dev, apdev):
            raise Exception("New radio does not support MCC")
 
         # Clear possible PBC session overlap from previous test case
-        dev[1].request("BSS_FLUSH 0")
-        dev[1].request("SCAN freq=2412 only_new=1")
+        dev[1].flush_scan_cache()
 
         wpas.request("SET p2p_no_group_iface 0")
 

diff --git a/tests/hwsim/test_p2p_concurrency.py b/tests/hwsim/test_p2p_concurrency.py
index 37b9ee016f4ca13342b033eefb9024f1775c2336..c143484fabdbaa9f8c258112751e10b7363425d2 100644 (file)
--- a/tests/hwsim/test_p2p_concurrency.py
+++ b/tests/hwsim/test_p2p_concurrency.py
@@ -119,8 +119,7 @@ def test_concurrent_grpform_while_connecting2(dev, apdev):
     logger.info("Start connection to an infrastructure AP")
     hapd = hostapd.add_ap(apdev[0]['ifname'], { "ssid": "test-open" })
     dev[0].connect("test-open", key_mgmt="NONE", wait_connect=False)
-    dev[1].request("BSS_FLUSH 0")
-    dev[1].scan(freq="2412", only_new=True)
+    dev[1].flush_scan_cache()
 
     logger.info("Form a P2P group while connecting to an AP")
     dev[0].request("SET p2p_no_group_iface 0")

diff --git a/tests/hwsim/test_p2p_grpform.py b/tests/hwsim/test_p2p_grpform.py
index 9e1d253abf0017f3ee6d7e783e8bfb6650e2239e..9a978cd422641d59da483ffe4407be2e3c2989e0 100644 (file)
--- a/tests/hwsim/test_p2p_grpform.py
+++ b/tests/hwsim/test_p2p_grpform.py
@@ -603,16 +603,16 @@ def test_go_neg_two_peers(dev):
 def clear_pbc_overlap(dev, ifname):
     hapd_global = hostapd.HostapdGlobal()
     hapd_global.remove(ifname)
+    dev[0].request("P2P_CANCEL")
+    dev[1].request("P2P_CANCEL")
     dev[0].p2p_stop_find()
     dev[1].p2p_stop_find()
     dev[0].dump_monitor()
     dev[1].dump_monitor()
     time.sleep(0.1)
-    dev[0].request("BSS_FLUSH 0")
-    dev[0].request("SCAN freq=2412 only_new=1")
-    dev[1].request("BSS_FLUSH 0")
-    dev[1].request("SCAN freq=2412 only_new=1")
-    time.sleep(1)
+    dev[0].flush_scan_cache()
+    dev[1].flush_scan_cache()
+    time.sleep(0.1)
 
 def test_grpform_pbc_overlap(dev, apdev):
     """P2P group formation during PBC overlap"""

diff --git a/tests/hwsim/wpasupplicant.py b/tests/hwsim/wpasupplicant.py
index e4f1de462fdb89e77891af373c3c2a7da09c7e95..6d809d81dd243eb071ea714aeaef9578b8d5fc9f 100644 (file)
--- a/tests/hwsim/wpasupplicant.py
+++ b/tests/hwsim/wpasupplicant.py
@@ -809,7 +809,7 @@ class WpaSupplicant:
         else:
             cmd = "SCAN"
         if freq:
-            cmd = cmd + " freq=" + freq
+            cmd = cmd + " freq=" + str(freq)
         if only_new:
             cmd += " only_new=1"
         if not no_wait:
@@ -831,6 +831,10 @@ class WpaSupplicant:
                 return
         raise Exception("Could not find BSS " + bssid + " in scan")
 
+    def flush_scan_cache(self):
+        self.request("BSS_FLUSH 0")
+        self.scan(freq=2412, only_new=True)
+
     def roam(self, bssid, fail_test=False):
         self.dump_monitor()
         if "OK" not in self.request("ROAM " + bssid):