]> Cypherpunks repositories - gostls13.git/commitdiff
time: deflake TestAfterQueuing
authorBrad Fitzpatrick <bradfitz@golang.org>
Wed, 6 May 2015 17:54:20 +0000 (10:54 -0700)
committerBrad Fitzpatrick <bradfitz@golang.org>
Wed, 6 May 2015 19:26:27 +0000 (19:26 +0000)
Ramp up the delay on subsequent attempts. Fast builders have the same delay.

Not a perfect fix, but should make it better. And this easy.

Fixes #9903 maybe
Fixes #10680 maybe

Change-Id: I967380c2cb8196e6da9a71116961229d37b36335
Reviewed-on: https://go-review.googlesource.com/9795
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
src/time/sleep_test.go

index 2fce75343527701be1c3d2526d4afd9d0d28d00d..dd0a820dd6e3fe4ee344ecbaa6bf4a26c3f361a3 100644 (file)
@@ -223,13 +223,10 @@ func TestAfterStop(t *testing.T) {
 func TestAfterQueuing(t *testing.T) {
        // This test flakes out on some systems,
        // so we'll try it a few times before declaring it a failure.
-       const attempts = 3
+       const attempts = 5
        err := errors.New("!=nil")
        for i := 0; i < attempts && err != nil; i++ {
-               delta := 100 * Millisecond
-               if i == 0 && testing.Short() {
-                       delta = 20 * Millisecond
-               }
+               delta := Duration(20+i*50) * Millisecond
                if err = testAfterQueuing(t, delta); err != nil {
                        t.Logf("attempt %v failed: %v", i, err)
                }