don't cover docs

.coveragerc

@@ -0,0 +1,2 @@
+[report]
+omit = */docs/*

src/pyff/store.py

@@ -316,6 +316,7 @@ def reset(self):
 
     def periodic(self, stats):
         now = _now()
+        stats['Last Periodic Maintenance'] = now
         log.debug("periodic maintentance...")
         self.rc.zremrangebyscore("members", "-inf", now)
         for c in self.rc.smembers("#collections"):
@@ -438,8 +439,8 @@ def lookup(self, key):
         if m and ';' in m.group(2):
             hk = hex_digest(key)
             if not self.rc.exists("%s#members" % hk):
-                self.rc.zunionstore(hk, ["{%s}%s#members" % (m.group(1), v) for v in m.group(2).split(';')], 'min')
-                self.rc.expire(hk, 30)  # XXX bad juju - only to keep clients from hammering
+                self.rc.zunionstore("%s#members" % hk, ["{%s}%s#members" % (m.group(1), v) for v in m.group(2).split(';')], 'min')
+                self.rc.expire("%s#members" % hk, 30)  # XXX bad juju - only to keep clients from hammering
             return self.lookup(hk)
         elif self.rc.exists("%s#alias" % key):
             return self.lookup(self.rc.get("%s#alias" % key))

src/pyff/test/test_store.py

@@ -67,7 +67,6 @@ def test_lookup_intersect(self):
         entity_id = root(self.t).get('entityID')
         assert (entity_id is not None)
         e = store.lookup("{%s}%s+{%s}%s" % (ATTRS['domain'], 'example.com', ATTRS['role'], 'idp'))
-        print e
         assert (len(e) == 1)
         assert (e[0] is not None)
         assert (e[0].get('entityID') is not None)
@@ -79,9 +78,16 @@ def test_lookup_intersect_empty(self):
         entity_id = root(self.t).get('entityID')
         assert (entity_id is not None)
         e = store.lookup("{%s}%s+{%s}%s" % (ATTRS['domain'], 'example.com', ATTRS['role'], 'sp'))
-        print e
         assert (len(e) == 0)
 
+    def test_maintain(self):
+        store = self._redis_store()
+        store.update(self.t)
+        entity_id = root(self.t).get('entityID')
+        assert (entity_id is not None)
+        d = dict()
+        store.periodic(d)
+        assert('Last Periodic Maintenance' in d)
 
 class TestMemoryStore(TestCase):
     def setUp(self):