From d2dd2e15242a57f5bac04cdb70cb3736241b7973 Mon Sep 17 00:00:00 2001 From: Alex Brainman Date: Mon, 26 Mar 2018 19:28:51 +1100 Subject: [PATCH] os: do not test Lstat in TestDevNullFile CL 102456 added Lstat check to TestDevNullFile. But some systems have /dev/null as a symlink, so Lstat test is wrong. Remove the test. Fixes #24521 Change-Id: I149110b08dd05db6495ec4eccbcf943e444332f9 Reviewed-on: https://go-review.googlesource.com/102461 Run-TryBot: Alex Brainman Reviewed-by: Tobias Klauser TryBot-Result: Gobot Gobot --- src/os/os_test.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/os/os_test.go b/src/os/os_test.go index 38cbfca272..3886739389 100644 --- a/src/os/os_test.go +++ b/src/os/os_test.go @@ -1835,12 +1835,6 @@ func testDevNullFile(t *testing.T, devNullName string, ignoreCase bool) { t.Fatalf("Stat(%s): %v", devNullName, err) } testDevNullFileInfo(t, "Stat", devNullName, fi, ignoreCase) - - fi, err = Lstat(devNullName) - if err != nil { - t.Fatalf("Lstat(%s): %v", devNullName, err) - } - testDevNullFileInfo(t, "Lstat", devNullName, fi, ignoreCase) } func TestDevNullFile(t *testing.T) { -- 2.48.1