Fix all_tests.go test counts
The numerator is per shard, but the denominator wasn't.
Change-Id: I1afd784038c51b8db51192b9a2b391073675e390
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/70867
Reviewed-by: Bob Beck <bbe@google.com>
Commit-Queue: Bob Beck <bbe@google.com>
Auto-Submit: David Benjamin <davidben@google.com>
diff --git a/util/all_tests.go b/util/all_tests.go
index 3b31ded..6587b54 100644
--- a/util/all_tests.go
+++ b/util/all_tests.go
@@ -395,10 +395,12 @@
testOutput := testresult.NewResults()
var failed, skipped []test
+ var total int
for testResult := range results {
test := testResult.Test
args := test.Cmd
+ total++
if testResult.Error == errTestSkipped {
fmt.Printf("%s\n", test.longName())
fmt.Printf("%s was skipped\n", args[0])
@@ -427,14 +429,14 @@
}
if len(skipped) > 0 {
- fmt.Printf("\n%d of %d tests were skipped:\n", len(skipped), len(testCases))
+ fmt.Printf("\n%d of %d tests were skipped:\n", len(skipped), total)
for _, test := range skipped {
fmt.Printf("\t%s\n", test.shortName())
}
}
if len(failed) > 0 {
- fmt.Printf("\n%d of %d tests failed:\n", len(failed), len(testCases))
+ fmt.Printf("\n%d of %d tests failed:\n", len(failed), total)
for _, test := range failed {
fmt.Printf("\t%s\n", test.shortName())
}