From 9d549b5b62f3afd9704c66b5e348a976b2614909 Mon Sep 17 00:00:00 2001 From: Russ Cox Date: Tue, 29 Dec 2015 12:33:16 -0500 Subject: [PATCH] runtime/pprof: shorten a few tests For #10571. Change-Id: I4bdad64e2dfd692ef2adccf2e5e82e9b1996a8ea Reviewed-on: https://go-review.googlesource.com/18206 Reviewed-by: Austin Clements Run-TryBot: Austin Clements --- src/runtime/pprof/pprof_test.go | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) diff --git a/src/runtime/pprof/pprof_test.go b/src/runtime/pprof/pprof_test.go index 757bde4f28..b87e47eb2f 100644 --- a/src/runtime/pprof/pprof_test.go +++ b/src/runtime/pprof/pprof_test.go @@ -311,7 +311,11 @@ func TestGoroutineSwitch(t *testing.T) { // Test that profiling of division operations is okay, especially on ARM. See issue 6681. func TestMathBigDivide(t *testing.T) { testCPUProfile(t, nil, func() { - t := time.After(5 * time.Second) + duration := 5 * time.Second + if testing.Short() { + duration = 200 * time.Millisecond + } + t := time.After(duration) pi := new(big.Int) for { for i := 0; i < 100; i++ { @@ -336,7 +340,11 @@ func TestStackBarrierProfiling(t *testing.T) { if runtime.GOARCH == "ppc64" || runtime.GOARCH == "ppc64le" { t.Skip("gcstackbarrierall doesn't work on ppc64") } - cmd := exec.Command(os.Args[0], "-test.run=TestStackBarrierProfiling") + args := []string{"-test.run=TestStackBarrierProfiling"} + if testing.Short() { + args = append(args, "-test.short") + } + cmd := exec.Command(os.Args[0], args...) cmd.Env = append([]string{"GODEBUG=gcstackbarrierall=1", "GOGC=1"}, os.Environ()...) if out, err := cmd.CombinedOutput(); err != nil { t.Fatalf("subprocess failed with %v:\n%s", err, out) @@ -349,7 +357,7 @@ func TestStackBarrierProfiling(t *testing.T) { // two samples in stackBarrier. duration := 5 * time.Second if testing.Short() { - duration = 1 * time.Second + duration = 200 * time.Millisecond } t := time.After(duration) for {