summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorOran Agra <oran@redislabs.com>2018-11-11 09:22:42 +0200
committerOran Agra <oran@redislabs.com>2018-11-12 10:26:10 +0200
commitd0850369c4a06d6362dbaf12d873e54d6ce931cc (patch)
tree708b47aff5fd5d99a06696e84fdc731f0c21ffe4
parent7721fe83e92dee581f8703e178f8368dce86abdf (diff)
downloadredis-d0850369c4a06d6362dbaf12d873e54d6ce931cc.tar.gz
fix small test suite race conditions
-rw-r--r--tests/support/util.tcl8
-rw-r--r--tests/unit/memefficiency.tcl1
-rw-r--r--tests/unit/type/stream-cgroups.tcl2
3 files changed, 11 insertions, 0 deletions
diff --git a/tests/support/util.tcl b/tests/support/util.tcl
index 181c865fc..74f491e48 100644
--- a/tests/support/util.tcl
+++ b/tests/support/util.tcl
@@ -91,6 +91,14 @@ proc wait_for_sync r {
}
}
+proc wait_for_ofs_sync {r1 r2} {
+ wait_for_condition 50 100 {
+ [status $r1 master_repl_offset] eq [status $r2 master_repl_offset]
+ } else {
+ fail "replica didn't sync in time"
+ }
+}
+
# Random integer between 0 and max (excluded).
proc randomInt {max} {
expr {int(rand()*$max)}
diff --git a/tests/unit/memefficiency.tcl b/tests/unit/memefficiency.tcl
index 8972d577a..d152e212c 100644
--- a/tests/unit/memefficiency.tcl
+++ b/tests/unit/memefficiency.tcl
@@ -90,6 +90,7 @@ start_server {tags {"defrag"}} {
test "Active defrag big keys" {
r flushdb
r config resetstat
+ r config set save "" ;# prevent bgsave from interfereing with save below
r config set activedefrag no
r config set active-defrag-max-scan-fields 1000
r config set active-defrag-threshold-lower 5
diff --git a/tests/unit/type/stream-cgroups.tcl b/tests/unit/type/stream-cgroups.tcl
index d2e0d6539..b96bc8e98 100644
--- a/tests/unit/type/stream-cgroups.tcl
+++ b/tests/unit/type/stream-cgroups.tcl
@@ -144,6 +144,8 @@ start_server {
}
}
+ wait_for_ofs_sync $master $slave
+
# Turn slave into master
$slave slaveof no one