chore: Assert readiness directly instead of waiting on a timeout (#532)

Author: jsonbailey

ldclient/testing/impl/datasource/test_polling_processor.py

@@ -165,7 +165,7 @@ def test_unexpected_http_error_moves_to_the_extended_regime(ignore_mock):
 
     # The extended regime starts at five minutes, so only the first poll runs.
     wait_until(lambda: retry.next_delay > 0.1)
-    assert not ready.wait(0.1)
+    assert not ready.is_set()
     assert mock_requester.request_count == 1
 
 

ldclient/testing/impl/datasource/test_streaming.py

@@ -315,7 +315,7 @@ def test_unexpected_http_error_backs_off_a_long_way(status):
                 wait_until(lambda: sp._retry.next_delay > NORMAL_STREAMING_CEILING_DELAY)
 
                 # Initialization is not falsely unblocked.
-                assert not ready.wait(0.1)
+                assert not ready.is_set()
                 assert not sp.initialized()
                 assert sp.is_alive()
                 server.should_have_requests(1)
@@ -927,7 +927,7 @@ def test_failure_transitions_from_valid():
             wait_until(lambda: len(spy.statuses) == 2)
 
             # The 401 is retried five minutes out, so readiness never fires.
-            assert not ready.wait(0.1)
+            assert not ready.is_set()
             server.should_have_requests(1)
 
             assert len(spy.statuses) == 2