diff --git a/lib/spack/spack/test/__init__.py b/lib/spack/spack/test/__init__.py
index cb75f15ca19901ab833edc7f4089a4fe95ab654f..7da3895cecc37a47ac7f0a7356774d1cf97f154e 100644
--- a/lib/spack/spack/test/__init__.py
+++ b/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)
 
diff --git a/lib/spack/spack/test/tally_plugin.py b/lib/spack/spack/test/tally_plugin.py
index c167d5852946675acc9cfa9ec84d4f863e311be3..9ca898c47ce0f01686aecbcdfaa82453918df1e9 100644
--- a/lib/spack/spack/test/tally_plugin.py
+++ b/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):