registering timeout handler synchronously (#3810)

bazelbuild · Jan 3, 2024 · 5b9d2c0 · 5b9d2c0
1 parent 5540b97
commit 5b9d2c0
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/go/tools/bzltestutil/timeout.go b/go/tools/bzltestutil/timeout.go
@@ -22,13 +22,13 @@ import (
 )
 
 func RegisterTimeoutHandler() {
+	// When the Bazel test timeout is reached, Bazel sends a SIGTERM. We
+	// panic just like native go test would so that the user gets stack
+	// traces of all running go routines.
+	// See https://github.com/golang/go/blob/e816eb50140841c524fd07ecb4eaa078954eb47c/src/testing/testing.go#L2351
+	c := make(chan os.Signal, 1)
+	signal.Notify(c, syscall.SIGTERM)
 	go func() {
-		// When the Bazel test timeout is reached, Bazel sends a SIGTERM. We
-		// panic just like native go test would so that the user gets stack
-		// traces of all running go routines.
-		// See https://github.com/golang/go/blob/e816eb50140841c524fd07ecb4eaa078954eb47c/src/testing/testing.go#L2351
-		c := make(chan os.Signal, 1)
-		signal.Notify(c, syscall.SIGTERM)
 		<-c
 		debug.SetTraceback("all")
 		panic("test timed out")