mirror
/
go
mirror of https://github.com/golang/go.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

path/filepath: use a temp dir in path_test.go 

We should avoid writing temp files to GOROOT, since it might be readonly.

Fixes #23881

Change-Id: I500ca0e0944b6053fd8fd2879ff88f5636424dab
This commit is contained in:
Mostyn Bramley-Moore 2018-03-04 10:44:49 +01:00
parent 8ce74b7d11
commit de0211df84
1 changed files with 17 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
src/path/filepath/path_test.go
View File

@ -433,6 +433,22 @@ func TestWalk(t *testing.T) {
defer restore()
}
}
tmpDir, err := ioutil.TempDir("", "TestWalk")
if err != nil {
t.Fatal("creating temp dir:", err)
}
defer os.RemoveAll(tmpDir)
origDir, err := os.Getwd()
if err != nil {
t.Fatal("finding working dir:", err)
}
if err = os.Chdir(tmpDir); err != nil {
t.Fatal("entering temp dir:", err)
}
defer os.Chdir(origDir)
makeTree(t)
errors := make([]error, 0, 10)
clear := true
@ -440,7 +456,7 @@ func TestWalk(t *testing.T) {
return mark(info, err, &errors, clear)
}
// Expect no errors.
err := filepath.Walk(tree.name, markFn)
err = filepath.Walk(tree.name, markFn)
if err != nil {
t.Fatalf("no error expected, found: %s", err)
}
@ -499,11 +515,6 @@ func TestWalk(t *testing.T) {
os.Chmod(filepath.Join(tree.name, tree.entries[1].name), 0770)
os.Chmod(filepath.Join(tree.name, tree.entries[3].name), 0770)
}
// cleanup
if err := os.RemoveAll(tree.name); err != nil {
t.Errorf("removeTree: %v", err)
}
}
func touch(t *testing.T, name string) {