More comment tweaking.

Opentrons · Dec 4, 2024 · 9fb0d43 · 9fb0d43
1 parent 20becaf
commit 9fb0d43
Show file tree

Hide file tree

Showing 16 changed files with 26 additions and 16 deletions.
diff --git a/api/tests/opentrons/protocol_engine/state/test_addressable_area_store_old.py b/api/tests/opentrons/protocol_engine/state/test_addressable_area_store_old.py
@@ -2,7 +2,7 @@
 
 DEPRECATED: Testing AddressableAreaStore independently of AddressableAreaView is no
 longer helpful. Try to add new tests to test_addressable_area_state.py, where they can be
-tested together.
+tested together, treating AddressableAreaState as a private implementation detail.
 """
 
 import pytest

diff --git a/api/tests/opentrons/protocol_engine/state/test_addressable_area_view_old.py b/api/tests/opentrons/protocol_engine/state/test_addressable_area_view_old.py
@@ -2,7 +2,7 @@
 
 DEPRECATED: Testing AddressableAreaView independently of AddressableAreaStore is no
 longer helpful. Try to add new tests to test_addressable_area_state.py, where they can be
-tested together.
+tested together, treating AddressableAreaState as a private implementation detail.
 """
 
 import inspect

diff --git a/api/tests/opentrons/protocol_engine/state/test_command_store_old.py b/api/tests/opentrons/protocol_engine/state/test_command_store_old.py
@@ -1,7 +1,8 @@
 """Tests for CommandStore.
 
 DEPRECATED: Testing CommandStore independently of CommandView is no longer helpful.
-Try to add new tests to test_command_state.py, where they can be tested together.
+Try to add new tests to test_command_state.py, where they can be tested together,
+treating CommandState as a private implementation detail.
 """
 
 

diff --git a/api/tests/opentrons/protocol_engine/state/test_command_view_old.py b/api/tests/opentrons/protocol_engine/state/test_command_view_old.py
@@ -1,7 +1,8 @@
 """Tests for CommandView.
 
 DEPRECATED: Testing CommandView independently of CommandStore is no longer helpful.
-Try to add new tests to test_command_state.py, where they can be tested together.
+Try to add new tests to test_command_state.py, where they can be tested together,
+treating CommandState as a private implementation detail.
 """
 
 

diff --git a/api/tests/opentrons/protocol_engine/state/test_labware_store_old.py b/api/tests/opentrons/protocol_engine/state/test_labware_store_old.py
@@ -2,7 +2,7 @@
 
 DEPRECATED: Testing LabwareStore independently of LabwareView is no
 longer helpful. Try to add new tests to test_labware_state.py, where they can be
-tested together.
+tested together, treating LabwareState as a private implementation detail.
 """
 from typing import Optional
 from opentrons.protocol_engine.state import update_types

diff --git a/api/tests/opentrons/protocol_engine/state/test_labware_view_old.py b/api/tests/opentrons/protocol_engine/state/test_labware_view_old.py
@@ -2,7 +2,7 @@
 
 DEPRECATED: Testing LabwareView independently of LabwareStore is no
 longer helpful. Try to add new tests to test_labware_state.py, where they can be
-tested together.
+tested together, treating LabwareState as a private implementation detail.
 """
 import pytest
 from datetime import datetime

diff --git a/api/tests/opentrons/protocol_engine/state/test_liquid_class_store_old.py b/api/tests/opentrons/protocol_engine/state/test_liquid_class_store_old.py
@@ -2,7 +2,7 @@
 
 DEPRECATED: Testing LiquidClassStore independently of LiquidClassView is no
 longer helpful. Try to add new tests to test_liquid_class_state.py, where they can be
-tested together.
+tested together, treating LiquidClassState as a private implementation detail.
 """
 import pytest
 

diff --git a/api/tests/opentrons/protocol_engine/state/test_liquid_class_view_old.py b/api/tests/opentrons/protocol_engine/state/test_liquid_class_view_old.py
@@ -2,7 +2,7 @@
 
 DEPRECATED: Testing LiquidClassView independently of LiquidClassStore is no
 longer helpful. Try to add new tests to test_liquid_class_state.py, where they can be
-tested together.
+tested together, treating LiquidClassState as a private implementation detail.
 """
 import pytest
 

