fixup: drain executor at exit and relax non-blocking test timing margin

Signed-off-by: Todd Baert <todd.baert@dynatrace.com>

Author: toddbaert

openfeature/_event_support.py

@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import atexit
 import threading
 import typing
 from collections import defaultdict
@@ -20,6 +21,7 @@
 
 logger = getLogger("openfeature")
 _event_executor = ThreadPoolExecutor(thread_name_prefix="openfeature-event-handler")
+atexit.register(_event_executor.shutdown, wait=True)
 
 _global_lock = threading.RLock()
 _global_handlers: dict[ProviderEvent, list[EventHandler]] = defaultdict(list)

tests/test_client.py

@@ -643,7 +643,8 @@ def slow_handler(details):
 
     # Then
     assert handler_started.wait(timeout=1)
-    assert elapsed < 0.2
+    # emit must return well before the handler's blocking wait (1s) would finish
+    assert elapsed < 0.5
     release_handler.set()
     assert handler_finished.wait(timeout=1)