test devices

Signed-off-by: Arjun Raja Yogidas <[email protected]>

Author: coderbirju

cmd/nerdctl/container/container_inspect_linux_test.go

@@ -17,6 +17,7 @@
 package container
 
 import (
+	"encoding/json"
 	"fmt"
 	"strings"
 	"testing"
@@ -251,7 +252,6 @@ func TestContainerInspectHostConfig(t *testing.T) {
 		"--read-only",
 		"--shm-size", "256m",
 		"--uts", "host",
-		"--device", "/dev/null:/dev/null",
 		"--sysctl", "net.core.somaxconn=1024",
 		"--runtime", "io.containerd.runc.v2",
 		testutil.AlpineImage, "sleep", "infinity").AssertOK()
@@ -277,11 +277,6 @@ func TestContainerInspectHostConfig(t *testing.T) {
 	assert.Equal(t, true, inspect.HostConfig.ReadonlyRootfs)
 	assert.Equal(t, "host", inspect.HostConfig.UTSMode)
 	assert.Equal(t, int64(268435456), inspect.HostConfig.ShmSize)
-	// assert.Equal(t, "io.containerd.runtime.v1.linux", inspect.HostConfig.Runtime)
-	// expectedSysctls := map[string]string{
-	// 	"net.core.somaxconn": "1024",
-	// }
-	// assert.Equal(t, bool(true), inspect.HostConfig.OomKillDisable)
 }
 
 func TestContainerInspectHostConfigDefaults(t *testing.T) {
@@ -305,19 +300,14 @@ func TestContainerInspectHostConfigDefaults(t *testing.T) {
 	assert.Equal(t, "private", inspect.HostConfig.IpcMode)
 	assert.Equal(t, "", inspect.HostConfig.LogConfig.Driver)
 	assert.Equal(t, int64(0), inspect.HostConfig.Memory)
-	t.Logf("inspect.HostConfig.Memory in TestContainerInspectHostConfigDefaults: %+v", inspect.HostConfig.Memory)
 	assert.Equal(t, int64(0), inspect.HostConfig.MemorySwap)
-	t.Logf("inspect.HostConfig.MemorySwap in TestContainerInspectHostConfigDefaults: %+v", inspect.HostConfig.MemorySwap)
 	assert.Equal(t, bool(false), inspect.HostConfig.OomKillDisable)
-	t.Logf("inspect.HostConfig.OomKillDisable in TestContainerInspectHostConfigDefaults: %+v", inspect.HostConfig.OomKillDisable)
 	assert.Equal(t, bool(false), inspect.HostConfig.ReadonlyRootfs)
 	assert.Equal(t, "", inspect.HostConfig.UTSMode)
 	assert.Equal(t, int64(67108864), inspect.HostConfig.ShmSize)
 	assert.Equal(t, "runc", inspect.HostConfig.Runtime)
 	assert.Equal(t, 0, len(inspect.HostConfig.Sysctls))
 	assert.Equal(t, 0, len(inspect.HostConfig.Devices))
-	// t.Logf("len(inspect.HostConfig.Sysctls) in TestContainerInspectHostConfigDefaults: %+v", len(inspect.HostConfig.Sysctls))
-	// t.Logf("len(inspect.HostConfig.Devices) in TestContainerInspectHostConfigDefaults: %+v", len(inspect.HostConfig.Devices))
 }
 
 func TestContainerInspectHostConfigDNS(t *testing.T) {
@@ -406,62 +396,34 @@ func TestContainerInspectHostConfigPIDDefaults(t *testing.T) {
 	assert.Equal(t, "", inspect.HostConfig.PidMode)
 }
 
-// assert.Equal(t, true, inspect.HostConfig.ReadonlyRootfs)
-// 	assert.Equal(t, "host", inspect.HostConfig.UTSMode)
-// 	assert.Equal(t, int64(268435456), inspect.HostConfig.ShmSize)
-// 	assert.Equal(t, "io.containerd.runtime.v1.linux", inspect.HostConfig.Runtime)
-// 	expectedSysctls := map[string]string{
-// 		"net.core.somaxconn": "1024",
-// 	}
-// 	assert.DeepEqual(t, expectedSysctls, inspect.HostConfig.Sysctls)
-// 	expectedDevices := []string{"/dev/null:/dev/null"}
-// 	assert.DeepEqual(t, expectedDevices, inspect.HostConfig.Devices)
-
-// func TestContainerInspectHostConfigAdvanced(t *testing.T) {
-// 	testContainer := testutil.Identifier(t)
-
-// 	base := testutil.NewBase(t)
-// 	defer base.Cmd("rm", "-f", testContainer).Run()
-
-// 	// Run a container with various advanced HostConfig options
-// 	base.Cmd("run", "-d", "--name", testContainer,
-// 		"--read-only",
-// 		"--uts", "host",
-// 		"--shm-size", "256m",
-// 		"--runtime", "io.containerd.runtime.v1.linux",
-// 		"--sysctl", "net.core.somaxconn=1024",
-// 		"--device", "/dev/null:/dev/null",
-// 		testutil.AlpineImage, "sleep", "infinity").AssertOK()
-
-// 	inspect := base.InspectContainer(testContainer)
-
-// 	// Check ReadonlyRootfs
-// 	assert.Equal(t, true, inspect.HostConfig.ReadonlyRootfs)
-
-// 	// Check UTSMode
-// 	assert.Equal(t, "host", inspect.HostConfig.UTSMode)
-
-// 	// Check ShmSize
-// 	assert.Equal(t, int64(268435456), inspect.HostConfig.ShmSize)
-
-// 	// Check Runtime
-// 	assert.Equal(t, "io.containerd.runtime.v1.linux", inspect.HostConfig.Runtime)
-
-// 	// Check Sysctls
-// 	expectedSysctls := map[string]string{
-// 		"net.core.somaxconn": "1024",
-// 	}
-// 	assert.DeepEqual(t, expectedSysctls, inspect.HostConfig.Sysctls)
-
-// 	// Check Devices
-// 	expectedDevices := []string{"/dev/null:/dev/null"}
-// 	assert.DeepEqual(t, expectedDevices, inspect.HostConfig.Devices)
-
-// 	// Log the entire HostConfig for debugging
-// 	hostConfigJSON, err := json.MarshalIndent(inspect.HostConfig, "", "  ")
-// 	if err != nil {
-// 		t.Errorf("Failed to marshal HostConfig: %v", err)
-// 	} else {
-// 		t.Logf("HostConfig in TestContainerInspectHostConfigAdvanced:\n%s", string(hostConfigJSON))
-// 	}
-// }
+func TestContainerInspectDevices(t *testing.T) {
+	testContainer := testutil.Identifier(t)
+
+	base := testutil.NewBase(t)
+	defer base.Cmd("rm", "-f", testContainer).Run()
+
+	// Run a container with various advanced HostConfig options
+	base.Cmd("run", "-d", "--name", testContainer,
+		"--device", "/dev/null:/dev/null",
+		testutil.AlpineImage, "sleep", "infinity").AssertOK()
+
+	inspect := base.InspectContainer(testContainer)
+
+	// Check Devices
+	expectedDevices := []dockercompat.DeviceMapping{
+		{
+			PathOnHost:        "/dev/null",
+			PathInContainer:   "/dev/null",
+			CgroupPermissions: "rwm",
+		},
+	}
+	assert.DeepEqual(t, expectedDevices, inspect.HostConfig.Devices)
+
+	// Log the entire HostConfig for debugging
+	hostConfigJSON, err := json.MarshalIndent(inspect.HostConfig, "", "  ")
+	if err != nil {
+		t.Errorf("Failed to marshal HostConfig: %v", err)
+	} else {
+		t.Logf("HostConfig in TestContainerInspectDevices:\n%s", string(hostConfigJSON))
+	}
+}