WIP installer_iso: allow use of a callable to depend on version

FIXME:
- doc
- squash into "add "version" test parameter" ?

Author: ydirson

tests/install/conftest.py

@@ -88,6 +88,13 @@ def iso_remaster(request, answerfile):
     assert marker is not None, "iso_remaster fixture requires 'installer_iso' marker"
     iso_key = marker.args[0]
 
+    if callable(iso_key):
+        try:
+            version = request.getfixturevalue("version")
+        except pytest.FixtureLookupError as e:
+            raise RuntimeError("'version' test parameter not found") from e
+        iso_key = iso_key(version=version)
+
     from data import ISO_IMAGES, ISOSR_SRV, ISOSR_PATH, PXE_CONFIG_SERVER, TEST_SSH_PUBKEY, TOOLS
     assert "iso-remaster" in TOOLS
     iso_remaster = TOOLS["iso-remaster"]

tests/install/test.py

@@ -16,14 +16,9 @@
 
 @pytest.mark.dependency()
 class TestNested:
-    @pytest.mark.parametrize("version", [
-        pytest.param("821.1", marks=[
-            pytest.mark.installer_iso("xcpng-8.2.1-2023"),
-        ]),
-        pytest.param("83b2", marks=[
-            pytest.mark.installer_iso("xcpng-8.3-beta2"),
-        ]),
-    ])
+    @pytest.mark.parametrize("version", ("821.1", "83b2"))
+    @pytest.mark.installer_iso(lambda version: {"821.1": "xcpng-8.2.1-2023",
+                                                "83b2": "xcpng-8.3-beta2"}[version])
     @pytest.mark.vm_definitions(
         dict(name="vm 1",
              template="Other install media",
@@ -46,7 +41,7 @@ class TestNested:
             "source": {"type": "local"},
             "primary-disk": {"text": "nvme0n1"},
         })
-    def test_install_uefi(self, request, iso_remaster, create_vms, version):
+    def test_install_uefi(self, request, version, iso_remaster, create_vms):
         assert len(create_vms) == 1
         host_vm = create_vms[0]
         # FIXME should be part of vm def
@@ -261,20 +256,17 @@ def test_firstboot_uefi(self, request, create_vms, mode):
             pytest.mark.dependency(depends=["TestNested::test_firstboot_uefi[821.1]"]),
             pytest.mark.vm_definitions(
                 dict(name="vm 1", image="install/test::Nested::firstboot_uefi[821.1]")),
-            pytest.mark.installer_iso("xcpng-8.2.1-2023"),
         ]),
         pytest.param("821.1", "83b2", marks=[
             pytest.mark.dependency(depends=["TestNested::test_firstboot_uefi[821.1]"]),
             pytest.mark.vm_definitions(
                 dict(name="vm 1", image="install/test::Nested::firstboot_uefi[821.1]")),
-            pytest.mark.installer_iso("xcpng-8.3-beta2"),
         ]),
         # 8.3b2 disabled the upgrade from 8.3
         #pytest.param("83b2", "83b2", marks=[
         #    pytest.mark.dependency(depends=["TestNested::test_firstboot_uefi[83b2]"]),
         #    pytest.mark.vm_definitions(
         #        dict(name="vm 1", image="install/test::Nested::firstboot_uefi[83b2]")),
-        #    pytest.mark.installer_iso("xcpng-8.3-beta2"),
         #]),
     ])
     @pytest.mark.answerfile(
@@ -283,6 +275,8 @@ def test_firstboot_uefi(self, request, create_vms, mode):
             "source": {"type": "local"},
             "existing-installation": {"text": "nvme0n1"},
         })
+    @pytest.mark.installer_iso(lambda version: {"821.1": "xcpng-8.2.1-2023",
+                                                "83b2": "xcpng-8.3-beta2"}[version])
     def test_upgrade_uefi(self, request, iso_remaster, create_vms, orig_version, version):
         host_vm = create_vms[0]
         vif = host_vm.vifs()[0]