MAINT: refactor metadata validation

Move to stand-alone function for clarity and to enable unit testing.

Author: dnicolodi

mesonpy/__init__.py

@@ -618,13 +618,28 @@ def _string_or_strings(value: Any, name: str) -> List[str]:
     return config
 
 
-class Project():
-    """Meson project wrapper to generate Python artifacts."""
+def _validate_metadata(metadata: pyproject_metadata.StandardMetadata) -> None:
+    """Validate package metadata."""
 
-    _ALLOWED_DYNAMIC_FIELDS: ClassVar[List[str]] = [
+    allowed_dynamic_fields = [
         'version',
     ]
 
+    # check for unsupported dynamic fields
+    unsupported_dynamic = {key for key in metadata.dynamic if key not in allowed_dynamic_fields}
+    if unsupported_dynamic:
+        s = ', '.join(f'"{x}"' for x in unsupported_dynamic)
+        raise ConfigError(f'unsupported dynamic metadata fields: {s}')
+
+    # check if we are running on an unsupported interpreter
+    if metadata.requires_python:
+        metadata.requires_python.prereleases = True
+        if platform.python_version().rstrip('+') not in metadata.requires_python:
+            raise ConfigError(f'building with Python {platform.python_version()}, version {metadata.requires_python} required')
+
+
+class Project():
+    """Meson project wrapper to generate Python artifacts."""
     def __init__(
         self,
         source_dir: Path,
@@ -722,7 +737,7 @@ def __init__(
                 '{yellow}{bold}! Using Meson to generate the project metadata '
                 '(no `project` section in pyproject.toml){reset}'.format(**_STYLES)
             )
-        self._validate_metadata()
+        _validate_metadata(self._metadata)
 
         # set version from meson.build if dynamic
         if 'version' in self._metadata.dynamic:
@@ -764,27 +779,6 @@ def _configure(self, reconfigure: bool = False) -> None:
 
         self._run(['meson', 'setup', *setup_args])
 
-    def _validate_metadata(self) -> None:
-        """Check the pyproject.toml metadata and see if there are any issues."""
-
-        # check for unsupported dynamic fields
-        unsupported_dynamic = {
-            key for key in self._metadata.dynamic
-            if key not in self._ALLOWED_DYNAMIC_FIELDS
-        }
-        if unsupported_dynamic:
-            s = ', '.join(f'"{x}"' for x in unsupported_dynamic)
-            raise MesonBuilderError(f'Unsupported dynamic fields: {s}')
-
-        # check if we are running on an unsupported interpreter
-        if self._metadata.requires_python:
-            self._metadata.requires_python.prereleases = True
-            if platform.python_version().rstrip('+') not in self._metadata.requires_python:
-                raise MesonBuilderError(
-                    f'Unsupported Python version {platform.python_version()}, '
-                    f'expected {self._metadata.requires_python}'
-                )
-
     @cached_property
     def _wheel_builder(self) -> _WheelBuilder:
         return _WheelBuilder(

tests/test_project.py

@@ -45,14 +45,14 @@ def test_version(package):
 
 
 def test_unsupported_dynamic(package_unsupported_dynamic):
-    with pytest.raises(mesonpy.MesonBuilderError, match='Unsupported dynamic fields: "dependencies"'):
+    with pytest.raises(mesonpy.ConfigError, match='unsupported dynamic metadata fields: "dependencies"'):
         with mesonpy.Project.with_temp_working_dir():
             pass
 
 
 def test_unsupported_python_version(package_unsupported_python_version):
-    with pytest.raises(mesonpy.MesonBuilderError, match=(
-        f'Unsupported Python version {platform.python_version()}, expected ==1.0.0'
+    with pytest.raises(mesonpy.ConfigError, match=(
+        f'building with Python {platform.python_version()}, version ==1.0.0 required'
     )):
         with mesonpy.Project.with_temp_working_dir():
             pass