test: mise à jour de la suite de tests pour la cohérence globale

Author: raidos23

tests/conftest.py

@@ -15,9 +15,9 @@
 
 from __future__ import annotations
 
-from pathlib import Path
 import shutil
 import sys
+from pathlib import Path
 
 import pytest
 

tests/test_allversion.py

@@ -19,13 +19,13 @@
 
 from Core.allversion import (
     VersionInfo,
+    get_all_versions,
+    get_bcasl_version,
     get_core_version,
     get_engine_sdk_version,
-    get_bcasl_version,
     get_system_version,
-    get_all_versions,
-    get_versions_dict,
     get_version_string,
+    get_versions_dict,
 )
 
 

tests/test_bcasl_validator.py

@@ -16,8 +16,8 @@
 """Tests for BCASL plugin compatibility validator."""
 
 from bcasl.validator import (
-    parse_version,
     check_plugin_compatibility,
+    parse_version,
     validate_plugins_compatibility,
 )
 

tests/test_cli_build_context_pipeline.py

@@ -27,7 +27,9 @@ class Completed:
         stderr = ""
 
     monkeypatch.setattr("Core.engine.create", lambda _engine_id: FakeEngine())
-    monkeypatch.setattr(helpers.subprocess, "run", lambda *_args, **_kwargs: Completed())
+    monkeypatch.setattr(
+        helpers.subprocess, "run", lambda *_args, **_kwargs: Completed()
+    )
 
     context = BuildContext(
         project_name="demo",

tests/test_engine_loader_dynamic.py

@@ -22,8 +22,8 @@
 
 import Core.engine.registry as engine_registry
 import engine_sdk
-from Loaders.EngineLoader import loader as engine_loader
 from Core.engine.base import CompilerEngine
+from Loaders.EngineLoader import loader as engine_loader
 
 
 def _snapshot_registry_state():

tests/test_engine_tool_install_order.py

@@ -77,7 +77,9 @@ def test_ensure_tools_installed_runs_system_before_python(monkeypatch) -> None:
 
     import Core.SysDependencyManager as SysDependencyManager
 
-    monkeypatch.setattr(SysDependencyManager, "check_system_packages", lambda _packages: False)
+    monkeypatch.setattr(
+        SysDependencyManager, "check_system_packages", lambda _packages: False
+    )
     monkeypatch.setattr(
         SysDependencyManager,
         "SysDependencyManager",
@@ -103,7 +105,9 @@ def test_ensure_tools_installed_continues_python_after_system_timeout(
 
     import Core.SysDependencyManager as SysDependencyManager
 
-    monkeypatch.setattr(SysDependencyManager, "check_system_packages", lambda _packages: False)
+    monkeypatch.setattr(
+        SysDependencyManager, "check_system_packages", lambda _packages: False
+    )
     monkeypatch.setattr(
         SysDependencyManager,
         "SysDependencyManager",

tests/test_sys_deps_headless.py

@@ -34,7 +34,9 @@ def __init__(self, returncode: int = 0) -> None:
 
     monkeypatch.setattr("platform.system", lambda: "Linux")
     monkeypatch.setattr(
-        SysDependencyManager.shutil, "which", lambda name: "/usr/bin/apt" if name == "apt" else None
+        SysDependencyManager.shutil,
+        "which",
+        lambda name: "/usr/bin/apt" if name == "apt" else None,
     )
     monkeypatch.setattr(SysDependencyManager.os, "geteuid", lambda: 1000, raising=False)
 
@@ -71,7 +73,9 @@ def test_install_system_packages_headless_does_not_use_qt_manager(monkeypatch) -
 
     monkeypatch.setattr("platform.system", lambda: "Linux")
     monkeypatch.setattr(
-        SysDependencyManager.shutil, "which", lambda name: "/usr/bin/dnf" if name == "dnf" else None
+        SysDependencyManager.shutil,
+        "which",
+        lambda name: "/usr/bin/dnf" if name == "dnf" else None,
     )
     monkeypatch.setattr(SysDependencyManager.os, "geteuid", lambda: 0, raising=False)
     monkeypatch.setattr(
@@ -88,7 +92,9 @@ def __init__(self, returncode: int = 0) -> None:
             self.stdout = ""
             self.stderr = ""
 
-    monkeypatch.setattr(SysDependencyManager.subprocess, "run", lambda *_a, **_k: _Result(0))
+    monkeypatch.setattr(
+        SysDependencyManager.subprocess, "run", lambda *_a, **_k: _Result(0)
+    )
 
     assert SysDependencyManager.install_system_packages(["cloc"], gui=None) is True
 

tests/test_theme_coverage.py

@@ -25,7 +25,7 @@
 
 from PySide6.QtWidgets import QApplication
 
-from Ui.Gui.UiConnection import apply_theme, _list_available_themes
+from Ui.Gui.UiConnection import _list_available_themes, apply_theme
 
 REQUIRED_SELECTORS = [
     "QToolButton",

tests/test_venv_manager_detection.py

@@ -51,40 +51,11 @@ def _make_fake_venv(path: Path) -> Path:
     return path
 
 
-def test_create_venv_prefers_manager_mapping(test_workspace: Path, monkeypatch) -> None:
-    # Simulate a Poetry-managed project
-    pyproject = test_workspace / "pyproject.toml"
-    pyproject.write_text("[tool.poetry]\nname = 'demo'\n", encoding="utf-8")
-
+def test_resolve_existing_venv_prefers_local(monkeypatch, test_workspace: Path) -> None:
     parent = DummyParent()
     parent.workspace_dir = str(test_workspace)
     mgr = VenvManager(parent)
 
-    called: dict[str, str] = {}
-
-    def fake_create(workspace_dir: str, venv_path: str | None = None) -> None:
-        called["workspace"] = workspace_dir
-        called["venv_path"] = venv_path or ""
-
-    monkeypatch.setattr(mgr, "create_venv_with_manager", fake_create)
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: True)
-
-    mgr.create_venv_if_needed(str(test_workspace))
-
-    assert called.get("workspace") == str(test_workspace)
-    assert called.get("venv_path", "").endswith(os.path.join("", ".venv"))
-
-
-def test_resolve_existing_venv_prefers_manager(
-    monkeypatch, test_workspace: Path
-) -> None:
-    parent = DummyParent()
-    parent.workspace_dir = str(test_workspace)
-    mgr = VenvManager(parent)
-
-    monkeypatch.setattr(
-        mgr, "_detect_manager_existing_venv", lambda base: "/tmp/manager-venv"
-    )
     called = {"select": False}
 
     def fake_select(_base: str) -> str | None:
@@ -95,149 +66,8 @@ def fake_select(_base: str) -> str | None:
 
     result = mgr.resolve_existing_venv(str(test_workspace))
 
-    assert result == "/tmp/manager-venv"
-    assert called["select"] is False
-
-
-def test_install_requirements_prefers_manager(
-    monkeypatch, test_workspace: Path
-) -> None:
-    parent = DummyParent()
-    mgr = VenvManager(parent)
-
-    called: dict[str, str] = {}
-
-    def fake_install(workspace_dir: str, venv_path: str | None = None) -> None:
-        called["workspace"] = workspace_dir
-
-    monkeypatch.setattr(mgr, "install_dependencies_with_manager", fake_install)
-    monkeypatch.setattr(mgr, "_detect_environment_manager", lambda path: "poetry")
-    monkeypatch.setattr(
-        mgr,
-        "_get_requirements_file",
-        lambda _path: (_ for _ in ()).throw(AssertionError("should not call")),
-    )
-
-    mgr.install_requirements_if_needed(str(test_workspace))
-
-    assert called.get("workspace") == str(test_workspace)
-
-
-def test_detect_manager_existing_venv_poetry(test_workspace: Path, monkeypatch) -> None:
-    pyproject = test_workspace / "pyproject.toml"
-    pyproject.write_text("[tool.poetry]\nname = 'demo'\n", encoding="utf-8")
-    venv_path = _make_fake_venv(test_workspace / "poetry-venv")
-
-    mgr = VenvManager(DummyParent())
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: True)
-    monkeypatch.setattr(
-        mgr, "_run_cmd_capture", lambda cmd, cwd, timeout=5: str(venv_path)
-    )
-
-    result = mgr._detect_manager_existing_venv(str(test_workspace))
-    assert result == str(venv_path)
-
-
-def test_detect_manager_existing_venv_pipenv(test_workspace: Path, monkeypatch) -> None:
-    pipfile = test_workspace / "Pipfile"
-    pipfile.write_text("[packages]\n", encoding="utf-8")
-    venv_path = _make_fake_venv(test_workspace / "pipenv-venv")
-
-    mgr = VenvManager(DummyParent())
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: True)
-    monkeypatch.setattr(
-        mgr, "_run_cmd_capture", lambda cmd, cwd, timeout=5: str(venv_path)
-    )
-
-    result = mgr._detect_manager_existing_venv(str(test_workspace))
-    assert result == str(venv_path)
-
-
-def test_detect_manager_existing_venv_pdm(test_workspace: Path, monkeypatch) -> None:
-    pyproject = test_workspace / "pyproject.toml"
-    pyproject.write_text("[tool.pdm]\n", encoding="utf-8")
-    venv_path = _make_fake_venv(test_workspace / "pdm-venv")
-
-    mgr = VenvManager(DummyParent())
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: True)
-    monkeypatch.setattr(
-        mgr, "_run_cmd_capture", lambda cmd, cwd, timeout=5: str(venv_path)
-    )
-
-    result = mgr._detect_manager_existing_venv(str(test_workspace))
-    assert result == str(venv_path)
-
-
-def test_detect_manager_existing_venv_conda_prefix(
-    test_workspace: Path, monkeypatch
-) -> None:
-    venv_path = _make_fake_venv(test_workspace / "conda-env")
-    env_file = test_workspace / "environment.yml"
-    env_file.write_text(f"name: demo\nprefix: {venv_path}\n", encoding="utf-8")
-
-    mgr = VenvManager(DummyParent())
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: True)
-
-    result = mgr._detect_manager_existing_venv(str(test_workspace))
-    assert result == str(venv_path)
-
-
-def test_detect_manager_existing_venv_conda_name(
-    test_workspace: Path, monkeypatch
-) -> None:
-    venv_path = _make_fake_venv(test_workspace / "conda-name-env")
-    env_file = test_workspace / "environment.yml"
-    env_name = venv_path.name
-    env_file.write_text(f"name: {env_name}\n", encoding="utf-8")
-
-    mgr = VenvManager(DummyParent())
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: True)
-
-    class _FakeResult:
-        returncode = 0
-        stdout = f'{{"envs": ["{venv_path}"]}}'
-        stderr = ""
-
-    monkeypatch.setattr(
-        "Core.Venv_Manager.Manager.subprocess.run", lambda *a, **k: _FakeResult()
-    )
-
-    result = mgr._detect_manager_existing_venv(str(test_workspace))
-    assert result == str(venv_path)
-
-
-def test_create_venv_with_manager_fallback(monkeypatch, test_workspace: Path) -> None:
-    mgr = VenvManager(DummyParent())
-    monkeypatch.setattr(mgr, "_detect_environment_manager", lambda path: "poetry")
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: False)
-    called: dict[str, bool] = {}
-
-    def fake_create(path: str, prefer_manager: bool = True):
-        called["prefer_manager"] = prefer_manager
-
-    monkeypatch.setattr(mgr, "create_venv_if_needed", fake_create)
-
-    mgr.create_venv_with_manager(str(test_workspace))
-    assert called.get("prefer_manager") is False
-
-
-def test_install_dependencies_with_manager_fallback(
-    monkeypatch, test_workspace: Path
-) -> None:
-    mgr = VenvManager(DummyParent())
-    monkeypatch.setattr(mgr, "_detect_environment_manager", lambda path: "poetry")
-    monkeypatch.setattr(mgr, "_is_tool_available", lambda tool: False)
-    called: dict[str, bool] = {}
-
-    def fake_install(path: str, force_pip: bool = False):
-        called["force_pip"] = force_pip
-        called["path"] = path
-
-    monkeypatch.setattr(mgr, "install_requirements_if_needed", fake_install)
-
-    mgr.install_dependencies_with_manager(str(test_workspace))
-    assert called.get("force_pip") is True
-    assert called.get("path") == str(test_workspace)
+    assert result == "/tmp/local-venv"
+    assert called["select"] is True
 
 
 def test_pip_break_system_args_enabled_only_on_linux_system_python(monkeypatch) -> None: