commit cb4ecd9fe905910cbdf110dafaad3f2a00ddc85e
author Kir Kolyshkin <kolyshkin@gmail.com> Fri Sep 08 17:03:24 2023 -0700
committer Gopher Robot <gobot@golang.org> Tue Sep 12 16:41:14 2023 +0000
tree c015b373a118531f70acf7845adb3025f38aacdf
parent 4848eb04791849112096f6e86131c18d41b54820

unix: use filepath in tests where appropriate

Change-Id: I0b80cdf4fcf48b80139bf4439f4c78954c972382
Reviewed-on: https://go-review.googlesource.com/c/sys/+/527155
TryBot-Result: Gopher Robot <gobot@golang.org>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Reviewed-by: Tobias Klauser <tobias.klauser@gmail.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Heschi Kreinick <heschi@google.com>
Run-TryBot: Ian Lance Taylor <iant@golang.org>

unix/syscall_darwin_test.go

diff --git a/unix/syscall_darwin_test.go b/unix/syscall_darwin_test.go
index e8ad6bf..4eb055c 100644
--- a/unix/syscall_darwin_test.go
+++ b/unix/syscall_darwin_test.go
@@ -9,7 +9,7 @@
 	"io/ioutil"
 	"net"
 	"os"
-	"path"
+	"path/filepath"
 	"testing"
 
 	"golang.org/x/sys/unix"
@@ -126,8 +126,8 @@
 		t.Fatal(err)
 	}
 
-	src := path.Base(srcFile.Name())
-	dst := path.Base(dstFile.Name())
+	src := filepath.Base(srcFile.Name())
+	dst := filepath.Base(dstFile.Name())
 	err = unix.Clonefileat(srcFd, src, dstFd, dst, 0)
 	if err == unix.ENOSYS || err == unix.ENOTSUP {
 		t.Skip("clonefileat is not available or supported, skipping test")

unix/syscall_freebsd_test.go

diff --git a/unix/syscall_freebsd_test.go b/unix/syscall_freebsd_test.go
index 6f26694..e27689a 100644
--- a/unix/syscall_freebsd_test.go
+++ b/unix/syscall_freebsd_test.go
@@ -13,7 +13,6 @@
 	"net"
 	"os"
 	"os/exec"
-	"path"
 	"path/filepath"
 	"runtime"
 	"testing"
@@ -102,7 +101,7 @@
 const testfile2 = testfile + "2"
 
 func CapEnterTest() {
-	_, err := os.OpenFile(path.Join(procArg, testfile), os.O_WRONLY|os.O_CREATE|os.O_APPEND, 0666)
+	_, err := os.OpenFile(filepath.Join(procArg, testfile), os.O_WRONLY|os.O_CREATE|os.O_APPEND, 0666)
 	if err != nil {
 		panic(fmt.Sprintf("OpenFile: %s", err))
 	}
@@ -112,7 +111,7 @@
 		panic(fmt.Sprintf("CapEnter: %s", err))
 	}
 
-	_, err = os.OpenFile(path.Join(procArg, testfile2), os.O_WRONLY|os.O_CREATE|os.O_APPEND, 0666)
+	_, err = os.OpenFile(filepath.Join(procArg, testfile2), os.O_WRONLY|os.O_CREATE|os.O_APPEND, 0666)
 	if err == nil {
 		panic("OpenFile works!")
 	}