diff --git a/api/tests/opentrons/protocol_engine/state/test_liquid_store_old.py b/api/tests/opentrons/protocol_engine/state/test_liquid_store_old.py
@@ -1,7 +1,8 @@
 """Liquid state store tests.
 
 DEPRECATED: Testing LiquidStore independently of LiquidView is no longer helpful.
-Try to add new tests to test_liquid_state.py, where they can be tested together.
+Try to add new tests to test_liquid_state.py, where they can be tested together,
+treating LiquidState as a private implementation detail.
 """
 import pytest
 from opentrons.protocol_engine.state.liquids import LiquidStore

diff --git a/api/tests/opentrons/protocol_engine/state/test_liquid_view_old.py b/api/tests/opentrons/protocol_engine/state/test_liquid_view_old.py
@@ -1,7 +1,8 @@
 """Liquid view tests.
 
 DEPRECATED: Testing LiquidView independently of LiquidStore is no longer helpful.
-Try to add new tests to test_liquid_state.py, where they can be tested together.
+Try to add new tests to test_liquid_state.py, where they can be tested together,
+treating LiquidState as a private implementation detail.
 """
 import pytest
 

diff --git a/api/tests/opentrons/protocol_engine/state/test_module_store_old.py b/api/tests/opentrons/protocol_engine/state/test_module_store_old.py
@@ -1,7 +1,8 @@
 """Module state store tests.
 
 DEPRECATED: Testing ModuleStore independently of ModuleView is no longer helpful.
-Try to add new tests to test_module_state.py, where they can be tested together.
+Try to add new tests to test_module_state.py, where they can be tested together,
+treating ModuleState as a private implementation detail.
 """
 from typing import List, Set, cast, Dict, Optional
 

diff --git a/api/tests/opentrons/protocol_engine/state/test_module_view_old.py b/api/tests/opentrons/protocol_engine/state/test_module_view_old.py
@@ -1,7 +1,8 @@
 """Tests for module state accessors in the protocol engine state store.
 
 DEPRECATED: Testing ModuleView independently of ModuleView is no longer helpful.
-Try to add new tests to test_module_state.py, where they can be tested together.
+Try to add new tests to test_module_state.py, where they can be tested together,
+treating ModuleState as a private implementation detail.
 """
 import pytest
 from math import isclose

diff --git a/api/tests/opentrons/protocol_engine/state/test_pipette_store_old.py b/api/tests/opentrons/protocol_engine/state/test_pipette_store_old.py
@@ -1,7 +1,8 @@
 """Tests for pipette state changes in the protocol_engine state store.
 
 DEPRECATED: Testing PipetteStore independently of PipetteView is no longer helpful.
-Try to add new tests to test_pipette_state.py, where they can be tested together.
+Try to add new tests to test_pipette_state.py, where they can be tested together,
+treating PipetteState as a private implementation detail.
 """
 import pytest
 

diff --git a/api/tests/opentrons/protocol_engine/state/test_pipette_view_old.py b/api/tests/opentrons/protocol_engine/state/test_pipette_view_old.py
@@ -1,7 +1,8 @@
 """Tests for pipette state accessors in the protocol_engine state store.
 
 DEPRECATED: Testing PipetteView independently of PipetteStore is no longer helpful.
-Try to add new tests to test_pipette_state.py, where they can be tested together.
+Try to add new tests to test_pipette_state.py, where they can be tested together,
+treating PipetteState as a private implementation detail.
 """
 
 from collections import OrderedDict

diff --git a/api/tests/opentrons/protocol_engine/state/test_well_store_old.py b/api/tests/opentrons/protocol_engine/state/test_well_store_old.py
@@ -1,7 +1,8 @@
 """Well state store tests.
 
 DEPRECATED: Testing WellStore independently of WellView is no longer helpful.
-Try to add new tests to well_state.py, where they can be tested together.
+Try to add new tests to well_state.py, where they can be tested together,
+treating WellState as a private implementation detail.
 """
 
 import pytest

diff --git a/api/tests/opentrons/protocol_engine/state/test_well_view_old.py b/api/tests/opentrons/protocol_engine/state/test_well_view_old.py
@@ -1,7 +1,8 @@
 """Well view tests.
 
 DEPRECATED: Testing WellView independently of WellStore is no longer helpful.
-Try to add new tests to test_well_state.py, where they can be tested together.
+Try to add new tests to test_well_state.py, where they can be tested together,
+treating WellState as a private implementation detail.
 """
 from datetime import datetime
 from opentrons.protocol_engine.types import (