Skip to content
Snippets Groups Projects
Commit b292799d authored by Dmitri Shuralyov's avatar Dmitri Shuralyov Committed by Gopher Robot
Browse files

testing: use temp dir without symlinks in TestChdir/relative

When paths with symlinks are involved, it's not viable to compare them
with string equality. Don't use a temporary directory with symlinks in
it as input, so the test works in more environments.

For #62516.

Change-Id: I95d774365cc2f90eb0ffcffa61229ed5cee43e3e
Reviewed-on: https://go-review.googlesource.com/c/go/+/607035


Reviewed-by: default avatarIan Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: default avatarDmitri Shuralyov <dmitshur@google.com>
parent 98ed865a
No related branches found
No related tags found
No related merge requests found
...@@ -293,7 +293,11 @@ func TestChdir(t *testing.T) { ...@@ -293,7 +293,11 @@ func TestChdir(t *testing.T) {
} }
defer os.Chdir(oldDir) defer os.Chdir(oldDir)
tmp := t.TempDir() // The "relative" test case relies on tmp not being a symlink.
tmp, err := filepath.EvalSymlinks(t.TempDir())
if err != nil {
t.Fatal(err)
}
rel, err := filepath.Rel(oldDir, tmp) rel, err := filepath.Rel(oldDir, tmp)
if err != nil { if err != nil {
t.Fatal(err) t.Fatal(err)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment