move away from bild_backend to more stable build_backend and add tests

Author: thrau

plux/build/config.py

@@ -59,7 +59,7 @@ class PluxConfiguration:
     entrypoint_static_file: str = "plux.ini"
     """The name of the entrypoint ini file if entrypoint_build_mode is set to MANUAL."""
 
-    bild_backend: BuildBackend = BuildBackend.AUTO
+    build_backend: BuildBackend = BuildBackend.AUTO
     """The build backend to use. If set to ``auto``, the build backend will be detected automatically from the config."""
 
     def merge(
@@ -69,7 +69,7 @@ def merge(
         include: list[str] = None,
         entrypoint_build_mode: EntrypointBuildMode = None,
         entrypoint_static_file: str = None,
-        bild_backend: BuildBackend = None,
+        build_backend: BuildBackend = None,
     ) -> "PluxConfiguration":
         """
         Merges or overwrites the given values into the current configuration and returns a new configuration object.
@@ -85,7 +85,7 @@ def merge(
             entrypoint_static_file=entrypoint_static_file
             if entrypoint_static_file is not None
             else self.entrypoint_static_file,
-            bild_backend=bild_backend if bild_backend is not None else self.bild_backend,
+            build_backend=build_backend if build_backend is not None else self.build_backend,
         )
 
 
@@ -193,9 +193,9 @@ def determine_build_backend_from_config(workdir: str) -> BuildBackend:
     # parse config to get build backend
     plux_config = read_plux_config_from_workdir(workdir)
 
-    if plux_config.bild_backend != BuildBackend.AUTO:
+    if plux_config.build_backend != BuildBackend.AUTO:
         # first, check if the user configured one
-        return plux_config.bild_backend
+        return plux_config.build_backend
 
     # otherwise, try to determine it from the build-backend attribute in the pyproject.toml
     try:

tests/build/test_config.py

@@ -0,0 +1,60 @@
+import os
+import textwrap
+import uuid
+
+import pytest
+
+from plux.build.config import read_plux_config_from_workdir, BuildBackend, determine_build_backend_from_config
+
+
+@pytest.fixture
+def create_config_workdir(tmp_path):
+    """
+    Factory fixture that creates a temporary directory with a pyproject.toml file in it using the given string.
+    When the factory is invoked, it also changes the working directory to that temp directory.
+    """
+    cur_dir = os.getcwd()
+
+    def _create(config: str):
+        tmp_dir = tmp_path / f"project.{str(uuid.uuid4())[-8:]}"
+        tmp_dir.mkdir()
+        config_path = tmp_dir / "pyproject.toml"
+        config_path.write_text(config)
+
+        os.chdir(tmp_dir)
+
+        return config_path
+
+    yield _create
+
+    os.chdir(cur_dir)
+
+
+@pytest.mark.parametrize("backend", ["setuptools", "hatchling"])
+def test_config_with_explicit_build_backend(backend, create_config_workdir):
+    config = textwrap.dedent(f"""
+    [tool.plux]
+    entrypoint_build_mode = "manual"
+    build_backend = "{backend}"
+    """)
+
+    create_config_workdir(config)
+
+    cfg = read_plux_config_from_workdir()
+    assert cfg.build_backend == BuildBackend(backend)
+
+
+def test_build_backend_is_read_from_config(create_config_workdir):
+    # this test makes sure the build-system.build-backend extraction works
+    config = textwrap.dedent("""
+    [build-system]
+    requires = ["hatchling", "wheel"]
+    build-backend = "hatchling.build"
+    """)
+
+    create_config_workdir(config)
+
+    cfg = read_plux_config_from_workdir()
+    assert cfg.build_backend == BuildBackend.AUTO
+
+    assert determine_build_backend_from_config(os.getcwd()) == BuildBackend.HATCHLING