diff options
| author | Peter Scheibel <scheibel1@llnl.gov> | 2015-11-25 18:08:17 -0800 |
|---|---|---|
| committer | Peter Scheibel <scheibel1@llnl.gov> | 2015-11-25 18:08:17 -0800 |
| commit | fc96f62794a5c718afb5904d93bb9370ed4a261d (patch) | |
| tree | e3207ae30d830f0933d1f0d0023bf1597c55bdd1 | |
| parent | d50a18d9ebe38137690dfd0a6972616c0e07271c (diff) | |
| download | spack-fc96f62794a5c718afb5904d93bb9370ed4a261d.tar.gz spack-fc96f62794a5c718afb5904d93bb9370ed4a261d.tar.bz2 spack-fc96f62794a5c718afb5904d93bb9370ed4a261d.tar.xz spack-fc96f62794a5c718afb5904d93bb9370ed4a261d.zip | |
Updated documentation/naming in tally_plugin to reflect that the count of the
total number of tests run excludes skipped tests
| -rw-r--r-- | lib/spack/spack/test/__init__.py | 2 | ||||
| -rw-r--r-- | lib/spack/spack/test/tally_plugin.py | 4 |
2 files changed, 3 insertions, 3 deletions
diff --git a/lib/spack/spack/test/__init__.py b/lib/spack/spack/test/__init__.py index cb75f15ca1..7da3895cec 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 c167d58529..9ca898c47c 100644 --- a/lib/spack/spack/test/tally_plugin.py +++ b/lib/spack/spack/test/tally_plugin.py @@ -35,9 +35,9 @@ class Tally(Plugin): 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): |