diff mbox series

[1/2] tests: fst: leave time to process session request

Message ID 20240124171247.b36a9fa85ce7.I4822967ade1cd8adfba6b85e0ba833687014f169@changeid
State Accepted
Headers show
Series [1/2] tests: fst: leave time to process session request | expand

Commit Message

Johannes Berg Jan. 24, 2024, 4:12 p.m. UTC 
From: Johannes Berg <johannes.berg@intel.com>

Due to scheduling in UML time-travel, the test may continue
running and find that the failure didn't trigger when really
the frame just didn't make it through to the other side. Add
some time for the necessary processing.

Signed-off-by: Johannes Berg <johannes.berg@intel.com>
---
 tests/hwsim/test_fst_module.py | 3 +++
 1 file changed, 3 insertions(+)

Comments

Jouni Malinen Jan. 25, 2024, 6:25 p.m. UTC | #1 
On Wed, Jan 24, 2024 at 05:12:48PM +0100, Johannes Berg wrote:
> Due to scheduling in UML time-travel, the test may continue
> running and find that the failure didn't trigger when really
> the frame just didn't make it through to the other side. Add
> some time for the necessary processing.

Thanks, both patches applied.
diff mbox series

Patch

diff --git a/tests/hwsim/test_fst_module.py b/tests/hwsim/test_fst_module.py
index 4dbcfc890a74..8093ebff0fff 100644
--- a/tests/hwsim/test_fst_module.py
+++ b/tests/hwsim/test_fst_module.py
@@ -2211,6 +2211,9 @@  def test_fst_session_oom(dev, apdev, test_params):
             res = initiator.grequest("FST-MANAGER SESSION_INITIATE " + sid)
             if not res.startswith("OK"):
                 raise Exception("Unexpected SESSION_INITIATE result")
+            # leave some time for the frame to be transmitted, received
+            # and processed
+            time.sleep(0.1)
     finally:
         fst_module_aux.disconnect_two_ap_sta_pairs(ap1, ap2, sta1, sta2)
         fst_module_aux.stop_two_ap_sta_pairs(ap1, ap2, sta1, sta2)