install: add a "firmware" parameter

Author: ydirson

tests/install/test.py

@@ -13,35 +13,43 @@ class TestNested:
     @pytest.mark.parametrize("iso_version", (
         "821.1", "83b2",
     ))
-    @pytest.mark.vm_definitions(
-        dict(name="vm1",
-             template="Other install media",
-             params=(
-                 # dict(param_name="", value=""),
-                 dict(param_name="memory-static-max", value="4GiB"),
-                 dict(param_name="memory-dynamic-max", value="4GiB"),
-                 dict(param_name="memory-dynamic-min", value="4GiB"),
-                 dict(param_name="platform", key="exp-nested-hvm", value="true"), # FIXME < 8.3 host?
-                 dict(param_name="HVM-boot-params", key="firmware", value="uefi"),
-                 dict(param_name="HVM-boot-params", key="order", value="dc"),
-                 dict(param_name="platform", key="device-model", value="qemu-upstream-uefi"),
-             ),
-             vdis=[dict(name="vm1 system disk", size="100GiB", device="xvda", userdevice="0")],
-             vifs=[dict(index=0, network_uuid="eabc1038-e40f-2ae5-0781-a3adbec1cae8")], # FIXME
-             ))
+    @pytest.mark.parametrize("firmware", ("uefi", "bios"))
+    @pytest.mark.vm_definitions(lambda firmware: dict(
+        name="vm1",
+        template="Other install media",
+        params=(
+            # dict(param_name="", value=""),
+            dict(param_name="memory-static-max", value="4GiB"),
+            dict(param_name="memory-dynamic-max", value="4GiB"),
+            dict(param_name="memory-dynamic-min", value="4GiB"),
+            dict(param_name="platform", key="exp-nested-hvm", value="true"), # FIXME < 8.3 host?
+            dict(param_name="HVM-boot-params", key="order", value="dc"),
+        ) + {
+            "uefi": (
+                dict(param_name="HVM-boot-params", key="firmware", value="uefi"),
+                dict(param_name="platform", key="device-model", value="qemu-upstream-uefi"),
+            ),
+            "bios": (),
+        }[firmware],
+        vdis=[dict(name="vm1 system disk", size="100GiB", device="xvda", userdevice="0")],
+        vifs=[dict(index=0, network_uuid="eabc1038-e40f-2ae5-0781-a3adbec1cae8")], # FIXME
+    ),
+                                param_mapping={"firmware": "firmware"})
     @pytest.mark.installer_iso(
         lambda version: {
             "821.1": "xcpng-8.2.1-2023",
             "83b2": "xcpng-8.3-beta2",
         }[version],
         param_mapping={"version": "iso_version"})
-    @pytest.mark.answerfile(
-        {
-            "base": "INSTALL",
-            "source": {"type": "local"},
-            "primary-disk": {"text": "nvme0n1"},
-        })
-    def test_install_uefi(self, request, iso_remaster, create_vms, iso_version):
+    @pytest.mark.answerfile(lambda firmware: {
+        "base": "INSTALL",
+        "source": {"type": "local"},
+        "primary-disk": {"text": {"uefi": "nvme0n1",
+                                  "bios": "sda"}[firmware]
+                         },
+    },
+                            param_mapping={"firmware": "firmware"})
+    def test_install(self, request, firmware, iso_remaster, create_vms, iso_version):
         assert len(create_vms) == 1
         host_vm = create_vms[0]
         # FIXME should be part of vm def
@@ -125,16 +133,17 @@ def test_install_uefi(self, request, iso_remaster, create_vms, iso_version):
         "821.1",
         "821.1-821.1",
     ))
-    @pytest.mark.continuation_of(
-        lambda params: [dict(vm="vm1",
-                             image_test=(f"TestNested::{{}}[{params}]".format(
-                                 {
-                                     1: "test_install_uefi",
-                                     2: "test_upgrade_uefi",
-                                 }[len(params.split("-"))]
-                             )))],
-        param_mapping={"params": "mode"})
-    def test_firstboot_uefi(self, request, create_vms, mode):
+    @pytest.mark.parametrize("firmware", ("uefi", "bios"))
+    @pytest.mark.continuation_of(lambda params, firmware: [dict(
+        vm="vm1",
+        image_test=(f"TestNested::{{}}[{firmware}-{params}]".format(
+            {
+                1: "test_install",
+                2: "test_upgrade",
+            }[len(params.split("-"))]
+        )))],
+                                 param_mapping={"params": "mode", "firmware": "firmware"})
+    def test_firstboot(self, firmware, request, create_vms, mode):
         host_vm = create_vms[0]
         vif = host_vm.vifs()[0]
         mac_address = vif.param_get('MAC')
@@ -237,23 +246,26 @@ def test_firstboot_uefi(self, request, create_vms, mode):
         ("821.1", "83b2"),
         #("83b2", "83b2"), # 8.3b2 disabled the upgrade from 8.3
     ])
-    @pytest.mark.continuation_of(
-        lambda params: [dict(vm="vm1",
-                             image_test=f"TestNested::test_firstboot_uefi[{params}]")],
-        param_mapping={"params": "orig_version"})
+    @pytest.mark.parametrize("firmware", ("uefi", "bios"))
+    @pytest.mark.continuation_of(lambda firmware, params: [dict(
+        vm="vm1",
+        image_test=f"TestNested::test_firstboot[{firmware}-{params}]")],
+                                 param_mapping={"params": "orig_version", "firmware": "firmware"})
     @pytest.mark.installer_iso(
         lambda version: {
             "821.1": "xcpng-8.2.1-2023",
             "83b2": "xcpng-8.3-beta2",
         }[version],
         param_mapping={"version": "iso_version"})
-    @pytest.mark.answerfile(
-        {
+    @pytest.mark.answerfile(lambda firmware: {
             "base": "UPGRADE",
             "source": {"type": "local"},
-            "existing-installation": {"text": "nvme0n1"},
-        })
-    def test_upgrade_uefi(self, iso_remaster, create_vms, orig_version, iso_version):
+            "existing-installation": {"text": {"uefi": "nvme0n1",
+                                               "bios": "sda"}[firmware]
+                                      },
+        },
+                            param_mapping={"firmware": "firmware"})
+    def test_upgrade(self, firmware, iso_remaster, create_vms, orig_version, iso_version):
         host_vm = create_vms[0]
         vif = host_vm.vifs()[0]
         mac_address = vif.param_get('MAC')