Skip to content

Commit a1ab85d

Browse files
committed
errors/fmt: re-enable TestPanics test on builders
The test has been fixed in CL 204078, so there isn't a need to skip it on builders anymore. Updates golang/go#30622 Change-Id: I88aeed35f963a9cdfe9b5ee6241956d23c2d23ad Reviewed-on: https://go-review.googlesource.com/c/exp/+/204140 Run-TryBot: Dmitri Shuralyov <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent 8994fa3 commit a1ab85d

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

errors/fmt/fmt_test.go

-4
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import (
99
. "fmt"
1010
"io"
1111
"math"
12-
"os"
1312
"reflect"
1413
"runtime"
1514
"strings"
@@ -1685,9 +1684,6 @@ var stripMethod = strings.NewReplacer(
16851684
)
16861685

16871686
func TestPanics(t *testing.T) {
1688-
if os.Getenv("GO_BUILDER_NAME") != "" {
1689-
t.Skipf("skipping broken test on builders; see https://golang.org/issue/30622")
1690-
}
16911687
for i, tt := range panictests {
16921688
s := Sprintf(tt.fmt, tt.in)
16931689
s = stripMethod.Replace(s)

0 commit comments

Comments
 (0)