]> Cypherpunks.ru repositories - gostls13.git/commitdiff
container/heap: call t.Helper() in verify()
authorTyler Bui-Palsulich <tpalsulich@google.com>
Wed, 13 Sep 2017 15:09:21 +0000 (08:09 -0700)
committerIan Lance Taylor <iant@golang.org>
Wed, 13 Sep 2017 17:22:00 +0000 (17:22 +0000)
I modified verify() to fail every time to test the change. Before adding
t.Helper() (line 37 is in verify()):
/.../go/src/container/heap/heap_test.go:37: forced failure
FAIL

Afer adding t.Helper() (line 67 is where verify() is called):
/.../go/src/container/heap/heap_test.go:67: forced failure
FAIL

Fixes #21863

Change-Id: I46f0c8ec413cc664358c568fc53e48bb4a1d03d0
Reviewed-on: https://go-review.googlesource.com/63570
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>

src/container/heap/heap_test.go

index d41110422e9fe2c9f140bb8df64578ac89e9196f..f19f9cfa74b79556ec4d110ca7a62b4718d056f7 100644 (file)
@@ -33,6 +33,7 @@ func (h *myHeap) Push(v interface{}) {
 }
 
 func (h myHeap) verify(t *testing.T, i int) {
+       t.Helper()
        n := h.Len()
        j1 := 2*i + 1
        j2 := 2*i + 2