diff options
| author | Ian Lance Taylor <iant@golang.org> | 2024-03-21 21:42:52 +0000 |
|---|---|---|
| committer | Gopher Robot <gobot@golang.org> | 2024-03-22 13:07:36 +0000 |
| commit | 27f41bb15391668fa8ba18561efe364bab9b8312 (patch) | |
| tree | 07faf4629bbfdd34c5b42798049d0097204f5b58 /src/testing/testing_test.go | |
| parent | b5e377cedcfd7eb629ee5718acabccad98281d51 (diff) | |
| download | go-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.go | 97 |
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 |
