tests: Make dbus_p2p_group_termination_by_go more robust

Set peer_group_removed only if peer_group_added has already been set.
This fixes an issue where a propertiesChanged event triggered by an
earlier test case was able to get dbus_p2p_group_termination_by_go
terminated too early. This happened, e.g., with sequence
"dbus_p2p_two_groups dbus_p2p_group_termination_by_go".

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

diff --git a/tests/hwsim/test_dbus.py b/tests/hwsim/test_dbus.py
index ff888b78e36a7ae91a0dbf63ab738d1d5335ee12..650c0d91547f05ef5fc03748f0135a993049f72a 100644 (file)
--- a/tests/hwsim/test_dbus.py
+++ b/tests/hwsim/test_dbus.py
@@ -4237,7 +4237,7 @@ def test_dbus_p2p_group_termination_by_go(dev, apdev):
                 return
             if len(changed_properties["Groups"]) > 0:
                 self.peer_group_added = True
-            if len(changed_properties["Groups"]) == 0:
+            if len(changed_properties["Groups"]) == 0 and self.peer_group_added:
                 self.peer_group_removed = True
                 self.loop.quit()