Redis test: More reliable BRPOPLPUSH replication test.

Now it uses the new wait_for_condition testing primitive. Also wait_for_condition implementation was fixed in this commit to properly escape the expr command and its argument.
2012-04-30 11:48:16 +02:00
parent 28500d193f
commit 9e066ae051
1 changed files with 5 additions and 2 deletions
@@ -10,8 +10,11 @@ start_server {tags {"repl"}} {
            set rd [redis_deferring_client]
            $rd brpoplpush a b 5
            r lpush a foo
-            after 1000
-            assert_equal [r debug digest] [r -1 debug digest]
+            wait_for_condition 50 100 {
+                [r debug digest] eq [r -1 debug digest]
+            } else {
+                fail "Master and slave have different digest: [r debug digest] VS [r -1 debug digest]"
+            }
        }

        test {BRPOPLPUSH replication, list exists} {