Parcourir la source

[master] Corrected intermittent D2 unit test failure

On sluggish machines D2UpdateMgrTest.multiTransactionTimeout
failed intermittantly. A run-away test check was a bit too
stringent.
Thomas Markwalder il y a 11 ans
Parent
commit
5a2b69ed6f
1 fichiers modifiés avec 4 ajouts et 4 suppressions
  1. 4 4
      src/bin/d2/tests/d2_update_mgr_unittests.cc

+ 4 - 4
src/bin/d2/tests/d2_update_mgr_unittests.cc

@@ -213,7 +213,7 @@ public:
     /// vary.
     void processAll(unsigned int timeout_millisec =
                     NameChangeTransaction::DNS_UPDATE_DEFAULT_TIMEOUT + 100,
-                    size_t max_passes = 20) {
+                    size_t max_passes = 100) {
         // Loop until all the transactions have been dequeued and run through to
         // completion.
         size_t passes = 0;
@@ -238,9 +238,9 @@ public:
             }
 
             // This is a last resort fail safe to ensure we don't go around
-            // forever. We cut it off the number of passes at 20.  This is
-            // roughly twice the number for the longest test (currently,
-            // multiTransactionTimeout).
+            // forever. We cut it off the number of passes at 100 (default
+            // value).  This is roughly ten times the number for the longest
+            // test (currently, multiTransactionTimeout).
             if (passes > max_passes) {
                 ADD_FAILURE() << "processALL failed, too many passes: "
                     << passes <<  ", total handlers executed: " << handlers;