1
0
mirror of https://github.com/golang/go synced 2024-11-22 03:54:39 -07:00

os: use Mkdir in TestCopyFSWithSymlinks

This code creates a few directories under a temporary directory that was
just created before, so using MkdirTemp is not needed here.

Change-Id: Icfc45b70349bc1927efb1647bcc9fd58aa82b792
Reviewed-on: https://go-review.googlesource.com/c/go/+/611037
Reviewed-by: Ian Lance Taylor <iant@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
This commit is contained in:
Kir Kolyshkin 2024-09-04 17:31:56 -07:00 committed by Gopher Robot
parent 6cb8e5ca9f
commit 86d74894cc

View File

@ -3404,9 +3404,9 @@ func TestCopyFSWithSymlinks(t *testing.T) {
// Create a directory and file outside.
tmpDir := t.TempDir()
outsideDir, err := MkdirTemp(tmpDir, "copyfs_out_")
if err != nil {
t.Fatalf("MkdirTemp: %v", err)
outsideDir := filepath.Join(tmpDir, "copyfs_out")
if err := Mkdir(outsideDir, 0755); err != nil {
t.Fatalf("Mkdir: %v", err)
}
outsideFile := filepath.Join(outsideDir, "file.out.txt")
@ -3415,9 +3415,9 @@ func TestCopyFSWithSymlinks(t *testing.T) {
}
// Create a directory and file inside.
insideDir, err := MkdirTemp(tmpDir, "copyfs_in_")
if err != nil {
t.Fatalf("MkdirTemp: %v", err)
insideDir := filepath.Join(tmpDir, "copyfs_in")
if err := Mkdir(insideDir, 0755); err != nil {
t.Fatalf("Mkdir: %v", err)
}
insideFile := filepath.Join(insideDir, "file.in.txt")
if err := WriteFile(insideFile, []byte("Testing CopyFS inside"), 0644); err != nil {
@ -3463,9 +3463,9 @@ func TestCopyFSWithSymlinks(t *testing.T) {
// Copy the directory tree and verify.
forceMFTUpdateOnWindows(t, insideDir)
fsys := DirFS(insideDir)
tmpDupDir, err := MkdirTemp(tmpDir, "copyfs_dup_")
if err != nil {
t.Fatalf("MkdirTemp: %v", err)
tmpDupDir := filepath.Join(tmpDir, "copyfs_dup")
if err := Mkdir(tmpDupDir, 0755); err != nil {
t.Fatalf("Mkdir: %v", err)
}
// TODO(panjf2000): symlinks are currently not supported, and a specific error