fix linter errors

hardware/opentrons_hardware/hardware_control/motor_position_status.py

@@ -42,6 +42,7 @@
 
 
 def extract_motor_status_info(msg: _MotorStatusMoves) -> MotorPositionStatus:
+    """Extract motor position status from CAN responses."""
     move_ack: Optional[MoveCompleteAck] = None
     if isinstance(msg, MoveCompleted) or isinstance(msg, TipActionResponse):
         move_ack = MoveCompleteAck(msg.payload.ack_id.value)

hardware/opentrons_hardware/hardware_control/types.py

@@ -44,6 +44,7 @@ class MotorPositionStatus:
     move_ack: Optional[MoveCompleteAck] = None
 
     def positions_only(self) -> Tuple[float, float]:
+        """Returns motor and encoder positions as a tuple."""
         return (
             self.motor_position,
             self.encoder_position,

hardware/tests/opentrons_hardware/hardware_control/test_move_group_runner.py

@@ -1,6 +1,6 @@
 """Tests for the move scheduler."""
 import pytest
-from typing import List, Any, Tuple
+from typing import List, Any
 from numpy import float64, float32, int32
 from mock import AsyncMock, call, MagicMock, patch
 from opentrons_shared_data.errors.exceptions import (

hardware/tests/opentrons_hardware/hardware_control/test_tool_sensors.py

@@ -281,11 +281,11 @@ def move_responder(
 
     message_send_loopback.add_responder(move_responder)
 
-    position, encoder_position = await capacitive_probe(
+    status = await capacitive_probe(
         mock_messenger, target_node, motor_node, distance, speed
     )
-    assert position == 10  # this comes from the current_position_um above
-    assert encoder_position == 10
+    assert status.motor_position == 10  # this comes from the current_position_um above
+    assert status.encoder_position == 10
     # this mock assert is annoying because something's __eq__ doesn't work
     assert mock_sensor_threshold.call_args_list[0][0][0] == SensorThresholdInformation(
         sensor=sensor_info,