fix: use VersionStr in model and add write test

Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>

snapcraft/models/project.py

@@ -22,7 +22,7 @@
 
 import pydantic
 from craft_application import models
-from craft_application.models import BuildInfo, UniqueStrList
+from craft_application.models import BuildInfo, UniqueStrList, VersionStr
 from craft_cli import emit
 from craft_grammar.models import GrammarSingleEntryDictList, GrammarStr, GrammarStrList
 from craft_providers import bases
@@ -45,10 +45,8 @@
 # fmt: off
 if TYPE_CHECKING:
     ProjectName = str
-    ProjectVersion = str
 else:
     ProjectName = constr(max_length=40)
-    ProjectVersion = constr(max_length=32, strict=True)
 # fmt: on
 
 
@@ -444,7 +442,7 @@ class Project(models.Project):
     name: ProjectName  # type: ignore[assignment]
     build_base: Optional[str]
     compression: Literal["lzo", "xz"] = "xz"
-    version: Optional[ProjectVersion]  # type: ignore[assignment]
+    version: Optional[VersionStr]  # type: ignore[assignment]
     donation: Optional[Union[str, UniqueStrList]]
     # snapcraft's `source_code` is more general than craft-application
     source_code: Optional[str]  # type: ignore[assignment]

snapcraft/parts/update_metadata.py

@@ -89,7 +89,7 @@ def _update_project_variables(project: Project, project_vars: Dict[str, str]):
     """Update project fields with values set during lifecycle processing."""
     try:
         if project_vars["version"]:
-            project.version = project_vars["version"]
+            project.version = cast(VersionStr, project_vars["version"])
         if project_vars["grade"]:
             project.grade = project_vars["grade"]  # type: ignore
     except pydantic.ValidationError as err:

tests/unit/services/test_package.py

@@ -17,14 +17,17 @@
 """Tests for the Snapcraft Package service."""
 
 from pathlib import Path
+from textwrap import dedent
 
-from craft_application.models import SummaryStr
+import pytest
+from craft_application.models import SummaryStr, VersionStr
 
 from snapcraft import linters, meta, pack, services
 from snapcraft.application import APP_METADATA
 
 
-def test_pack(package_service, default_factory, mocker):
+@pytest.mark.usefixtures("default_factory")
+def test_pack(package_service, mocker):
     mock_pack_snap = mocker.patch.object(pack, "pack_snap")
     mocker.patch.object(linters, "run_linters")
     mocker.patch.object(linters, "report")
@@ -72,7 +75,7 @@ def test_metadata(package_service, default_factory, default_build_plan, new_dir)
     assert package_service.metadata == meta.SnapMetadata(
         name="default",
         title=None,
-        version="1.0",
+        version=VersionStr("1.0"),
         summary=SummaryStr("default project"),
         description="default project",
         license="MIT",
@@ -96,3 +99,37 @@ def test_metadata(package_service, default_factory, default_build_plan, new_dir)
         provenance=None,
         links=None,
     )
+
+
+def test_write_metadata(
+    package_service,
+    default_factory,
+    default_build_plan,
+    new_dir,
+):
+    default_factory.set_kwargs(
+        "lifecycle",
+        work_dir=Path("work"),
+        cache_dir=new_dir,
+        build_plan=default_build_plan,
+    )
+
+    package_service.write_metadata(new_dir)
+
+    assert (new_dir / "meta" / "snap.yaml").read_text() == dedent(
+        """\
+        name: default
+        version: '1.0'
+        summary: default project
+        description: default project
+        license: MIT
+        architectures:
+        - amd64
+        base: core24
+        confinement: devmode
+        grade: devel
+        environment:
+          LD_LIBRARY_PATH: ${SNAP_LIBRARY_PATH}${LD_LIBRARY_PATH:+:$LD_LIBRARY_PATH}
+          PATH: $SNAP/usr/sbin:$SNAP/usr/bin:$SNAP/sbin:$SNAP/bin:$PATH
+    """
+    )