aboutsummaryrefslogtreecommitdiff
path: root/src/testing/testing_test.go
diff options
context:
space:
mode:
authorIan Lance Taylor <iant@golang.org>2024-03-21 21:42:52 +0000
committerGopher Robot <gobot@golang.org>2024-03-22 13:07:36 +0000
commit27f41bb15391668fa8ba18561efe364bab9b8312 (patch)
tree07faf4629bbfdd34c5b42798049d0097204f5b58 /src/testing/testing_test.go
parentb5e377cedcfd7eb629ee5718acabccad98281d51 (diff)
downloadgo-27f41bb15391668fa8ba18561efe364bab9b8312.tar.xz
Revert "testing: add TB.SetGOMAXPROCS function"
This reverts CL 519235. Reason for revert: Proposal is still in incoming. For #62020 Change-Id: Icccb930209f36097f5d930c01eda6b5042bdddc1 Reviewed-on: https://go-review.googlesource.com/c/go/+/573516 Reviewed-by: Than McIntosh <thanm@google.com> Auto-Submit: Ian Lance Taylor <iant@golang.org> LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com> Reviewed-by: Ian Lance Taylor <iant@google.com>
Diffstat (limited to 'src/testing/testing_test.go')
-rw-r--r--src/testing/testing_test.go97
1 files changed, 0 insertions, 97 deletions
diff --git a/src/testing/testing_test.go b/src/testing/testing_test.go
index 28b5809eea..d3822dfd57 100644
--- a/src/testing/testing_test.go
+++ b/src/testing/testing_test.go
@@ -13,7 +13,6 @@ import (
"os/exec"
"path/filepath"
"regexp"
- "runtime"
"slices"
"strings"
"sync"
@@ -259,102 +258,6 @@ func TestSetenvWithParallelGrandParentBeforeSetenv(t *testing.T) {
})
}
-func TestSetGOMAXPROCS(t *testing.T) {
- if runtime.GOARCH == "wasm" {
- t.Skip("not supported on wasm yet")
- }
- tests := []struct {
- name string
- newP int
- }{
- {
- name: "overriding value",
- newP: 1,
- },
- }
-
- for _, test := range tests {
- p := runtime.GOMAXPROCS(0)
- t.Run(test.name, func(t *testing.T) {
- t.SetGOMAXPROCS(test.newP + 1)
- if runtime.GOMAXPROCS(0) != test.newP+1 {
- t.Fatalf("unexpected value after t.SetGOMAXPROCS: got %d, want %d", runtime.GOMAXPROCS(0), test.newP+1)
- }
- })
- if runtime.GOMAXPROCS(0) != p {
- t.Fatalf("unexpected value after t.SetGOMAXPROCS cleanup: got %d, want %d", runtime.GOMAXPROCS(0), p)
- }
- }
-}
-
-func TestSetGOMAXPROCSWithParallelAfterSetGOMAXPROCS(t *testing.T) {
- if runtime.GOARCH == "wasm" {
- t.Skip("not supported on wasm yet")
- }
- defer func() {
- want := "testing: t.Parallel called after t.SetGOMAXPROCS; cannot set GOMAXPROCS in parallel tests"
- if got := recover(); got != want {
- t.Fatalf("expected panic; got %#v want %q", got, want)
- }
- }()
- p := runtime.GOMAXPROCS(0)
- t.SetGOMAXPROCS(p + 1)
- t.Parallel()
-}
-
-func TestSetGOMAXPROCSWithParallelBeforeSetGOMAXPROCS(t *testing.T) {
- if runtime.GOARCH == "wasm" {
- t.Skip("not supported on wasm yet")
- }
- defer func() {
- want := "testing: t.SetGOMAXPROCS called after t.Parallel; cannot set GOMAXPROCS in parallel tests"
- if got := recover(); got != want {
- t.Fatalf("expected panic; got %#v want %q", got, want)
- }
- }()
- t.Parallel()
- p := runtime.GOMAXPROCS(0)
- t.SetGOMAXPROCS(p + 1)
-}
-
-func TestSetGOMAXPROCSWithParallelParentBeforeSetGOMAXPROCS(t *testing.T) {
- if runtime.GOARCH == "wasm" {
- t.Skip("not supported on wasm yet")
- }
- t.Parallel()
- t.Run("child", func(t *testing.T) {
- defer func() {
- want := "testing: t.SetGOMAXPROCS called after t.Parallel; cannot set GOMAXPROCS in parallel tests"
- if got := recover(); got != want {
- t.Fatalf("expected panic; got %#v want %q", got, want)
- }
- }()
-
- p := runtime.GOMAXPROCS(0)
- t.SetGOMAXPROCS(p + 1)
- })
-}
-
-func TestSetGOMAXPROCSWithParallelGrandParentBeforeSetGOMAXPROCS(t *testing.T) {
- if runtime.GOARCH == "wasm" {
- t.Skip("not supported on wasm yet")
- }
- t.Parallel()
- t.Run("child", func(t *testing.T) {
- t.Run("grand-child", func(t *testing.T) {
- defer func() {
- want := "testing: t.SetGOMAXPROCS called after t.Parallel; cannot set GOMAXPROCS in parallel tests"
- if got := recover(); got != want {
- t.Fatalf("expected panic; got %#v want %q", got, want)
- }
- }()
-
- p := runtime.GOMAXPROCS(0)
- t.SetGOMAXPROCS(p + 1)
- })
- })
-}
-
// testingTrueInInit is part of TestTesting.
var testingTrueInInit = false