[Zodb-checkins] SVN: ZODB/branches/3.4/ checkCrossDBInvalidations: make this willing to wait longer in all,

Tim Peters tim.one at comcast.net
Mon Jun 6 12:32:14 EDT 2005


Log message for revision 30658:
  checkCrossDBInvalidations:  make this willing to wait longer in all,
  and increase the sleep time per failing loop iteration.  The hope is
  this will allow the test to pass on slower and/or "too busy" machines.
  I can't test that, though, since I couldn't get the test to fail at
  all on a Windows or a Linux box.
  

Changed:
  U   ZODB/branches/3.4/NEWS.txt
  U   ZODB/branches/3.4/src/ZEO/tests/ConnectionTests.py

-=-
Modified: ZODB/branches/3.4/NEWS.txt
===================================================================
--- ZODB/branches/3.4/NEWS.txt	2005-06-06 15:20:49 UTC (rev 30657)
+++ ZODB/branches/3.4/NEWS.txt	2005-06-06 16:32:13 UTC (rev 30658)
@@ -40,7 +40,14 @@
   error (for example, a corrupt database can cause this), the message it
   produces no longer contains unprintable characters.
 
+Tests
+-----
 
+- (3.4c2) ``checkCrossDBInvalidations`` suffered spurious failures too often
+  on slow and/or busy machines.  The test is willing to wait longer for
+  success now.
+
+
 What's new in ZODB3 3.4b1?
 ==========================
 Release date: 19-May-2005

Modified: ZODB/branches/3.4/src/ZEO/tests/ConnectionTests.py
===================================================================
--- ZODB/branches/3.4/src/ZEO/tests/ConnectionTests.py	2005-06-06 15:20:49 UTC (rev 30657)
+++ ZODB/branches/3.4/src/ZEO/tests/ConnectionTests.py	2005-06-06 16:32:13 UTC (rev 30658)
@@ -651,10 +651,13 @@
         transaction.commit()
 
         # Make sure the invalidation is received in the other client.
-        for i in range(10):
+        # We've had problems with this timing out on "slow" and/or "very
+        # busy" machines, so we increase the sleep time on each trip, and
+        # are willing to wait quite a long time.
+        for i in range(20):
             if r1._p_state == -1:
                 break
-            time.sleep(0.1)
+            time.sleep(i / 10.0)
         self.assertEqual(r1._p_state, -1) # ghost
 
         r1.keys() # unghostify



More information about the Zodb-checkins mailing list