commit 25f0c21c81201046436d44d011bfb3f1913c5cfe
author Jonas Termansen <sortie@google.com> Wed Nov 28 11:28:37 2018 +0000
committer Jonas Termansen <sortie@google.com> Wed Nov 28 11:28:37 2018 +0000
tree c7b57ab259e361e04a65bebeee78c9f0821152a5
parent f780b6ea84bfcacba433bcab81bfa8e587e77901

[infra] Fix approve_results reporting wrong number of changed tests

Change-Id: I70afb4319dfd83a38897582b4d3e93113953fdba
Reviewed-on: https://dart-review.googlesource.com/c/85447
Reviewed-by: Alexander Thomas <athom@google.com>

tools/approve_results.dart

diff --git a/tools/approve_results.dart b/tools/approve_results.dart
index 75d54fb..d8adccd 100755
--- a/tools/approve_results.dart
+++ b/tools/approve_results.dart
@@ -358,10 +358,10 @@
 
   // Stop if this is a dry run.
   if (options["no"]) {
-    if (unapprovedBots.length == 1) {
+    if (unapprovedTests.length == 1) {
       print("1 test has a changed result and needs approval");
     } else {
-      print("${unapprovedBots.length} "
+      print("${unapprovedTests.length} "
           "tests have changed results and need approval");
     }
     return;