unit-test: fix undefined struct field SupportVSocks on arm64

Since arch-specific func getExpectedHostDetails holds undefined struct
field SupportVSocks on arm64, unit test TestEnvGetEnvInfoSetsCPUType,
TestEnvGetHostInfo and so on failed.
I'm trying to use generic func genericgetExpectedHostDetails on arm64
to avoid similar issues.

Fixes: #1287

Signed-off-by: Penny Zheng <penny.zheng@arm.com>
This commit is contained in:
Penny Zheng 2019-03-01 10:32:34 +08:00
parent 6f2597ed11
commit 35672b5896
4 changed files with 15 additions and 86 deletions

View File

@ -16,7 +16,8 @@ import (
func getExpectedHostDetails(tmpdir string) (HostInfo, error) { func getExpectedHostDetails(tmpdir string) (HostInfo, error) {
expectedVendor := "moi" expectedVendor := "moi"
expectedModel := "awesome XI" expectedModel := "awesome XI"
return genericGetExpectedHostDetails(tmpdir, expectedVendor, expectedModel) expectedVMContainerCapable := false
return genericGetExpectedHostDetails(tmpdir, expectedVendor, expectedModel, expectedVMContainerCapable)
} }
func TestEnvGetEnvInfoSetsCPUType(t *testing.T) { func TestEnvGetEnvInfoSetsCPUType(t *testing.T) {

View File

@ -6,92 +6,14 @@
package main package main
import ( import (
"fmt"
"path/filepath"
goruntime "runtime"
"testing" "testing"
) )
func getExpectedHostDetails(tmpdir string) (HostInfo, error) { func getExpectedHostDetails(tmpdir string) (HostInfo, error) {
type filesToCreate struct { expectedVendor := "0x41"
file string expectedModel := "8"
contents string expectedVMContainerCapable := true
} return genericGetExpectedHostDetails(tmpdir, expectedVendor, expectedModel, expectedVMContainerCapable)
const expectedKernelVersion = "99.1"
const expectedArch = goruntime.GOARCH
expectedDistro := DistroInfo{
Name: "Foo",
Version: "42",
}
expectedCPU := CPUInfo{
Vendor: "0x41",
Model: "8",
}
expectedNormalizeCPU := CPUInfo{
Vendor: "ARM Limited",
Model: "v8",
}
expectedHostDetails := HostInfo{
Kernel: expectedKernelVersion,
Architecture: expectedArch,
Distro: expectedDistro,
CPU: expectedNormalizeCPU,
VMContainerCapable: true,
}
testProcCPUInfo := filepath.Join(tmpdir, "cpuinfo")
testOSRelease := filepath.Join(tmpdir, "os-release")
// XXX: This file is *NOT* created by this function on purpose
// (to ensure the only file checked by the tests is
// testOSRelease). osReleaseClr handling is tested in
// utils_test.go.
testOSReleaseClr := filepath.Join(tmpdir, "os-release-clr")
testProcVersion := filepath.Join(tmpdir, "proc-version")
// override
procVersion = testProcVersion
osRelease = testOSRelease
osReleaseClr = testOSReleaseClr
procCPUInfo = testProcCPUInfo
procVersionContents := fmt.Sprintf("Linux version %s a b c",
expectedKernelVersion)
osReleaseContents := fmt.Sprintf(`
NAME="%s"
VERSION_ID="%s"
`, expectedDistro.Name, expectedDistro.Version)
procCPUInfoContents := fmt.Sprintf(`
%s : %s
%s : %s
`,
archCPUVendorField,
expectedCPU.Vendor,
archCPUModelField,
expectedCPU.Model)
data := []filesToCreate{
{procVersion, procVersionContents},
{osRelease, osReleaseContents},
{procCPUInfo, procCPUInfoContents},
}
for _, d := range data {
err := createFile(d.file, d.contents)
if err != nil {
return HostInfo{}, err
}
}
return expectedHostDetails, nil
} }
func TestEnvGetEnvInfoSetsCPUType(t *testing.T) { func TestEnvGetEnvInfoSetsCPUType(t *testing.T) {

View File

@ -10,7 +10,8 @@ import "testing"
func getExpectedHostDetails(tmpdir string) (HostInfo, error) { func getExpectedHostDetails(tmpdir string) (HostInfo, error) {
expectedVendor := "" expectedVendor := ""
expectedModel := "POWER8" expectedModel := "POWER8"
return genericGetExpectedHostDetails(tmpdir, expectedVendor, expectedModel) expectedVMContainerCapable := false
return genericGetExpectedHostDetails(tmpdir, expectedVendor, expectedModel, expectedVMContainerCapable)
} }
func TestEnvGetEnvInfoSetsCPUType(t *testing.T) { func TestEnvGetEnvInfoSetsCPUType(t *testing.T) {

View File

@ -245,7 +245,7 @@ func getExpectedAgentDetails(config oci.RuntimeConfig) (AgentInfo, error) {
} }
// nolint: unused // nolint: unused
func genericGetExpectedHostDetails(tmpdir string, expectedVendor string, expectedModel string) (HostInfo, error) { func genericGetExpectedHostDetails(tmpdir string, expectedVendor string, expectedModel string, expectedVMContainerCapable bool) (HostInfo, error) {
type filesToCreate struct { type filesToCreate struct {
file string file string
contents string contents string
@ -269,7 +269,7 @@ func genericGetExpectedHostDetails(tmpdir string, expectedVendor string, expecte
Architecture: expectedArch, Architecture: expectedArch,
Distro: expectedDistro, Distro: expectedDistro,
CPU: expectedCPU, CPU: expectedCPU,
VMContainerCapable: false, VMContainerCapable: expectedVMContainerCapable,
SupportVSocks: vcUtils.SupportsVsocks(), SupportVSocks: vcUtils.SupportsVsocks(),
} }
@ -320,6 +320,11 @@ VERSION_ID="%s"
} }
} }
if goruntime.GOARCH == "arm64" {
expectedHostDetails.CPU.Vendor = "ARM Limited"
expectedHostDetails.CPU.Model = "v8"
}
return expectedHostDetails, nil return expectedHostDetails, nil
} }