fix tests

pioreactor/automations/__init__.py

@@ -36,4 +36,4 @@ def execute(self) -> Optional[events.AutomationEvent]:
         """
         Overwrite in subclass
         """
-        raise NotImplementedError("Overwrite in base class")
+        return None

pioreactor/tests/test_dosing_control.py

@@ -350,8 +350,8 @@ def test_changing_parameters_over_mqtt_with_unknown_parameter() -> None:
     assert any(["garbage" in log["message"] for log in bucket])
 
 
-def test_pause_in_dosing_automation() -> None:
-    experiment = "test_pause_in_dosing_automation"
+def test_pause_in_dosing_automation_wont_work_use_controller_instead() -> None:
+    experiment = "test_pause_in_dosing_automation_wont_work_use_controller_instead"
     with DosingAutomationJob(
         target_growth_rate=0.05,
         target_od=1.0,
@@ -362,10 +362,6 @@ def test_pause_in_dosing_automation() -> None:
         pause()
         pubsub.publish(f"pioreactor/{unit}/{experiment}/dosing_automation/$state/set", "sleeping")
         pause()
-        assert algo.state == "sleeping"
-
-        pubsub.publish(f"pioreactor/{unit}/{experiment}/dosing_automation/$state/set", "ready")
-        pause()
         assert algo.state == "ready"
 
 
@@ -972,8 +968,8 @@ def test_changing_algo_over_mqtt_with_wrong_type_is_okay() -> None:
 def test_disconnect_cleanly() -> None:
     experiment = "test_disconnect_cleanly"
     algo = DosingController(
-        experiment,
         unit,
+        experiment,
         "turbidostat",
         target_normalized_od=1.0,
         duration=50,