From 089e37a931d30d4055c7468facb602c4cfa9b537 Mon Sep 17 00:00:00 2001 From: Johan Brandhorst-Satzkorn Date: Fri, 30 Jun 2023 13:48:10 -0700 Subject: [PATCH] bytes: remove builders check from compare test TestCompareBytes already took into account the -short testing flag, however, only if not run on one of the Go builders. This extra condition is no longer necessary as we have much better longtest coverage than we did when the check was added. Fixes #61071 Change-Id: I0fc716f4e7baa04019ee00608f223f27c931edcc Reviewed-on: https://go-review.googlesource.com/c/go/+/507416 Run-TryBot: Ian Lance Taylor Reviewed-by: Bryan Mills Auto-Submit: Johan Brandhorst-Satzkorn TryBot-Bypass: Johan Brandhorst-Satzkorn Auto-Submit: Ian Lance Taylor Reviewed-by: Ian Lance Taylor --- src/bytes/compare_test.go | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/bytes/compare_test.go b/src/bytes/compare_test.go index ac39f880f4..067530b6d8 100644 --- a/src/bytes/compare_test.go +++ b/src/bytes/compare_test.go @@ -7,7 +7,6 @@ package bytes_test import ( . "bytes" "fmt" - "internal/testenv" "testing" ) @@ -73,7 +72,7 @@ func TestCompareBytes(t *testing.T) { } lengths = append(lengths, 256, 512, 1024, 1333, 4095, 4096, 4097) - if !testing.Short() || testenv.Builder() != "" { + if !testing.Short() { lengths = append(lengths, 65535, 65536, 65537, 99999) } -- 2.50.0