Make the tester accurately report total test count

Even when a subset is selected.

Author: marijnh

test/driver.js

@@ -48,10 +48,6 @@ function runTests(callback) {
     }
     if (debug.length < 1) {
       debug = null;
-    } else {
-      if (totalTests > debug.length) {
-        totalTests = debug.length;
-      }
     }
   }
   var totalTime = 0;
@@ -67,19 +63,15 @@ function runTests(callback) {
         // Remove from array for reporting incorrect tests later
         debug.splice(debugIndex, 1);
       } else {
-        var wildcardName = test.name.split("_").shift() + "_*";
+        var wildcardName = test.name.split("_")[0] + "_*";
         debugIndex = indexOf(debug, wildcardName);
         if (debugIndex !== -1) {
           // Remove from array for reporting incorrect tests later
           debug.splice(debugIndex, 1);
           debugUsed.push(wildcardName);
         } else {
           debugIndex = indexOf(debugUsed, wildcardName);
-          if (debugIndex !== -1) {
-            totalTests++;
-          } else {
-            return step(i + 1);
-          }
+          if (debugIndex == -1) return step(i + 1);
         }
       }
     }
@@ -132,3 +124,15 @@ function eqPos(a, b, msg) {
 function is(a, msg) {
   if (!a) throw new Failure(label("assertion failed", msg));
 }
+
+function countTests() {
+  if (!debug) return tests.length;
+  var sum = 0;
+  for (var i = 0; i < tests.length; ++i) {
+    var name = tests[i].name;
+    if (indexOf(debug, name) != -1 ||
+        indexOf(debug, name.split("_")[0] + "_*") != -1)
+      ++sum;
+  }
+  return sum;
+}

test/index.html

@@ -107,7 +107,7 @@ <h1>CodeMirror: Test Suite</h1>
         bad = "";
         verbose = false;
         debugUsed = Array();
-        totalTests = tests.length;
+        totalTests = countTests();
         progressTotal.nodeValue = " of " + totalTests;
         progressRan.nodeValue = count;
         output.innerHTML = '';
@@ -141,8 +141,6 @@ <h1>CodeMirror: Test Suite</h1>
         var message = "???";
         if (type != "done") ++count;
         progress.style.width = (count * (progress.parentNode.clientWidth - 2) / totalTests) + "px";
-        progressTotal.nodeValue = " of " + totalTests +
-          (debugUsed.length && type != "done" ? "+" : "");
         progressRan.nodeValue = count;
         if (type == "ok") {
           message = "Test '" + name + "' succeeded";