Merge pull request #5992 from sujil02/v2-pod-inspect

Enable pod inspect integration test

4 files changed, 5 insertions, 28 deletions

diff --git a/libpod/pod.go b/libpod/pod.go
index 4cdeb1033..b5a14c165 100644
--- a/libpod/pod.go
+++ b/libpod/pod.go
@@ -76,27 +76,6 @@ type podState struct {
 	InfraContainerID string
 }
 
-// PodInspect represents the data we want to display for
-// podman pod inspect
-type PodInspect struct {
-	Config     *PodConfig
-	State      *PodInspectState
-	Containers []PodContainerInfo
-}
-
-// PodInspectState contains inspect data on the pod's state
-type PodInspectState struct {
-	CgroupPath       string `json:"cgroupPath"`
-	InfraContainerID string `json:"infraContainerID"`
-	Status           string `json:"status"`
-}
-
-// PodContainerInfo keeps information on a container in a pod
-type PodContainerInfo struct {
-	ID    string `json:"id"`
-	State string `json:"state"`
-}
-
 // InfraContainerConfig is the configuration for the pod's infra container
 type InfraContainerConfig struct {
 	HasInfraContainer  bool                 `json:"makeInfraContainer"`
diff --git a/pkg/api/handlers/swagger/swagger.go b/pkg/api/handlers/swagger/swagger.go
index 87891d4a8..7d8932f56 100644
--- a/pkg/api/handlers/swagger/swagger.go
+++ b/pkg/api/handlers/swagger/swagger.go
@@ -143,7 +143,7 @@ type swagListPodsResponse struct {
 type swagInspectPodResponse struct {
 	// in:body
 	Body struct {
-		libpod.PodInspect
+		define.InspectPodData
 	}
 }
 
diff --git a/test/e2e/common_test.go b/test/e2e/common_test.go
index d93ee8d3a..160af1bd5 100644
--- a/test/e2e/common_test.go
+++ b/test/e2e/common_test.go
@@ -14,7 +14,6 @@ import (
 	"testing"
 	"time"
 
-	"github.com/containers/libpod/libpod"
 	"github.com/containers/libpod/libpod/define"
 	"github.com/containers/libpod/pkg/inspect"
 	"github.com/containers/libpod/pkg/rootless"
@@ -501,8 +500,8 @@ func (s *PodmanSessionIntegration) InspectContainerToJSON() []define.InspectCont
 }
 
 // InspectPodToJSON takes the sessions output from a pod inspect and returns json
-func (s *PodmanSessionIntegration) InspectPodToJSON() libpod.PodInspect {
-	var i libpod.PodInspect
+func (s *PodmanSessionIntegration) InspectPodToJSON() define.InspectPodData {
+	var i define.InspectPodData
 	err := json.Unmarshal(s.Out.Contents(), &i)
 	Expect(err).To(BeNil())
 	return i
diff --git a/test/e2e/pod_inspect_test.go b/test/e2e/pod_inspect_test.go
index d86c36f58..f87bbe047 100644
--- a/test/e2e/pod_inspect_test.go
+++ b/test/e2e/pod_inspect_test.go
@@ -54,8 +54,7 @@ var _ = Describe("Podman pod inspect", func() {
 		inspect.WaitWithDefaultTimeout()
 		Expect(inspect.ExitCode()).To(Equal(0))
 		Expect(inspect.IsJSONOutputValid()).To(BeTrue())
-		// FIXME sujil, disabled for now
-		//podData := inspect.InspectPodToJSON()
-		//Expect(podData.Config.ID).To(Equal(podid))
+		podData := inspect.InspectPodToJSON()
+		Expect(podData.ID).To(Equal(podid))
 	})
 })