Commit b53a1025 authored by Jason Madden's avatar Jason Madden

Let checkQuickVerificationWith2Clients use a longer timeaut and raise an...

Let checkQuickVerificationWith2Clients use a longer timeaut and raise an assertion error if the timeout fails. Trying to debug intermittant failures on Travis, seen in all versions of Python.
parent e57c61d1
...@@ -137,7 +137,7 @@ class CommonSetupTearDown(StorageTestBase): ...@@ -137,7 +137,7 @@ class CommonSetupTearDown(StorageTestBase):
os.waitpid(pid, 0) os.waitpid(pid, 0)
except OSError: except OSError:
pass # The subprocess module may already have waited pass # The subprocess module may already have waited
for c in self.caches: for c in self.caches:
for i in 0, 1: for i in 0, 1:
for ext in "", ".trace", ".lock": for ext in "", ".trace", ".lock":
...@@ -619,8 +619,9 @@ class InvqTests(CommonSetupTearDown): ...@@ -619,8 +619,9 @@ class InvqTests(CommonSetupTearDown):
perstorage = self.openClientStorage(cache="test") perstorage = self.openClientStorage(cache="test")
forker.wait_until( forker.wait_until(
(lambda : perstorage.verify_result == "quick verification"), func=(lambda : perstorage.verify_result == "quick verification"),
onfail=(lambda : None)) timeout=60,
label="perstorage.verify_result to be quick verification")
self.assertEqual(perstorage.verify_result, "quick verification") self.assertEqual(perstorage.verify_result, "quick verification")
self.assertEqual(perstorage._server._last_invals, self.assertEqual(perstorage._server._last_invals,
...@@ -1119,7 +1120,7 @@ class TimeoutTests(CommonSetupTearDown): ...@@ -1119,7 +1120,7 @@ class TimeoutTests(CommonSetupTearDown):
self.assertRaises(ConflictError, storage.tpc_vote, t) self.assertRaises(ConflictError, storage.tpc_vote, t)
# Abort this one and try a transaction that should succeed. # Abort this one and try a transaction that should succeed.
storage.tpc_abort(t) storage.tpc_abort(t)
# Now do a store. # Now do a store.
obj.value = 11 obj.value = 11
t = Transaction() t = Transaction()
......
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