It appears the same test object was returned multiple times for different

unit tests, so tracking tests with sets wouldn't work unless I extracted the
details relevant to the particular test. For now a simple count will work so
using a set was unnecessary anyways.

1 files changed, 6 insertions, 18 deletions

diff --git a/lib/spack/spack/test/tally_plugin.py b/lib/spack/spack/test/tally_plugin.py
index b38f4f3134..e314fae6ce 100644
--- a/lib/spack/spack/test/tally_plugin.py
+++ b/lib/spack/spack/test/tally_plugin.py
@@ -31,21 +31,9 @@ class Tally(Plugin):
 
     def __init__(self):
         super(Tally, self).__init__()
-        self.successes = set()
-        self.failures = set()
-        self.errors = set()
-
-    @property
-    def successCount(self):
-        return len(self.successes)
-    
-    @property
-    def failCount(self):
-        return len(self.failures)
-    
-    @property
-    def errorCount(self):
-        return len(self.errors)
+        self.successCount = 0
+        self.failCount = 0
+        self.errorCount = 0
     
     @property
     def numberOfTests(self):
@@ -58,13 +46,13 @@ class Tally(Plugin):
         super(Tally, self).configure(options, conf)
 
     def addSuccess(self, test):
-        self.successes.add(test)
+        self.successCount += 1
         
     def addError(self, test, err):
-        self.errors.add(test)
+        self.errorCount += 1
             
     def addFailure(self, test, err):
-        test.failures.add(test)
+        self.failCount += 1
 
     def finalize(self, result):
         pass