]> Cypherpunks repositories - gostls13.git/commitdiff
os: use Mkdir in TestCopyFSWithSymlinks
authorKir Kolyshkin <kolyshkin@gmail.com>
Thu, 5 Sep 2024 00:31:56 +0000 (17:31 -0700)
committerGopher Robot <gobot@golang.org>
Fri, 6 Sep 2024 13:23:10 +0000 (13:23 +0000)
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>

src/os/os_test.go

index 9f6f5312888d03bdf934cb4cbf7ba1b1aa78ba69..6a921328451010713e74a4ad59a96d6e8a78c366 100644 (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