Use daemon argument to ensure that polling manager is killed (#47)

Author: matthewelwell

flagsmith/flagsmith.py

@@ -96,6 +96,7 @@ def __init__(
                 EnvironmentDataPollingManager(
                     main=self,
                     refresh_interval_seconds=environment_refresh_interval_seconds,
+                    daemon=True,  # noqa
                 )
             )
             self.environment_data_polling_manager_thread.start()

flagsmith/polling_manager.py

@@ -8,9 +8,13 @@
 
 class EnvironmentDataPollingManager(threading.Thread):
     def __init__(
-        self, main: "Flagsmith", refresh_interval_seconds: typing.Union[int, float] = 10
+        self,
+        *args,
+        main: "Flagsmith",
+        refresh_interval_seconds: typing.Union[int, float] = 10,
+        **kwargs
     ):
-        super(EnvironmentDataPollingManager, self).__init__()
+        super(EnvironmentDataPollingManager, self).__init__(*args, **kwargs)
         self._stop_event = threading.Event()
         self.main = main
         self.refresh_interval_seconds = refresh_interval_seconds

tests/conftest.py

@@ -57,7 +57,7 @@ def local_eval_flagsmith(server_api_key, environment_json, mocker):
 
     yield flagsmith
 
-    flagsmith.__del__()
+    del flagsmith
 
 
 @pytest.fixture()