From 112e0f638103a54632c3be17fcc8f34bf97014af Mon Sep 17 00:00:00 2001 From: Jianyong Wu Date: Tue, 5 Oct 2021 16:38:36 +0800 Subject: [PATCH] check: fix typecheck failure in qemu_arm64_test.go fix typecheck failure in qemu_arm64_test.go Signed-off-by: Jianyong Wu --- src/runtime/virtcontainers/qemu_arch_base.go | 12 ++++++++---- src/runtime/virtcontainers/qemu_arm64_test.go | 17 ++++++----------- 2 files changed, 14 insertions(+), 15 deletions(-) diff --git a/src/runtime/virtcontainers/qemu_arch_base.go b/src/runtime/virtcontainers/qemu_arch_base.go index 6638abea7..45b9bbc3f 100644 --- a/src/runtime/virtcontainers/qemu_arch_base.go +++ b/src/runtime/virtcontainers/qemu_arch_base.go @@ -159,19 +159,23 @@ const ( //Intel Trust Domain Extensions //https://software.intel.com/content/www/us/en/develop/articles/intel-trust-domain-extensions.html - tdxProtection + // Exclude from lint checking for it won't be used on arm64 code + tdxProtection //nolint // AMD Secure Encrypted Virtualization // https://developer.amd.com/sev/ - sevProtection + // Exclude from lint checking for it won't be used on arm64 code + sevProtection //nolint // IBM POWER 9 Protected Execution Facility // https://www.kernel.org/doc/html/latest/powerpc/ultravisor.html - pefProtection + // Exclude from lint checking for it won't be used on arm64 code + pefProtection //nolint // IBM Secure Execution (IBM Z & LinuxONE) // https://www.kernel.org/doc/html/latest/virt/kvm/s390-pv.html - seProtection + // Exclude from lint checking for it won't be used on arm64 code + seProtection //nolint ) type qemuArchBase struct { diff --git a/src/runtime/virtcontainers/qemu_arm64_test.go b/src/runtime/virtcontainers/qemu_arm64_test.go index e421aa79b..60d4358d2 100644 --- a/src/runtime/virtcontainers/qemu_arm64_test.go +++ b/src/runtime/virtcontainers/qemu_arm64_test.go @@ -102,14 +102,11 @@ func TestQemuArm64AppendImage(t *testing.T) { imageStat, err := f.Stat() assert.NoError(err) - // save default supportedQemuMachines options - machinesCopy := make([]govmmQemu.Machine, len(supportedQemuMachines)) - assert.Equal(len(supportedQemuMachines), copy(machinesCopy, supportedQemuMachines)) - cfg := qemuConfig(QemuVirt) cfg.ImagePath = f.Name() - arm64 := newQemuArch(cfg) - assert.Contains(m.machine().Options, qemuNvdimmOption) + arm64, err := newQemuArch(cfg) + assert.NoError(err) + assert.Contains(arm64.machine().Options, qemuNvdimmOption) expectedOut := []govmmQemu.Device{ govmmQemu.Object{ @@ -125,9 +122,6 @@ func TestQemuArm64AppendImage(t *testing.T) { devices, err = arm64.appendImage(context.Background(), devices, f.Name()) assert.NoError(err) assert.Equal(expectedOut, devices) - - //restore default supportedQemuMachines options - assert.Equal(len(supportedQemuMachines), copy(supportedQemuMachines, machinesCopy)) } func TestQemuArm64AppendNvdimmImage(t *testing.T) { @@ -168,7 +162,8 @@ func TestQemuArm64WithInitrd(t *testing.T) { cfg := qemuConfig(QemuVirt) cfg.InitrdPath = "dummy-initrd" - arm64 := newQemuArch(cfg) + arm64, err := newQemuArch(cfg) + assert.NoError(err) - assert.NotContains(m.machine().Options, qemuNvdimmOption) + assert.NotContains(arm64.machine().Options, qemuNvdimmOption) }