Address copilot review comments

Author: 1Blademaster

radio/app/controllers/paramsController.py

@@ -373,21 +373,21 @@ def saveParam(self, param_name: str, param_value: Number, param_type: int) -> No
                 }
             )
 
-    def getSingleParam(self, params: str) -> Union[CachedParam, dict]:
+    def getSingleParam(self, param_id: str) -> Union[CachedParam, dict]:
         """
         Get a single parameter from the cached params.
 
         Args:
-            params (Optional[str]): The name of the parameter to get
+            param_id (str): The ID of the parameter to get
         """
-        if isinstance(params, str):
+        if isinstance(param_id, str):
             try:
-                return next((x for x in self.params if x["param_id"] == params))
+                return next((x for x in self.params if x["param_id"] == param_id))
             except StopIteration:
-                self.drone.logger.error(f"Param {params} not found in cached params")
+                self.drone.logger.error(f"Param {param_id} not found in cached params")
                 return {}
         else:
-            self.drone.logger.error(f"Invalid params type, got {type(params)}")
+            self.drone.logger.error(f"Invalid param_id type, got {type(param_id)}")
             return {}
 
     def exportParamsToFile(self, file_path: str) -> Response:

radio/tests/test_gripper.py

@@ -37,7 +37,7 @@ def run_once_after_all_tests():
 
     # Ensure gripper is definitely re-enabled even if tests pass
     droneStatus.drone.paramsController.setParam(
-        "GRIP_ENABLE", 1, mavutil.mavlink.MAV_PARAM_TYPE_REAL32
+        "GRIP_ENABLE", 1, mavutil.mavlink.MAV_PARAM_TYPE_UINT8
     )
     droneStatus.drone.paramsController.saveParam(
         "GRIP_ENABLE", 1, mavutil.mavlink.MAV_PARAM_TYPE_UINT8
@@ -98,7 +98,7 @@ def test_setGripper(socketio_client: SocketIOTestClient, droneStatus):
 
 def test_gripperDisabled(socketio_client: SocketIOTestClient, droneStatus) -> None:
     droneStatus.drone.paramsController.setParam(
-        "GRIP_ENABLE", 0, mavutil.mavlink.MAV_PARAM_TYPE_REAL32
+        "GRIP_ENABLE", 0, mavutil.mavlink.MAV_PARAM_TYPE_UINT8
     )
     # Allow time for gripper to be updated
     time.sleep(0.5)