]> Cypherpunks repositories - gostls13.git/commit
[release-branch.go1.9] testing: don't fail all tests after racy test failure
authorIan Lance Taylor <iant@golang.org>
Fri, 18 Aug 2017 19:58:08 +0000 (12:58 -0700)
committerIan Lance Taylor <iant@golang.org>
Fri, 18 Aug 2017 21:10:46 +0000 (21:10 +0000)
commitfbf7e1f2952fba64af0fae32c760cf5011eca53e
treee36241a17122d47b8671e51b3c8d1b590482dca1
parent21312a4b5ef42670dfb1d5d38d8a9e4e97cf13a8
[release-branch.go1.9] testing: don't fail all tests after racy test failure

The code was adding race.Errors to t.raceErrors before checking
Failed, but Failed was using t.raceErrors+race.Errors. We don't want
to change Failed, since that would affect tests themselves, so modify
the harness to not unnecessarily change t.raceErrors.

Updates #19851
Fixes #21338

Change-Id: I483f27c68c340928f1cbdef160abc0a5716efb5d
Reviewed-on: https://go-review.googlesource.com/57151
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
src/runtime/race/output_test.go
src/testing/testing.go