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()) }