Use placeholder implementation instead of raising NotImplementedError.

This seems a little more friendly to UI tests.
Opentrons · Oct 23, 2024 · c36ceef · c36ceef
1 parent 964d2c0
commit c36ceef
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/robot-server/robot_server/runs/error_recovery_mapping.py b/robot-server/robot_server/runs/error_recovery_mapping.py
@@ -105,7 +105,7 @@ def _map_error_recovery_type(reaction_if_match: ReactionIfMatch) -> ErrorRecover
         case ReactionIfMatch.ASSUME_FALSE_POSITIVE_AND_CONTINUE:
             # todo(mm, 2024-10-23): Connect to work in
             # https://github.com/Opentrons/opentrons/pull/16556.
-            raise NotImplementedError
+            return ErrorRecoveryType.IGNORE_AND_CONTINUE
         case ReactionIfMatch.FAIL_RUN:
             return ErrorRecoveryType.FAIL_RUN
         case ReactionIfMatch.WAIT_FOR_RECOVERY:

diff --git a/robot-server/robot_server/runs/run_controller.py b/robot-server/robot_server/runs/run_controller.py
@@ -103,7 +103,7 @@ def create_action(
             ):
                 # todo(mm, 2024-10-23): Connect to work in
                 # https://github.com/Opentrons/opentrons/pull/16556.
-                raise NotImplementedError
+                self._run_orchestrator_store.resume_from_recovery()
 
             else:
                 assert_never(action_type)