fix kwarg issue

Author: connorgallopo

Software/web-server/tests/test_calibration_manager.py

@@ -40,7 +40,7 @@ def test_init_with_custom_binary(self):
         mock_config_manager.register_callback = Mock()
         custom_binary = "/custom/path/pitrac_lm"
 
-        manager = CalibrationManager(mock_config_manager, custom_binary)
+        manager = CalibrationManager(mock_config_manager, pitrac_binary=custom_binary)
 
         assert manager.pitrac_binary == custom_binary
 
@@ -206,7 +206,7 @@ def mock_config_manager(self):
 
     def test_ball_location_command_single_mode(self, mock_config_manager):
         """Test command building for ball location in single mode"""
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         config = mock_config_manager.get_config()
         system_mode = config.get("system", {}).get("mode", "single")
@@ -233,7 +233,7 @@ def test_ball_location_command_dual_mode(self):
         }
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         config = mock_config_manager.get_config()
         system_mode = config.get("system", {}).get("mode", "single")
@@ -362,7 +362,7 @@ async def test_capture_still_image_success(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -387,7 +387,7 @@ async def test_capture_still_image_file_not_created(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -410,7 +410,7 @@ async def test_capture_still_image_process_error(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_subprocess.side_effect = Exception("Camera not found")
@@ -448,7 +448,7 @@ async def test_log_file_creation_during_calibration(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}, "calibration": {}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -493,7 +493,7 @@ async def test_check_ball_location_timeout(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}, "calibration": {}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -517,7 +517,7 @@ async def test_calibration_process_failure(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}, "calibration": {}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -634,7 +634,7 @@ async def test_auto_calibration_success_workflow(self):
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
         mock_config_manager.reload = Mock()
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -680,7 +680,7 @@ async def test_manual_calibration_success_workflow(self):
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
         mock_config_manager.reload = Mock()
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -710,7 +710,7 @@ async def test_calibration_failure_no_results(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}, "calibration": {}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -753,7 +753,7 @@ async def test_ball_location_detection_success(self):
         }
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -778,7 +778,7 @@ async def test_still_image_capture_with_dual_mode(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "dual"}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -813,7 +813,7 @@ async def test_full_calibration_workflow_success(self):
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
         mock_config_manager.reload = Mock()
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()
@@ -860,7 +860,7 @@ async def test_calibration_workflow_with_failures(self):
         mock_config_manager.get_config.return_value = {"system": {"mode": "single"}, "calibration": {}}
         mock_config_manager.generated_config_path = "/tmp/test_config.yaml"
 
-        manager = CalibrationManager(mock_config_manager, "/test/pitrac_lm")
+        manager = CalibrationManager(mock_config_manager, pitrac_binary="/test/pitrac_lm")
 
         with patch("calibration_manager.asyncio.create_subprocess_exec") as mock_subprocess:
             mock_process = AsyncMock()