fix(tests): address review comments on TestFindNS

Use a dummy path instead of a real proc path in the "namespace type
missing from spec" case, merge the assert.False call and its message
onto one line, and drop the unnecessary file creation in the "existing
path" case since t.TempDir() already returns a real directory. Also
fixes the gosec G306 warning on the removed WriteFile call.

Signed-off-by: Anand-240 <anandprakashsrivastava68@gmail.com>

Author: Anand-240

pkg/unikontainers/utils_test.go

@@ -24,7 +24,6 @@ import (
 
 	"github.com/opencontainers/runtime-spec/specs-go"
 	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
 )
 
 func TestWritePidFile(t *testing.T) {
@@ -252,13 +251,12 @@ func TestFindNS(t *testing.T) {
 	t.Run("namespace type missing from spec", func(t *testing.T) {
 		t.Parallel()
 		namespaces := []specs.LinuxNamespace{
-			{Type: specs.NetworkNamespace, Path: "/proc/1/ns/net"},
+			{Type: specs.NetworkNamespace, Path: "/dummy/path"},
 		}
 		path, err := findNS(namespaces, specs.MountNamespace)
 		assert.Empty(t, path)
 		assert.Error(t, err)
-		assert.False(t, errors.Is(err, ErrNotExistingNS),
-			"a namespace type absent from the spec must not be reported as ErrNotExistingNS")
+		assert.False(t, errors.Is(err, ErrNotExistingNS), "a namespace type absent from the spec must not be reported as ErrNotExistingNS")
 	})
 
 	t.Run("namespace present without a path yet", func(t *testing.T) {
@@ -273,8 +271,7 @@ func TestFindNS(t *testing.T) {
 
 	t.Run("namespace present with an existing path", func(t *testing.T) {
 		t.Parallel()
-		nsPath := filepath.Join(t.TempDir(), "mnt")
-		require.NoError(t, os.WriteFile(nsPath, []byte{}, 0644))
+		nsPath := t.TempDir()
 
 		namespaces := []specs.LinuxNamespace{
 			{Type: specs.MountNamespace, Path: nsPath},