limayaml/defaults_test.go: keep expect as dExpect

Because `d` is no longer modified to expected values, it needs to keep `dExpect` to creating `expect` variable later.

Signed-off-by: Norio Nomura <[email protected]>

Author: norio-nomura

pkg/limayaml/defaults_test.go

@@ -451,6 +451,8 @@ func TestFillDefault(t *testing.T) {
 	FillDefault(&y, &d, &LimaYAML{}, filePath)
 	assert.DeepEqual(t, &y, &expect, opts...)
 
+	dExpect := expect
+
 	// ------------------------------------------------------------------------------------
 	// User-provided defaults should not override user-provided config values
 
@@ -460,26 +462,27 @@ func TestFillDefault(t *testing.T) {
 
 	expect = y
 
-	expect.Provision = append(append([]Provision{}, y.Provision...), d.Provision...)
-	expect.Probes = append(append([]Probe{}, y.Probes...), d.Probes...)
-	expect.PortForwards = append(append([]PortForward{}, y.PortForwards...), d.PortForwards...)
-	expect.CopyToHost = append(append([]CopyToHost{}, y.CopyToHost...), d.CopyToHost...)
-	expect.Containerd.Archives = append(append([]File{}, y.Containerd.Archives...), d.Containerd.Archives...)
-	expect.AdditionalDisks = append(append([]Disk{}, y.AdditionalDisks...), d.AdditionalDisks...)
-	expect.Firmware.Images = append(append([]FileWithVMType{}, y.Firmware.Images...), d.Firmware.Images...)
+	expect.Provision = append(append([]Provision{}, y.Provision...), dExpect.Provision...)
+	expect.Probes = append(append([]Probe{}, y.Probes...), dExpect.Probes...)
+	expect.PortForwards = append(append([]PortForward{}, y.PortForwards...), dExpect.PortForwards...)
+	expect.CopyToHost = append(append([]CopyToHost{}, y.CopyToHost...), dExpect.CopyToHost...)
+	expect.Containerd.Archives = append(append([]File{}, y.Containerd.Archives...), dExpect.Containerd.Archives...)
+	expect.Containerd.Archives[2].Arch = *expect.Arch
+	expect.AdditionalDisks = append(append([]Disk{}, y.AdditionalDisks...), dExpect.AdditionalDisks...)
+	expect.Firmware.Images = append(append([]FileWithVMType{}, y.Firmware.Images...), dExpect.Firmware.Images...)
 
 	// Mounts and Networks start with lowest priority first, so higher priority entries can overwrite
-	expect.Mounts = append(append([]Mount{}, d.Mounts...), y.Mounts...)
-	expect.Networks = append(append([]Network{}, d.Networks...), y.Networks...)
+	expect.Mounts = append(append([]Mount{}, dExpect.Mounts...), y.Mounts...)
+	expect.Networks = append(append([]Network{}, dExpect.Networks...), y.Networks...)
 
-	expect.HostResolver.Hosts["default"] = d.HostResolver.Hosts["default"]
+	expect.HostResolver.Hosts["default"] = dExpect.HostResolver.Hosts["default"]
 
-	// d.DNS will be ignored, and not appended to y.DNS
+	// dExpect.DNS will be ignored, and not appended to y.DNS
 
-	// "TWO" does not exist in filledDefaults.Env, so is set from d.Env
-	expect.Env["TWO"] = d.Env["TWO"]
+	// "TWO" does not exist in filledDefaults.Env, so is set from dExpect.Env
+	expect.Env["TWO"] = dExpect.Env["TWO"]
 
-	expect.Param["TWO"] = d.Param["TWO"]
+	expect.Param["TWO"] = dExpect.Param["TWO"]
 
 	t.Logf("d.vmType=%q, y.vmType=%q, expect.vmType=%q", *d.VMType, *y.VMType, *expect.VMType)
 
@@ -625,19 +628,20 @@ func TestFillDefault(t *testing.T) {
 
 	expect = o
 
-	expect.Provision = append(append(o.Provision, y.Provision...), d.Provision...)
-	expect.Probes = append(append(o.Probes, y.Probes...), d.Probes...)
-	expect.PortForwards = append(append(o.PortForwards, y.PortForwards...), d.PortForwards...)
-	expect.CopyToHost = append(append(o.CopyToHost, y.CopyToHost...), d.CopyToHost...)
-	expect.Containerd.Archives = append(append(o.Containerd.Archives, y.Containerd.Archives...), d.Containerd.Archives...)
-	expect.AdditionalDisks = append(append(o.AdditionalDisks, y.AdditionalDisks...), d.AdditionalDisks...)
-	expect.Firmware.Images = append(append(o.Firmware.Images, y.Firmware.Images...), d.Firmware.Images...)
+	expect.Provision = append(append(o.Provision, y.Provision...), dExpect.Provision...)
+	expect.Probes = append(append(o.Probes, y.Probes...), dExpect.Probes...)
+	expect.PortForwards = append(append(o.PortForwards, y.PortForwards...), dExpect.PortForwards...)
+	expect.CopyToHost = append(append(o.CopyToHost, y.CopyToHost...), dExpect.CopyToHost...)
+	expect.Containerd.Archives = append(append(o.Containerd.Archives, y.Containerd.Archives...), dExpect.Containerd.Archives...)
+	expect.Containerd.Archives[3].Arch = *expect.Arch
+	expect.AdditionalDisks = append(append(o.AdditionalDisks, y.AdditionalDisks...), dExpect.AdditionalDisks...)
+	expect.Firmware.Images = append(append(o.Firmware.Images, y.Firmware.Images...), dExpect.Firmware.Images...)
 
-	expect.HostResolver.Hosts["default"] = d.HostResolver.Hosts["default"]
-	expect.HostResolver.Hosts["MY.Host"] = d.HostResolver.Hosts["host.lima.internal"]
+	expect.HostResolver.Hosts["default"] = dExpect.HostResolver.Hosts["default"]
+	expect.HostResolver.Hosts["MY.Host"] = dExpect.HostResolver.Hosts["host.lima.internal"]
 
-	// o.Mounts just makes d.Mounts[0] writable because the Location matches
-	expect.Mounts = append(append([]Mount{}, d.Mounts...), y.Mounts...)
+	// o.Mounts just makes dExpect.Mounts[0] writable because the Location matches
+	expect.Mounts = append(append([]Mount{}, dExpect.Mounts...), y.Mounts...)
 	expect.Mounts[0].Writable = ptr.Of(true)
 	expect.Mounts[0].SSHFS.Cache = ptr.Of(false)
 	expect.Mounts[0].SSHFS.FollowSymlinks = ptr.Of(true)
@@ -650,8 +654,8 @@ func TestFillDefault(t *testing.T) {
 	expect.MountType = ptr.Of(NINEP)
 	expect.MountInotify = ptr.Of(true)
 
-	// o.Networks[1] is overriding the d.Networks[0].Lima entry for the "def0" interface
-	expect.Networks = append(append(d.Networks, y.Networks...), o.Networks[0])
+	// o.Networks[1] is overriding the dExpect.Networks[0].Lima entry for the "def0" interface
+	expect.Networks = append(append(dExpect.Networks, y.Networks...), o.Networks[0])
 	expect.Networks[0].Lima = o.Networks[1].Lima
 
 	// Only highest prio DNS are retained