From bea8301a8688073a03651ea23ff19bb760ec9e34 Mon Sep 17 00:00:00 2001
From: Jhon Honce <jhonce@redhat.com>
Date: Tue, 3 May 2022 10:51:58 -0700
Subject: Implement --format for machine inspect

* Fix issue of nil pointer derefence

Signed-off-by: Jhon Honce <jhonce@redhat.com>
---
 pkg/machine/e2e/inspect_test.go | 12 +++++++-----
 pkg/machine/e2e/machine_test.go | 11 +++++++++--
 pkg/machine/qemu/machine.go     |  3 ++-
 3 files changed, 18 insertions(+), 8 deletions(-)

(limited to 'pkg')

diff --git a/pkg/machine/e2e/inspect_test.go b/pkg/machine/e2e/inspect_test.go
index e282dd21d..b34285dd8 100644
--- a/pkg/machine/e2e/inspect_test.go
+++ b/pkg/machine/e2e/inspect_test.go
@@ -43,9 +43,11 @@ var _ = Describe("podman machine stop", func() {
 		Expect(foo2).To(Exit(0))
 
 		inspect := new(inspectMachine)
+		inspect = inspect.withFormat("{{.Name}}")
 		inspectSession, err := mb.setName("foo1").setCmd(inspect).run()
 		Expect(err).To(BeNil())
 		Expect(inspectSession).To(Exit(0))
+		Expect(inspectSession.Bytes()).To(ContainSubstring("foo1"))
 
 		type fakeInfos struct {
 			Status string
@@ -56,13 +58,13 @@ var _ = Describe("podman machine stop", func() {
 		Expect(err).ToNot(HaveOccurred())
 		Expect(len(infos)).To(Equal(2))
 
-		//rm := new(rmMachine)
-		////	Must manually clean up due to multiple names
-		//for _, name := range []string{"foo1", "foo2"} {
+		// rm := new(rmMachine)
+		// //	Must manually clean up due to multiple names
+		// for _, name := range []string{"foo1", "foo2"} {
 		//	mb.setName(name).setCmd(rm.withForce()).run()
 		//	mb.names = []string{}
-		//}
-		//mb.names = []string{}
+		// }
+		// mb.names = []string{}
 
 	})
 })
diff --git a/pkg/machine/e2e/machine_test.go b/pkg/machine/e2e/machine_test.go
index 2b3b60b2b..657014b05 100644
--- a/pkg/machine/e2e/machine_test.go
+++ b/pkg/machine/e2e/machine_test.go
@@ -23,14 +23,20 @@ func TestMain(m *testing.M) {
 
 const (
 	defaultStream string = "podman-testing"
-	tmpDir        string = "/var/tmp"
 )
 
 var (
+	tmpDir         = "/var/tmp"
 	fqImageName    string
 	suiteImageName string
 )
 
+func init() {
+	if value, ok := os.LookupEnv("TMPDIR"); ok {
+		tmpDir = value
+	}
+}
+
 // TestLibpod ginkgo master function
 func TestMachine(t *testing.T) {
 	RegisterFailHandler(Fail)
@@ -70,7 +76,8 @@ var _ = SynchronizedAfterSuite(func() {},
 	})
 
 func setup() (string, *machineTestBuilder) {
-	homeDir, err := ioutil.TempDir("/var/tmp", "podman_test")
+	// Set TMPDIR if this needs a new directory
+	homeDir, err := ioutil.TempDir("", "podman_test")
 	if err != nil {
 		Fail(fmt.Sprintf("failed to create home directory: %q", err))
 	}
diff --git a/pkg/machine/qemu/machine.go b/pkg/machine/qemu/machine.go
index 91e15c2af..fe33e580c 100644
--- a/pkg/machine/qemu/machine.go
+++ b/pkg/machine/qemu/machine.go
@@ -525,10 +525,11 @@ func (v *MachineVM) Start(name string, _ machine.StartOptions) error {
 		time.Sleep(wait)
 		wait++
 	}
-	defer qemuSocketConn.Close()
 	if err != nil {
 		return err
 	}
+	defer qemuSocketConn.Close()
+
 	fd, err := qemuSocketConn.(*net.UnixConn).File()
 	if err != nil {
 		return err
-- 
cgit v1.2.3-54-g00ecf