tests: add tests for regex and template

Signed-off-by: Henry Schreiner <[email protected]>

Author: henryiii

src/scikit_build_core/build/metadata.py

@@ -13,7 +13,7 @@
     extras_build_system,
     extras_top_level,
 )
-from ..builder._load_provider import load_dynamic_metadata
+from ..builder._load_provider import process_dynamic_metadata
 
 if TYPE_CHECKING:
     from collections.abc import Mapping
@@ -41,7 +41,7 @@ def get_standard_metadata(
     new_pyproject_dict = copy.deepcopy(dict(pyproject_dict))
 
     # Handle any dynamic metadata
-    new_pyproject_dict["project"] = load_dynamic_metadata(
+    new_pyproject_dict["project"] = process_dynamic_metadata(
         new_pyproject_dict["project"], settings.metadata
     )
 

src/scikit_build_core/builder/_load_provider.py

@@ -17,7 +17,7 @@
 
 from ..metadata import _ALL_FIELDS
 
-__all__ = ["load_dynamic_metadata", "load_provider"]
+__all__ = ["load_provider", "process_dynamic_metadata"]
 
 
 def __dir__() -> list[str]:
@@ -30,7 +30,7 @@ def dynamic_metadata(
         self,
         fields: Iterable[str],
         settings: dict[str, Any],
-        metadata: Mapping[str, Any],
+        project: Mapping[str, Any],
     ) -> dict[str, Any]: ...
 
 
@@ -43,9 +43,7 @@ def get_requires_for_dynamic_metadata(
 
 @runtime_checkable
 class DynamicMetadataWheelProtocol(DynamicMetadataProtocol, Protocol):
-    def dynamic_wheel(
-        self, field: str, settings: Mapping[str, Any] | None = None
-    ) -> bool: ...
+    def dynamic_wheel(self, field: str, settings: Mapping[str, Any]) -> bool: ...
 
 
 DMProtocols = Union[
@@ -75,7 +73,7 @@ def load_provider(
 
 def _load_dynamic_metadata(
     metadata: Mapping[str, Mapping[str, str]],
-) -> Generator[tuple[str, DMProtocols, dict[str, str]], None, None]:
+) -> Generator[tuple[str, DMProtocols, dict[str, Any]], None, None]:
     for field, orig_config in metadata.items():
         if "provider" not in orig_config:
             msg = "Missing provider in dynamic metadata"
@@ -92,7 +90,7 @@ def _load_dynamic_metadata(
 
 
 @dataclasses.dataclass
-class DynamicSettings(StrMapping):
+class DynamicPyProject(StrMapping):
     settings: dict[str, dict[str, Any]]
     project: dict[str, Any]
     providers: dict[str, DMProtocols]
@@ -117,9 +115,7 @@ def __getitem__(self, key: str) -> Any:
                 key, self.settings[key]
             )
         else:
-            self.project[key] = provider.dynamic_metadata(
-                key, self.settings[key], self.project
-            )
+            self.project[key] = provider.dynamic_metadata(key, self.settings[key], self)
         self.project["dynamic"].remove(key)
 
         return self.project[key]
@@ -138,14 +134,14 @@ def __contains__(self, key: object) -> bool:
         return key in self.project or key in self.providers
 
 
-def load_dynamic_metadata(
+def process_dynamic_metadata(
     project: Mapping[str, Any],
-    metadata: Mapping[str, Mapping[str, str]],
+    metadata: Mapping[str, Mapping[str, Any]],
 ) -> dict[str, Any]:
     initial = {f: (p, c) for (f, p, c) in _load_dynamic_metadata(metadata)}
 
-    settings = DynamicSettings(
-        settings={f: c for f, (v, c) in initial.items()},
+    settings = DynamicPyProject(
+        settings={f: c for f, (_, c) in initial.items()},
         project=dict(project),
         providers={k: v for k, (v, _) in initial.items()},
     )

src/scikit_build_core/metadata/__init__.py

@@ -68,7 +68,7 @@ def _process_dynamic_metadata(field: str, action: Callable[[str], str], result:
             msg = f"Field {field!r} must be a list of strings"
             raise RuntimeError(msg)
         return [action(r) for r in result]  # type: ignore[return-value]
-    if field in _DICT_STR_FIELDS:
+    if field in _DICT_STR_FIELDS | {"readme"}:
         if not isinstance(result, dict) or not all(
             isinstance(v, str) for v in result.values()
         ):

src/scikit_build_core/metadata/fancy_pypi_readme.py

@@ -21,7 +21,7 @@ def __dir__() -> list[str]:
 def dynamic_metadata(
     field: str,
     settings: dict[str, list[str] | str],
-    metadata: Mapping[str, Any],
+    project: Mapping[str, Any],
 ) -> str | dict[str, str]:
     from hatch_fancy_pypi_readme._builder import build_text
     from hatch_fancy_pypi_readme._config import load_and_validate_config
@@ -45,7 +45,7 @@ def dynamic_metadata(
         try:
             # We don't have access to the version at this point
             text = build_text(
-                config.fragments, config.substitutions, version=metadata["version"]
+                config.fragments, config.substitutions, version=project["version"]
             )
         except TypeError:
             # Version 23.2.0 and before don't have a version field

src/scikit_build_core/metadata/template.py

@@ -20,7 +20,7 @@ def __dir__() -> list[str]:
 def dynamic_metadata(
     field: str,
     settings: Mapping[str, str | list[str] | dict[str, str] | dict[str, list[str]]],
-    metadata: Mapping[str, Any],
+    project: Mapping[str, Any],
 ) -> str | list[str] | dict[str, str] | dict[str, list[str]]:
     if settings.keys() > KEYS:
         msg = f"Only {KEYS} settings allowed by this plugin"
@@ -34,6 +34,6 @@ def dynamic_metadata(
 
     return _process_dynamic_metadata(
         field,
-        lambda r: r.format(project=metadata),
+        lambda r: r.format(project=project),
         result,
     )