🐛 test

Author: acabarbaye

utils/filesystem/files_test.go

@@ -9,7 +9,6 @@ import (
 	"context"
 	"fmt"
 	"io"
-	"net"
 	"os"
 	"path/filepath"
 	"reflect"
@@ -23,7 +22,6 @@ import (
 	"github.com/spf13/afero"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
-	"golang.org/x/sys/unix"
 
 	"github.com/ARM-software/golang-utils/utils/collection"
 	"github.com/ARM-software/golang-utils/utils/commonerrors"
@@ -549,9 +547,6 @@ func TestConvertPaths(t *testing.T) {
 }
 
 func TestIsFile(t *testing.T) {
-	if platform.IsWindows() {
-		t.Skip("Windows doesn't have features such as named pipes or unix sockets")
-	}
 	for _, fsType := range FileSystemTypes {
 		t.Run(fmt.Sprint(fsType), func(t *testing.T) {
 			fs := NewFs(fsType)
@@ -567,38 +562,6 @@ func TestIsFile(t *testing.T) {
 				assert.True(t, b)
 			})
 
-			t.Run("special file", func(t *testing.T) {
-				if fsType == InMemoryFS {
-					t.Skip("In-memory file system won't have hardware devices or special files")
-				}
-
-				b, err := fs.IsFile("/dev/null")
-				require.NoError(t, err)
-				assert.True(t, b)
-
-				fifoPath := filepath.Join(tmpDir, faker.Word())
-				require.NoError(t, err)
-				defer func() { _ = fs.Rm(fifoPath) }()
-				err = unix.Mkfifo(fifoPath, 0666)
-				require.NoError(t, err)
-				b, err = fs.IsFile(fifoPath)
-				require.NoError(t, err)
-				assert.True(t, b)
-				err = fs.Rm(fifoPath)
-				require.NoError(t, err)
-
-				socketPath := filepath.Join(tmpDir, faker.Word())
-				require.NoError(t, err)
-				defer func() { _ = fs.Rm(socketPath) }()
-				l, err := net.Listen("unix", socketPath)
-				require.NoError(t, err)
-				defer func() { _ = l.Close() }()
-				b, err = fs.IsFile(socketPath)
-				require.NoError(t, err)
-				assert.True(t, b)
-				err = fs.Rm(socketPath)
-				require.NoError(t, err)
-			})
 		})
 	}
 }

utils/filesystem/files_unix_test.go

@@ -0,0 +1,68 @@
+//go:build !windows
+// +build !windows
+
+package filesystem
+
+import (
+	"fmt"
+	"net"
+	"path/filepath"
+	"testing"
+
+	"github.com/go-faker/faker/v4"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+	"golang.org/x/sys/unix"
+)
+
+func TestIsUnixFile(t *testing.T) {
+	for _, fsType := range FileSystemTypes {
+		t.Run(fmt.Sprint(fsType), func(t *testing.T) {
+			fs := NewFs(fsType)
+
+			tmpDir := t.TempDir()
+
+			t.Run("normal file", func(t *testing.T) {
+				filePath := filepath.Join(tmpDir, faker.Word())
+				err := fs.Touch(filePath)
+				require.NoError(t, err)
+				b, err := fs.IsFile(filePath)
+				require.NoError(t, err)
+				assert.True(t, b)
+			})
+
+			t.Run("special file", func(t *testing.T) {
+				if fsType == InMemoryFS {
+					t.Skip("In-memory file system won't have hardware devices or special files")
+				}
+
+				b, err := fs.IsFile("/dev/null")
+				require.NoError(t, err)
+				assert.True(t, b)
+
+				fifoPath := filepath.Join(tmpDir, faker.Word())
+				require.NoError(t, err)
+				defer func() { _ = fs.Rm(fifoPath) }()
+				err = unix.Mkfifo(fifoPath, 0666)
+				require.NoError(t, err)
+				b, err = fs.IsFile(fifoPath)
+				require.NoError(t, err)
+				assert.True(t, b)
+				err = fs.Rm(fifoPath)
+				require.NoError(t, err)
+
+				socketPath := filepath.Join(tmpDir, faker.Word())
+				require.NoError(t, err)
+				defer func() { _ = fs.Rm(socketPath) }()
+				l, err := net.Listen("unix", socketPath)
+				require.NoError(t, err)
+				defer func() { _ = l.Close() }()
+				b, err = fs.IsFile(socketPath)
+				require.NoError(t, err)
+				assert.True(t, b)
+				err = fs.Rm(socketPath)
+				require.NoError(t, err)
+			})
+		})
+	}
+}

utils/filesystem/lockfile_test.go

@@ -19,6 +19,7 @@ import (
 	"go.uber.org/goleak"
 
 	"github.com/ARM-software/golang-utils/utils/commonerrors"
+	"github.com/ARM-software/golang-utils/utils/commonerrors/errortest"
 )
 
 var (
@@ -361,18 +362,18 @@ func TestLockConcurrentSafeguard(t *testing.T) {
 			// FIXME it was noticed that there could be some race conditions happening in the in-memory file system
 			// see https://github.com/spf13/afero/issues/298
 			if err1 != nil {
-				require.Equal(t0, expectedError, err1)
+				errortest.RequireError(t0, err1, expectedError)
 			}
 			if err2 != nil {
-				require.Equal(t0, expectedError, err2)
+				errortest.RequireError(t0, err2, expectedError)
 			}
 		} else {
 			require.NotEqual(t0, err1, err2)
 			if err1 == nil {
-				require.Equal(t0, expectedError, err2)
+				errortest.RequireError(t0, err2, expectedError)
 			}
 			if err2 == nil {
-				require.Equal(t0, expectedError, err1)
+				errortest.RequireError(t0, err1, expectedError)
 			}
 		}
 	}