Fix tests

Author: simonkurtz-MSFT

…ython/test_setup_python_path_settings.py tests/python/test_local_setup.pytests/python/test_setup_python_path_settings.py renamed to tests/python/test_local_setup.py tests/python/test_setup_python_path_settings.py renamed to tests/python/test_local_setup.py

@@ -20,7 +20,7 @@
 if TYPE_CHECKING:  # pragma: no cover
     sps = cast(ModuleType, None)
 else:
-    sps = cast(ModuleType, importlib.import_module("setup_python_path"))
+    sps = cast(ModuleType, importlib.import_module("local_setup"))
 
 
 @pytest.fixture
@@ -46,11 +46,6 @@ def test_create_vscode_settings_creates_perf_excludes(temp_project_root: Path) -
 
     settings = _read_settings(temp_project_root)
 
-    assert settings["search.exclude"]["**/.venv"] is True
-    assert settings["search.exclude"]["**/.venv/**"] is True
-    assert settings["files.watcherExclude"]["**/.venv/**"] is True
-    assert settings["files.exclude"]["**/.venv"] is True
-
     assert settings["python.analysis.exclude"][: len(sps.DEFAULT_PYTHON_ANALYSIS_EXCLUDE)] == sps.DEFAULT_PYTHON_ANALYSIS_EXCLUDE
 
 
@@ -61,7 +56,6 @@ def test_create_vscode_settings_merges_excludes(temp_project_root: Path) -> None
     (vscode_dir / "settings.json").write_text(
         json.dumps(
             {
-                "search.exclude": {"custom/**": True, "**/.venv": False},
                 "python.analysis.exclude": ["custom2/**", "**/__pycache__"],
             },
             indent=4,
@@ -73,11 +67,6 @@ def test_create_vscode_settings_merges_excludes(temp_project_root: Path) -> None
 
     settings = _read_settings(temp_project_root)
 
-    # Required keys forced on, custom preserved
-    assert settings["search.exclude"]["custom/**"] is True
-    assert settings["search.exclude"]["**/.venv"] is True
-    assert settings["search.exclude"]["**/.venv/**"] is True
-
     # Required patterns come first, custom patterns preserved afterwards
     assert settings["python.analysis.exclude"][: len(sps.DEFAULT_PYTHON_ANALYSIS_EXCLUDE)] == sps.DEFAULT_PYTHON_ANALYSIS_EXCLUDE
     assert "custom2/**" in settings["python.analysis.exclude"]
@@ -90,7 +79,6 @@ def test_force_kernel_consistency_merges_excludes(temp_project_root: Path, monke
     (vscode_dir / "settings.json").write_text(
         json.dumps(
             {
-                "search.exclude": {"**/.venv/**": False},
                 "files.watcherExclude": {"other/**": True},
                 "python.analysis.exclude": ["custom3/**"],
             },
@@ -106,11 +94,6 @@ def test_force_kernel_consistency_merges_excludes(temp_project_root: Path, monke
 
     settings = _read_settings(temp_project_root)
 
-    assert settings["search.exclude"]["**/.venv"] is True
-    assert settings["search.exclude"]["**/.venv/**"] is True
-    assert settings["files.watcherExclude"]["**/.venv/**"] is True
-    assert settings["files.watcherExclude"]["other/**"] is True
-
     assert settings["python.analysis.exclude"][: len(sps.DEFAULT_PYTHON_ANALYSIS_EXCLUDE)] == sps.DEFAULT_PYTHON_ANALYSIS_EXCLUDE
     assert "custom3/**" in settings["python.analysis.exclude"]
 

tests/python/test_verify_local_setup.py

@@ -66,23 +66,6 @@ def fake_import(name: str, *args: Any, **kwargs: Any) -> Any:
     assert vls.check_required_packages() is False
 
 
-def test_check_vscode_settings_success(temp_cwd: Path) -> None:
-    """VS Code settings check should succeed when required keys are present."""
-
-    settings_dir = temp_cwd / ".vscode"
-    settings_dir.mkdir(parents=True)
-    (settings_dir / "settings.json").write_text(
-        '{\n'
-        '  "jupyter.defaultKernel": "apim-samples",\n'
-        '  "python.defaultInterpreterPath": ".venv/",\n'
-        '  "notebook.defaultLanguage": "python"\n'
-        '}\n',
-        encoding="utf-8",
-    )
-
-    assert vls.check_vscode_settings() is True
-
-
 def test_check_env_file_validation(temp_cwd: Path) -> None:
     """Environment file check should validate required keys."""