better comments

Opentrons · Apr 12, 2024 · c191867 · c191867
1 parent 4f8c39d
commit c191867
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/robot-server/robot_server/protocols/completed_analysis_store.py b/robot-server/robot_server/protocols/completed_analysis_store.py
@@ -349,13 +349,12 @@ async def add(self, completed_analysis_resource: CompletedAnalysisResource) -> N
         )
 
     def _make_room_for_new_analysis(self, protocol_id: str) -> None:
-        """Remove the oldest analysis in store if the number of analyses exceed the max allowed.
+        """Remove the oldest analyses in store if the number of analyses exceed the max allowed.
 
-        Unlike protocols, a protocol analysis IDs are not stored by any DB entities
+        Unlike protocols, protocol analysis IDs are not stored by any DB entities
         other than the analysis store itself. So we do not have to worry about cleaning up
-        any other tables,
+        any other tables.
         """
-        # Get length of analyses list for this protocol
         analyses_ids = self.get_ids_by_protocol(protocol_id)
 
         # Delete all analyses exceeding max number allowed,

diff --git a/robot-server/tests/protocols/test_completed_analysis_store.py b/robot-server/tests/protocols/test_completed_analysis_store.py
@@ -316,7 +316,7 @@ async def test_add_makes_room_for_new_analysis(
     """It should delete old analyses and make room for new analysis."""
     protocol_store.insert(make_dummy_protocol_resource("protocol-id"))
 
-    # Set up the database with extra analyses
+    # Set up the database with existing analyses
     resources = [
         _completed_analysis_resource(
             analysis_id=analysis_id,