Use testdata/firecracker when it can

The new CI instance doesn't have /usr/local/bin/firecracker.

Since other tests already use testdata/firecracker, using the binary
from these tests is better than having the binary on /usr/local.

Signed-off-by: Kazuyoshi Kato <[email protected]>

Author: kzys

machine_test.go

@@ -909,12 +909,19 @@ func TestPID(t *testing.T) {
 		DisableValidation: true,
 	}
 
-	m, err = NewMachine(context.Background(), cfg, WithLogger(fctesting.NewLogEntry(t)))
+	ctx := context.Background()
+
+	cmd := VMCommandBuilder{}.
+		WithSocketPath(cfg.SocketPath).
+		WithBin(getFirecrackerBinaryPath()).
+		Build(ctx)
+
+	m, err = NewMachine(ctx, cfg, WithProcessRunner(cmd), WithLogger(fctesting.NewLogEntry(t)))
 	if err != nil {
 		t.Errorf("expected no error during create machine, but received %v", err)
 	}
 
-	if err := m.Start(context.Background()); err != nil {
+	if err := m.Start(ctx); err != nil {
 		t.Errorf("expected no error during Start, but received %v", err)
 	}
 	defer m.StopVMM()
@@ -932,7 +939,7 @@ func TestPID(t *testing.T) {
 		t.Errorf("expected clean kill, but received %v", err)
 	}
 
-	m.Wait(context.Background())
+	m.Wait(ctx)
 
 	if _, err := m.PID(); err == nil {
 		t.Errorf("expected an error, but received none")

network_test.go

@@ -353,6 +353,11 @@ func startCNIMachine(t *testing.T,
 	err = ioutil.WriteFile(rootfsPath, rootfsBytes, 0666)
 	require.NoError(t, err, "failed to copy vm rootfs to %s", rootfsPath)
 
+	cmd := VMCommandBuilder{}.
+		WithSocketPath(firecrackerSockPath).
+		WithBin(getFirecrackerBinaryPath()).
+		Build(ctx)
+
 	m, err := NewMachine(ctx, Config{
 		SocketPath:      firecrackerSockPath,
 		KernelImagePath: getVmlinuxPath(t),
@@ -379,7 +384,7 @@ func startCNIMachine(t *testing.T,
 			},
 		}},
 		VMID: vmID,
-	})
+	}, WithProcessRunner(cmd))
 	require.NoError(t, err, "failed to create machine with CNI network interface")
 
 	err = m.Start(context.Background())