We should not even allow empty timestamps in. But if you guys think we should then an empty timestamp means we should generate one using time.time() and insert it into the DB. What do u think?
-- testInsertGetWithoutTimestamp testcase fails https://bugs.launchpad.net/bugs/486996 You received this bug notification because you are a member of Zeitgeist-Engine, which is the registrant for Zeitgeist Engine. Status in Zeitgeist Engine: Triaged Bug description: Running python test/resonance-engine-test.py I get one failed testcase ====================================================================== FAIL: testInsertGetWithoutTimestamp (__main__.ZeitgeistEngineTest) ---------------------------------------------------------------------- Traceback (most recent call last): File "test/resonance-engine-test.py", line 87, in testInsertGetWithoutTimestamp self.assertTrue(ev.timestamp) AssertionError ---------------------------------------------------------------------- I would have fixed this if I were totally sure what the correct fix is. Right now the default timestamp of an event is an empty string, which is why this testcase fails. Is this the expected behaviour, and should we fix the test. Or should the default timestamp be a string containing the timestamp of object creation time? _______________________________________________ Mailing list: https://launchpad.net/~zeitgeist Post to : zeitgeist@lists.launchpad.net Unsubscribe : https://launchpad.net/~zeitgeist More help : https://help.launchpad.net/ListHelp
