fix more tests

Author: CamDavidsonPilon

pioreactor/background_jobs/base.py

@@ -101,15 +101,7 @@ def logger(self):
 
     def __del__(self):
         if self._logger:
-            for handler in self._logger.handlers[:]:
-                try:
-                    handler.close()
-                except Exception:
-                    pass  # Don't raise in destructor
-                try:
-                    self._logger.removeHandler(handler)
-                except Exception:
-                    pass
+            self._logger.clean_up()
 
 
 class PostInitCaller(type):

pioreactor/tests/test_background_job.py

@@ -442,6 +442,7 @@ def action_to_do_after_od_reading(self) -> None:
                     assert len(bucket) > 4, bucket
 
         ODReader._pre_read = []
+        ODReader._post_read = []
 
 
 def test_dodging_when_od_reading_stops_first() -> None:

pioreactor/tests/test_stirring.py

@@ -131,7 +131,7 @@ def test_publish_measured_rpm() -> None:
 
 
 def test_rpm_isnt_updated_if_there_is_no_rpm_measurement() -> None:
-    exp = "test_publish_measured_rpm"
+    exp = "test_rpm_isnt_updated_if_there_is_no_rpm_measurement"
 
     publish(f"pioreactor/{unit}/{exp}/stirring/measured_rpm", None, retain=True)
     pause()

pioreactor/utils/__init__.py

@@ -343,7 +343,7 @@ def __enter__(self):
         self.cursor = self.conn.cursor()
         self.cursor.executescript(
             """
-            PRAGMA busy_timeout = 30000;
+            PRAGMA busy_timeout = 5000;
             PRAGMA synchronous = 1; -- aka NORMAL, recommended when using WAL
             PRAGMA temp_store = 2;  -- stop writing small files to disk, use mem
             PRAGMA foreign_keys = ON;
@@ -632,7 +632,7 @@ def __init__(self) -> None:
         self.conn = sqlite3.connect(db_path, isolation_level=None)
         self.conn.executescript(
             """
-            PRAGMA busy_timeout = 30000;
+            PRAGMA busy_timeout = 5000;
             PRAGMA synchronous = NORMAL;
             PRAGMA temp_store = 2;
             PRAGMA foreign_keys = ON;