go/parser: skip TestParseDepthLimit for short tests

Because it requires a not insignificant amount of memory to run. Also
remove the WASM-only skip from TestScopeDepthLimit, which is less
intensive.

Fixes #53816

Change-Id: I8463046510ce4dd3d8f6d66938828d5e6963c3be
Reviewed-on: https://go-review.googlesource.com/c/go/+/417657
Auto-Submit: Roland Shoemaker <roland@golang.org>
Reviewed-by: Bryan Mills <bcmills@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Roland Shoemaker <roland@golang.org>

diff --git a/src/go/parser/parser_test.go b/src/go/parser/parser_test.go
index 1a46c878663478c02673baaf07172f64577c15ef..0c278924c9ba6080d6a07f746b002dbf00f3313b 100644 (file)
--- a/src/go/parser/parser_test.go
+++ b/src/go/parser/parser_test.go
@@ -10,7 +10,6 @@ import (
        "go/ast"
        "go/token"
        "io/fs"
-       "runtime"
        "strings"
        "testing"
 )
@@ -653,8 +652,8 @@ func split(x string) (pre, mid, post string) {
 }
 
 func TestParseDepthLimit(t *testing.T) {
-       if runtime.GOARCH == "wasm" {
-               t.Skip("causes call stack exhaustion on js/wasm")
+       if testing.Short() {
+               t.Skip("test requires significant memory")
        }
        for _, tt := range parseDepthTests {
                for _, size := range []string{"small", "big"} {
@@ -699,9 +698,6 @@ func TestParseDepthLimit(t *testing.T) {
 }
 
 func TestScopeDepthLimit(t *testing.T) {
-       if runtime.GOARCH == "wasm" {
-               t.Skip("causes call stack exhaustion on js/wasm")
-       }
        for _, tt := range parseDepthTests {
                if !tt.scope {
                        continue