refactor(api): Delete unused action_dispatcher argument (#14954)

Opentrons · Apr 19, 2024 · e877f3a · e877f3a
1 parent 6d91a56
commit e877f3a
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 17 deletions.
diff --git a/api/src/opentrons/protocol_engine/execution/create_queue_worker.py b/api/src/opentrons/protocol_engine/execution/create_queue_worker.py
@@ -39,7 +39,6 @@ def create_queue_worker(
     equipment_handler = EquipmentHandler(
         hardware_api=hardware_api,
         state_store=state_store,
-        action_dispatcher=action_dispatcher,
     )
 
     movement_handler = MovementHandler(

diff --git a/api/src/opentrons/protocol_engine/execution/equipment.py b/api/src/opentrons/protocol_engine/execution/equipment.py
@@ -22,7 +22,6 @@
     TemperatureModuleId,
     ThermocyclerModuleId,
 )
-from ..actions import ActionDispatcher
 from ..errors import (
     FailedToLoadPipetteError,
     LabwareDefinitionDoesNotExistError,
@@ -99,7 +98,6 @@ def __init__(
         self,
         hardware_api: HardwareControlAPI,
         state_store: StateStore,
-        action_dispatcher: ActionDispatcher,
         labware_data_provider: Optional[LabwareDataProvider] = None,
         module_data_provider: Optional[ModuleDataProvider] = None,
         model_utils: Optional[ModelUtils] = None,
@@ -110,7 +108,6 @@ def __init__(
         """Initialize an EquipmentHandler instance."""
         self._hardware_api = hardware_api
         self._state_store = state_store
-        self._action_dispatcher = action_dispatcher
         self._labware_data_provider = labware_data_provider or LabwareDataProvider()
         self._module_data_provider = module_data_provider or ModuleDataProvider()
         self._model_utils = model_utils or ModelUtils()

diff --git a/api/tests/opentrons/protocol_engine/execution/test_equipment_handler.py b/api/tests/opentrons/protocol_engine/execution/test_equipment_handler.py
@@ -22,7 +22,6 @@
 from opentrons.protocols.models import LabwareDefinition
 
 from opentrons.protocol_engine import errors
-from opentrons.protocol_engine.actions import ActionDispatcher
 from opentrons.protocol_engine.types import (
     DeckSlotLocation,
     DeckType,
@@ -84,12 +83,6 @@ def state_store(decoy: Decoy) -> StateStore:
     return decoy.mock(cls=StateStore)
 
 
-@pytest.fixture
-def action_dispatcher(decoy: Decoy) -> ActionDispatcher:
-    """Get a mocked out ActionDispatcher instance."""
-    return decoy.mock(cls=ActionDispatcher)
-
-
 @pytest.fixture
 def model_utils(decoy: Decoy) -> ModelUtils:
     """Get a mocked out ModelUtils instance."""
@@ -166,7 +159,6 @@ def virtual_pipette_data_provider(
 def subject(
     hardware_api: HardwareControlAPI,
     state_store: StateStore,
-    action_dispatcher: ActionDispatcher,
     labware_data_provider: LabwareDataProvider,
     module_data_provider: ModuleDataProvider,
     model_utils: ModelUtils,
@@ -176,7 +168,6 @@ def subject(
     return EquipmentHandler(
         hardware_api=hardware_api,
         state_store=state_store,
-        action_dispatcher=action_dispatcher,
         labware_data_provider=labware_data_provider,
         module_data_provider=module_data_provider,
         model_utils=model_utils,
@@ -614,7 +605,6 @@ async def test_load_pipette(
     model_utils: ModelUtils,
     hardware_api: HardwareControlAPI,
     state_store: StateStore,
-    action_dispatcher: ActionDispatcher,
     loaded_static_pipette_data: LoadedStaticPipetteData,
     subject: EquipmentHandler,
 ) -> None:
@@ -665,7 +655,6 @@ async def test_load_pipette_96_channels(
     model_utils: ModelUtils,
     hardware_api: HardwareControlAPI,
     state_store: StateStore,
-    action_dispatcher: ActionDispatcher,
     loaded_static_pipette_data: LoadedStaticPipetteData,
     subject: EquipmentHandler,
 ) -> None:
@@ -702,7 +691,6 @@ async def test_load_pipette_uses_provided_id(
     decoy: Decoy,
     hardware_api: HardwareControlAPI,
     state_store: StateStore,
-    action_dispatcher: ActionDispatcher,
     loaded_static_pipette_data: LoadedStaticPipetteData,
     subject: EquipmentHandler,
 ) -> None:
@@ -734,7 +722,6 @@ async def test_load_pipette_use_virtual(
     decoy: Decoy,
     model_utils: ModelUtils,
     state_store: StateStore,
-    action_dispatcher: ActionDispatcher,
     loaded_static_pipette_data: LoadedStaticPipetteData,
     subject: EquipmentHandler,
     virtual_pipette_data_provider: pipette_data_provider.VirtualPipetteDataProvider,