gvsafronov/futriix
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

If the computer running the Redis test is slow, we revert to --clients 1 to avoid false positives.

Browse Source
This commit is contained in:
antirez 2012-05-11 17:26:09 +02:00
parent a7be5ac4fe
commit 9d2a56f2d9
1 changed files with 17 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
tests/test_helper.tcl
View File

@ -404,6 +404,18 @@ for {set j 0} {$j < [llength $argv]} {incr j} {
} }
} }
# With the parallel test running multiple Redis instances at the same time
# we need a fast enough computer, otherwise a lot of tests may generate
# false positives.
# If the computer is too slow we revert the sequetial test without any
# parallelism, that is, clients == 1.
proc is_a_slow_computer {} {
set start [clock milliseconds]
for {set j 0} {$j < 1000000} {incr j} {}
set elapsed [expr [clock milliseconds]-$start]
expr {$elapsed > 200}
}
if {$::client} { if {$::client} {
if {[catch { test_client_main $::test_server_port } err]} { if {[catch { test_client_main $::test_server_port } err]} {
set estr "Executing test client: $err.\n$::errorInfo" set estr "Executing test client: $err.\n$::errorInfo"
@ -413,6 +425,11 @@ if {$::client} {
exit 1 exit 1
} }
} else { } else {
if {[is_a_slow_computer]} {
puts "** SLOW COMPUTER ** Using a single client to avoid false positives."
set ::numclients 1
}
if {[catch { test_server_main } err]} { if {[catch { test_server_main } err]} {
if {[string length $err] > 0} { if {[string length $err] > 0} {
# only display error when not generated by the test suite # only display error when not generated by the test suite