From 7bee42630fe67f31df5eba01e7842064f20fe690 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 ac39f880f4c3ad..067530b6d80cd0 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) }