Commit 6fe36b98 authored by Jim Fulton's avatar Jim Fulton

Merge pull request #62 from...

Merge pull request #62 from zopefoundation/dont-depend-on-implementation-detail-in-tpc_finish-test-assertions

Fixed a test that depended on some off behavior of tpc_finish in some…
parents 96183018 bf877dd2
...@@ -286,10 +286,9 @@ class TransactionalUndoStorage: ...@@ -286,10 +286,9 @@ class TransactionalUndoStorage:
t = Transaction() t = Transaction()
oids = self._begin_undos_vote(t, tid, tid1) oids = self._begin_undos_vote(t, tid, tid1)
self._storage.tpc_finish(t) self._storage.tpc_finish(t)
# We get the finalization stuff called an extra time: # We may get the finalization stuff called an extra time,
eq(len(oids), 4) # depending on the implementation.
unless(oid1 in oids) self.assertEqual(set(oids), set((oid1, oid2)))
unless(oid2 in oids)
data, revid1 = self._storage.load(oid1, '') data, revid1 = self._storage.load(oid1, '')
eq(zodb_unpickle(data), MinPO(30)) eq(zodb_unpickle(data), MinPO(30))
data, revid2 = self._storage.load(oid2, '') data, revid2 = self._storage.load(oid2, '')
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment