Revert "fixed tests"

This reverts commit 3324fb3.
Opentrons · Feb 9, 2024 · 0bd76ff · 0bd76ff
1 parent 8a74130
commit 0bd76ff
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/api/src/opentrons/protocol_engine/state/commands.py b/api/src/opentrons/protocol_engine/state/commands.py
@@ -273,7 +273,7 @@ def handle_action(self, action: Action) -> None:  # noqa: C901
                 error=action.error,
             )
             prev_entry = self._state.commands_by_id[action.command_id]
-            failed_command_entry = CommandEntry(
+            self._state.commands_by_id[action.command_id] = CommandEntry(
                 index=prev_entry.index,
                 # TODO(mc, 2022-06-06): add new "cancelled" status or similar
                 # and don't set `completedAt` in commands other than the
@@ -286,8 +286,8 @@ def handle_action(self, action: Action) -> None:  # noqa: C901
                     }
                 ),
             )
-            self._state.commands_by_id[action.command_id] = failed_command_entry
-            self._state.failed_command = failed_command_entry
+
+            self._state.failed_command = prev_entry
             if prev_entry.command.intent == CommandIntent.SETUP:
                 other_command_ids_to_fail = [
                     *[i for i in self._state.queued_setup_command_ids],

diff --git a/api/tests/opentrons/protocol_engine/state/test_command_view.py b/api/tests/opentrons/protocol_engine/state/test_command_view.py
@@ -816,7 +816,6 @@ def test_get_slice_default_cursor_failed_command() -> None:
     subject = get_command_view(
         commands=[command_1, command_2, command_3, command_4],
         run_result=RunResult.FAILED,
-        failed_command=CommandEntry(index=2, command=command_3),
     )
 
     result = subject.get_slice(cursor=None, length=3)

diff --git a/notify-server/pyproject.toml b/notify-server/pyproject.toml