Unverified Commit fb3457d1 authored by Oran Agra's avatar Oran Agra Committed by GitHub
Browse files

minor test suite cleanup, revive old test (#8497)

There are two tests in other.tcl that were dependant of the sha1 package
import which meant that they didn't usually run.
The reason it was like that was that prior to the creation of DEBUG
DIGEST, the test suite used to have an equivalent function, but that's
no longer the case and this dependency isn't needed.

The other change is to revert config changes done by the test before the
test suite continues. can be useful if using `--host` to run multiple
units against the same server
parent b1929bb2
...@@ -55,7 +55,7 @@ start_server {tags {"other"}} { ...@@ -55,7 +55,7 @@ start_server {tags {"other"}} {
} {*index is out of range*} } {*index is out of range*}
tags {consistency} { tags {consistency} {
if {![catch {package require sha1}]} { if {true} {
if {$::accurate} {set numops 10000} else {set numops 1000} if {$::accurate} {set numops 10000} else {set numops 1000}
test {Check consistency of different data types after a reload} { test {Check consistency of different data types after a reload} {
r flushdb r flushdb
......
...@@ -468,6 +468,7 @@ start_server {tags {"scripting"}} { ...@@ -468,6 +468,7 @@ start_server {tags {"scripting"}} {
r slaveof no one r slaveof no one
set res set res
} {102} } {102}
r config set aof-use-rdb-preamble yes
test {EVAL timeout from AOF} { test {EVAL timeout from AOF} {
# generate a long running script that is propagated to the AOF as script # generate a long running script that is propagated to the AOF as script
...@@ -505,8 +506,6 @@ start_server {tags {"scripting"}} { ...@@ -505,8 +506,6 @@ start_server {tags {"scripting"}} {
$rd close $rd close
r get x r get x
} {y} } {y}
r config set aof-use-rdb-preamble yes
r config set lua-replicate-commands yes
test {We can call scripts rewriting client->argv from Lua} { test {We can call scripts rewriting client->argv from Lua} {
r del myset r del myset
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment