test: fix Windows-specific tests and incorrect mock patches

Skip TestFindCommandRobustNodePaths on non-Windows platforms since it
tests Windows file paths. Correct mock patches to use find_command_robust
instead of find_command in MCP server configuration tests.

Author: alex-feel

tests/test_setup_environment.py

@@ -2971,6 +2971,7 @@ def test_windows_node_not_found(self, mock_find_robust, mock_which, mock_system,
         assert 'Node.js not found in PATH' in captured.err
 
 
+@pytest.mark.skipif(sys.platform != 'win32', reason='Windows-specific test')
 class TestFindCommandRobustNodePaths:
     """Test find_command_robust with various Node.js installation paths."""
 

tests/test_setup_environment_additional.py

@@ -568,7 +568,7 @@ def test_configure_mcp_server_claude_not_found(self, mock_exists, mock_find, _mo
         assert result is False
 
     @patch('platform.system', return_value='Windows')
-    @patch('setup_environment.find_command', return_value=None)
+    @patch('setup_environment.find_command_robust', return_value='claude')
     def test_configure_mcp_server_find_in_npm_path(self, mock_find, mock_system):
         """Test finding claude in npm path on Windows."""
         del mock_find  # Unused but required for patch
@@ -588,7 +588,7 @@ def test_configure_mcp_server_find_in_npm_path(self, mock_find, mock_system):
             assert mock_run.call_count == 4
 
     @patch('platform.system', return_value='Linux')
-    @patch('setup_environment.find_command', return_value=None)
+    @patch('setup_environment.find_command_robust', return_value='claude')
     def test_configure_mcp_server_find_in_unix_paths(self, mock_find, _mock_system):
         """Test finding claude in Unix paths."""
         del mock_find  # Unused but required for patch