Updated documentation/naming in tally_plugin to reflect that the count of the

total number of tests run excludes skipped tests

lib/spack/spack/test/__init__.py

@@ -104,7 +104,7 @@ def run(names, outputDir, verbose=False):
 
     succeeded = not tally.failCount and not tally.errorCount
     tty.msg("Tests Complete.",
-            "%5d tests run" % tally.numberOfTests,
+            "%5d tests run" % tally.numberOfTestsRun,
             "%5d failures" % tally.failCount,
             "%5d errors" % tally.errorCount)
 

lib/spack/spack/test/tally_plugin.py

@@ -35,9 +35,9 @@ def __init__(self):
         self.failCount = 0
         self.errorCount = 0
     
-    # TODO: this doesn't account for the possibility of skipped tests
     @property
-    def numberOfTests(self):
+    def numberOfTestsRun(self):
+        """Excludes skipped tests"""
         return self.errorCount + self.failCount + self.successCount
 
     def options(self, parser, env=os.environ):