expire any summarized jobs that are not expired for some reason. (#39)

benchmarking/manager.py

@@ -155,6 +155,10 @@ def get_completed_job_count(self):
             if j.status == 'done' and not j.is_summarized:
                 j.summarize()
 
+            # TODO: patched! sometimes jobs don't get expired?
+            elif j.status == 'done' and j.is_summarized and not j.is_expired:
+                j.expire()
+
         self.logger.info('%s created; %s finished; %s summarized; '
                          '%s; %s jobs total', created, expired, complete,
                          '; '.join('%s %s' % (v, k)

benchmarking/manager_test.py

@@ -104,6 +104,10 @@ def test_get_completed_job_count(self):
         j2.status = 'failed'
         assert mgr.get_completed_job_count() == 0
 
+        j1.status = 'done'
+        j2.status = 'done'
+        assert mgr.get_completed_job_count() == 0
+
         def fake_restart(delay):
             return None
 
@@ -112,6 +116,16 @@ def fake_restart(delay):
         j1.is_expired = True
         assert mgr.get_completed_job_count() == 1
 
+        # test patch, summarized but not expired
+        j1.failed = False
+        j1.status = 'done'
+        j1.created_at = 1
+        j1.finished_at = 1
+        j1.output_url = 1
+        j1.is_expired = False
+        j1.expire = lambda: None
+        assert mgr.get_completed_job_count() == 0
+
     def test_summarize(self):
 
         # pylint: disable=unused-argument