oops, last commit's diagnosis was incorrect. The timing test issue came from async dbus handling interfering with fast setup/teardown of automated tests. Revert the last change and fix by setting up the dbus signal subscription immediately upon getting the dbus connection.

1 files changed, 1 insertions, 1 deletions

diff --git a/tests/unit/usb-snap-test.cpp b/tests/unit/usb-snap-test.cpp
index 4300c47..84555cc 100644
--- a/tests/unit/usb-snap-test.cpp
+++ b/tests/unit/usb-snap-test.cpp
@@ -163,7 +163,7 @@ TEST_F(UsbSnapFixture, TestRoundTrip)
             QVariantList() << id << test.action_to_invoke);
 
         // test that UsbSnap emits on_user_response() as a result
-        wait_for([&user_response_set](){return user_response_set;}, 2000);
+        wait_for([&user_response_set](){return user_response_set;});
         EXPECT_TRUE(user_response_set);
         ASSERT_EQ(test.expected_response, user_response);