caleb-brown/redict
1
0
Fork 0
mirror of https://codeberg.org/redict/redict.git synced 2025-01-22 08:08:53 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix replica count check in migration tests. (#10140)

Browse Source 
Tests were not using loop index as node id, checking replica count of the same node over and over.
This commit is contained in:
Ozan Tezcan 2022-01-19 12:36:24 +03:00 committed by GitHub
parent 1af0a2c5ae
commit 72e1b5de4d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/cluster/tests/07-replica-migration.tcl
View File

@ -18,7 +18,7 @@ test "Each master should have two replicas attached" {
foreach_redis_id id {
if {$id < 5} {
wait_for_condition 1000 50 {
[llength [lindex [R 0 role] 2]] == 2
[llength [lindex [R $id role] 2]] == 2
} else {
fail "Master #$id does not have 2 slaves as expected"
}

2
tests/cluster/tests/12-replica-migration-2.tcl
View File

@ -21,7 +21,7 @@ test "Each master should have at least two replicas attached" {
foreach_redis_id id {
if {$id < 5} {
wait_for_condition 1000 50 {
[llength [lindex [R 0 role] 2]] >= 2
[llength [lindex [R $id role] 2]] >= 2
} else {
fail "Master #$id does not have 2 slaves as expected"
}

4
tests/cluster/tests/12.1-replica-migration-3.tcl
View File

@ -19,7 +19,7 @@ test "Each master should have at least two replicas attached" {
foreach_redis_id id {
if {$id < 5} {
wait_for_condition 1000 50 {
[llength [lindex [R 0 role] 2]] >= 2
[llength [lindex [R $id role] 2]] >= 2
} else {
fail "Master #$id does not have 2 slaves as expected"
}
@ -61,7 +61,7 @@ test "Each master should have at least two replicas attached" {
foreach_redis_id id {
if {$id < 5} {
wait_for_condition 1000 50 {
[llength [lindex [R 0 role] 2]] >= 2
[llength [lindex [R $id role] 2]] >= 2
} else {
fail "Master #$id does not have 2 slaves as expected"
}