]> Cypherpunks.ru repositories - gostls13.git/commitdiff
misc/cgo/test: make TestSetgidStress cheaper
authorCherry Mui <cherryyz@google.com>
Fri, 1 Jul 2022 16:56:39 +0000 (12:56 -0400)
committerCherry Mui <cherryyz@google.com>
Wed, 6 Jul 2022 14:23:50 +0000 (14:23 +0000)
TestSetgidStress spawns 1000 threads, which can be expensive on
some platforms or slow builders. Run with 50 threads in short
mode instead.

This makes the failure less reproducible even with buggy code. But
one can manually stress test it (e.g. when a flaky failure appear
on the builder).

Fixes #53641.

Change-Id: I33b5ea5ecaa8c7a56f59c16f9171657ee295db47
Reviewed-on: https://go-review.googlesource.com/c/go/+/415677
Reviewed-by: Austin Clements <austin@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Run-TryBot: Cherry Mui <cherryyz@google.com>

misc/cgo/test/setgid2_linux.go

index d239893f43ed2f8a1a61875a23beec0ea17e42f2..9069cff334368a6a2333dd8762fe255b2d661f28 100644 (file)
@@ -20,7 +20,10 @@ import (
 )
 
 func testSetgidStress(t *testing.T) {
-       const N = 1000
+       var N = 1000
+       if testing.Short() {
+               N = 50
+       }
        ch := make(chan int, N)
        for i := 0; i < N; i++ {
                go func() {