From: qiulaidongfeng <2645477756@qq.com> Date: Mon, 20 Nov 2023 23:08:53 +0000 (+0000) Subject: os: avoid TestFileChdir fail when GOROOT is a symbolic link X-Git-Tag: go1.22rc1~233 X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=78d037b0e016260819f2ff1dac745eda71070a18;p=gostls13.git os: avoid TestFileChdir fail when GOROOT is a symbolic link If GOROOT is a symbolic link, the paths obtained from the first and second Getwd of TestFileChdir are different, and this CL fixes the test failure in this situation. Fixes #64281 Change-Id: I53026b6c54a54be08833396e2c7081ca3ab8c282 GitHub-Last-Rev: 5cc418e6255f6fa530e5a43e3b4d96759e604571 GitHub-Pull-Request: golang/go#64001 Reviewed-on: https://go-review.googlesource.com/c/go/+/540521 Reviewed-by: Michael Knyszek TryBot-Result: Gopher Robot Auto-Submit: Bryan Mills Reviewed-by: Bryan Mills LUCI-TryBot-Result: Go LUCI --- diff --git a/src/os/os_test.go b/src/os/os_test.go index ae12b9ce1b..7e0e0b90be 100644 --- a/src/os/os_test.go +++ b/src/os/os_test.go @@ -1620,8 +1620,17 @@ func TestFileChdir(t *testing.T) { if err != nil { t.Fatalf("Getwd: %s", err) } - if !equal(wdNew, wd) { - t.Fatalf("fd.Chdir failed, got %s, want %s", wdNew, wd) + + wdInfo, err := fd.Stat() + if err != nil { + t.Fatal(err) + } + newInfo, err := Stat(wdNew) + if err != nil { + t.Fatal(err) + } + if !SameFile(wdInfo, newInfo) { + t.Fatalf("fd.Chdir failed: got %s, want %s", wdNew, wd) } }