Code review comment for lp://staging/~tealeg/landscape-client/keystone-scoped-resynch

Revision history for this message
Chris Glass (tribaal) wrote :

[1]
+class FakePersist(object):
+
+ def __init__(self):
+ self.called = False
+
+ def remove(self, key):
+ self.called = True

This should probably go to someplace else, like landscape/lib/persist or landscape/tests/helpers as I assume it can be reused by all other plugins using the persist mechanism?

(ongoing review, saving just because I'm afraid chromium will die on me again)

« Back to merge proposal