From 52f9291f797a7f24267115bedb396f16310456ed Mon Sep 17 00:00:00 2001 From: Shivshankar Date: Thu, 25 Apr 2024 03:13:21 -0400 Subject: [PATCH] Rename redis to valkey in test suite logs and test names. (#366) This PR covers below cases. 1. test suite's prints(i.e., puts statement logs). 2. Links refering to redis issues. 3. test names contains redis. Signed-off-by: Shivshankar-Reddy --- tests/instances.tcl | 20 ++++++++++---------- tests/integration/valkey-cli.tcl | 2 +- tests/sentinel/tests/00-base.tcl | 2 +- tests/sentinel/tests/02-slaves-reconf.tcl | 2 +- tests/sentinel/tests/05-manual.tcl | 2 +- tests/sentinel/tests/12-master-reboot.tcl | 2 +- tests/support/cluster.tcl | 4 ++-- tests/support/server.tcl | 2 +- tests/support/test.tcl | 2 +- tests/support/valkey.tcl | 2 +- tests/unit/expire.tcl | 2 +- tests/unit/geo.tcl | 4 ++-- tests/unit/introspection-2.tcl | 2 +- tests/unit/introspection.tcl | 6 +++--- tests/unit/maxmemory.tcl | 2 +- tests/unit/moduleapi/moduleauth.tcl | 2 +- tests/unit/other.tcl | 6 +++--- tests/unit/printver.tcl | 2 +- tests/unit/protocol.tcl | 2 +- tests/unit/pubsub.tcl | 2 +- tests/unit/scripting.tcl | 2 +- 21 files changed, 36 insertions(+), 36 deletions(-) diff --git a/tests/instances.tcl b/tests/instances.tcl index baeaddfe2..5b90a6009 100644 --- a/tests/instances.tcl +++ b/tests/instances.tcl @@ -40,7 +40,7 @@ set ::loop 0 if {[catch {cd tmp}]} { puts "tmp directory not found." - puts "Please run this test from the Redis source root." + puts "Please run this test from the Valkey source root." exit 1 } @@ -313,7 +313,7 @@ proc parse_options {} { puts "--fail Simulate a test failure." puts "--valgrind Run with valgrind." puts "--tls Run tests in TLS mode." - puts "--tls-module Run tests in TLS mode with Redis module." + puts "--tls-module Run tests in TLS mode with Valkey module." puts "--host Use hostname instead of 127.0.0.1." puts "--config Extra config argument(s)." puts "--stop Blocks once the first test fails." @@ -342,12 +342,12 @@ proc pause_on_error {} { set cmd [lindex $argv 0] if {$cmd eq {continue}} { break - } elseif {$cmd eq {show-redis-logs}} { + } elseif {$cmd eq {show-valkey-logs}} { set count 10 if {[lindex $argv 1] ne {}} {set count [lindex $argv 1]} foreach_valkey_id id { - puts "=== REDIS $id ====" - puts [exec tail -$count redis_$id/log.txt] + puts "=== VALKEY $id ====" + puts [exec tail -$count valkey_$id/log.txt] puts "---------------------\n" } } elseif {$cmd eq {show-sentinel-logs}} { @@ -360,7 +360,7 @@ proc pause_on_error {} { } } elseif {$cmd eq {ls}} { foreach_valkey_id id { - puts -nonewline "Redis $id" + puts -nonewline "Valkey $id" set errcode [catch { set str {} append str "@[RI $id tcp_port]: " @@ -391,13 +391,13 @@ proc pause_on_error {} { } } } elseif {$cmd eq {help}} { - puts "ls List Sentinel and Redis instances." + puts "ls List Sentinel and Valkey instances." puts "show-sentinel-logs \[N\] Show latest N lines of logs." - puts "show-redis-logs \[N\] Show latest N lines of logs." + puts "show-valkey-logs \[N\] Show latest N lines of logs." puts "S cmd ... arg Call command in Sentinel ." - puts "R cmd ... arg Call command in Redis ." + puts "R cmd ... arg Call command in Valkey ." puts "SI Show Sentinel INFO ." - puts "RI Show Redis INFO ." + puts "RI Show Valkey INFO ." puts "continue Resume test." } else { set errcode [catch {eval $line} retval] diff --git a/tests/integration/valkey-cli.tcl b/tests/integration/valkey-cli.tcl index 04b91caf8..066a70a6a 100644 --- a/tests/integration/valkey-cli.tcl +++ b/tests/integration/valkey-cli.tcl @@ -375,7 +375,7 @@ if {!$::tls} { ;# fake_redis_node doesn't support TLS [catch {close [socket "127.0.0.1" $port1]}] == 0 && \ [catch {close [socket "127.0.0.1" $port2]}] == 0 } else { - fail "Failed to start fake Redis nodes" + fail "Failed to start fake Valkey nodes" } # Run the cli assert_equal "OK" [run_cli_host_port_db "127.0.0.1" $port1 0 -c SET foo bar] diff --git a/tests/sentinel/tests/00-base.tcl b/tests/sentinel/tests/00-base.tcl index 274a84316..ee88469ec 100644 --- a/tests/sentinel/tests/00-base.tcl +++ b/tests/sentinel/tests/00-base.tcl @@ -104,7 +104,7 @@ test "All the other slaves now point to the new master" { wait_for_condition 1000 50 { [RI $id master_port] == [lindex $addr 1] } else { - fail "Redis ID $id not configured to replicate with new master" + fail "Valkey ID $id not configured to replicate with new master" } } } diff --git a/tests/sentinel/tests/02-slaves-reconf.tcl b/tests/sentinel/tests/02-slaves-reconf.tcl index 5bbd80fec..32e18d0eb 100644 --- a/tests/sentinel/tests/02-slaves-reconf.tcl +++ b/tests/sentinel/tests/02-slaves-reconf.tcl @@ -18,7 +18,7 @@ proc 02_test_slaves_replication {} { ([RI $id master_port] == $master_port) && ([RI $id master_link_status] eq {up}) } else { - fail "Redis slave $id is replicating from wrong master" + fail "Valkey slave $id is replicating from wrong master" } } } diff --git a/tests/sentinel/tests/05-manual.tcl b/tests/sentinel/tests/05-manual.tcl index f3cfb1e3a..183408564 100644 --- a/tests/sentinel/tests/05-manual.tcl +++ b/tests/sentinel/tests/05-manual.tcl @@ -48,7 +48,7 @@ test "All the other slaves now point to the new master" { wait_for_condition 1000 50 { [RI $id master_port] == [lindex $addr 1] } else { - fail "Redis ID $id not configured to replicate with new master" + fail "Valkey ID $id not configured to replicate with new master" } } } diff --git a/tests/sentinel/tests/12-master-reboot.tcl b/tests/sentinel/tests/12-master-reboot.tcl index df6ff49b7..ad536f7e3 100644 --- a/tests/sentinel/tests/12-master-reboot.tcl +++ b/tests/sentinel/tests/12-master-reboot.tcl @@ -88,7 +88,7 @@ test "All the other slaves now point to the new master" { wait_for_condition 1000 50 { [RI $id master_port] == [lindex $addr 1] } else { - fail "Redis ID $id not configured to replicate with new master" + fail "Valkey ID $id not configured to replicate with new master" } } } diff --git a/tests/support/cluster.tcl b/tests/support/cluster.tcl index 5027bacb7..2b9e44f64 100644 --- a/tests/support/cluster.tcl +++ b/tests/support/cluster.tcl @@ -201,7 +201,7 @@ proc ::valkey_cluster::__dispatch__ {id method args} { # Get the keys from the command. set keys [::valkey_cluster::get_keys_from_command $method $args] if {$keys eq {}} { - error "Redis command '$method' is not supported by valkey_cluster." + error "Valkey command '$method' is not supported by valkey_cluster." } # Resolve the keys in the corresponding hash slot they hash to. @@ -250,7 +250,7 @@ proc ::valkey_cluster::__dispatch__ {id method args} { return $e } } - error "Too many redirections or failures contacting Redis Cluster." + error "Too many redirections or failures contacting Valkey Cluster." } else { uplevel 1 [list ::valkey_cluster::__method__$method $id] $args } diff --git a/tests/support/server.tcl b/tests/support/server.tcl index 813f240bf..ddcdbcddb 100644 --- a/tests/support/server.tcl +++ b/tests/support/server.tcl @@ -4,7 +4,7 @@ set ::valgrind_errors {} proc start_server_error {config_file error} { set err {} - append err "Can't start the Redis server\n" + append err "Can't start the Valkey server\n" append err "CONFIGURATION:\n" append err [exec cat $config_file] append err "\nERROR:\n" diff --git a/tests/support/test.tcl b/tests/support/test.tcl index 3f2200b11..9c959d9de 100644 --- a/tests/support/test.tcl +++ b/tests/support/test.tcl @@ -258,7 +258,7 @@ proc test {name code {okpattern undefined} {tags {}}} { } if {$::traceleaks} { - set output [exec leaks redis-server] + set output [exec leaks valkey-server] if {![string match {*0 leaks*} $output]} { send_data_packet $::test_server_fd err "Detected a memory leak in test '$name': $output" } diff --git a/tests/support/valkey.tcl b/tests/support/valkey.tcl index 9797fc014..6f8727ace 100644 --- a/tests/support/valkey.tcl +++ b/tests/support/valkey.tcl @@ -19,7 +19,7 @@ # } # } # -# set r [redis] +# set r [valkey] # $r blocking 0 # $r get fo [list handlePong] # diff --git a/tests/unit/expire.tcl b/tests/unit/expire.tcl index e34686a4e..ff97a475e 100644 --- a/tests/unit/expire.tcl +++ b/tests/unit/expire.tcl @@ -455,7 +455,7 @@ start_server {tags {"expire"}} { # to replication lag. # 2) We want to unify the way TTLs are replicated in both RDB and replication # stream, which is as absolute timestamps. - # See: https://github.com/redis/redis/issues/8433 + # See Redis OSS issue: https://github.com/redis/redis/issues/8433 r flushall ; # Clean up keyspace to avoid interference by keys from other tests set repl [attach_to_replication_stream] diff --git a/tests/unit/geo.tcl b/tests/unit/geo.tcl index 19af86399..1e87e1eb4 100644 --- a/tests/unit/geo.tcl +++ b/tests/unit/geo.tcl @@ -656,7 +656,7 @@ start_server {tags {"geo"}} { if {$res != $res2} { set diff [compare_lists $res $res2] puts "*** Possible problem in GEO radius query ***" - puts "Redis: $res" + puts "Valkey: $res" puts "Tcl : $res2" puts "Diff : $diff" puts [join $debuginfo "\n"] @@ -664,7 +664,7 @@ start_server {tags {"geo"}} { if {[lsearch -exact $res2 $place] != -1} { set where "(only in Tcl)" } else { - set where "(only in Redis)" + set where "(only in Valkey)" } lassign [lindex [r geopos mypoints $place] 0] lon lat set mydist [geo_distance $lon $lat $search_lon $search_lat] diff --git a/tests/unit/introspection-2.tcl b/tests/unit/introspection-2.tcl index 89cb4b122..b8f4e0aed 100644 --- a/tests/unit/introspection-2.tcl +++ b/tests/unit/introspection-2.tcl @@ -118,7 +118,7 @@ start_server {tags {"introspection"}} { assert_match {*calls=1,*} [cmdstat geoadd] } {} {needs:config-resetstat} - test {COMMAND COUNT get total number of Redis commands} { + test {COMMAND COUNT get total number of commands} { assert_morethan [r command count] 0 } diff --git a/tests/unit/introspection.tcl b/tests/unit/introspection.tcl index 78ae945dd..582a019d2 100644 --- a/tests/unit/introspection.tcl +++ b/tests/unit/introspection.tcl @@ -1,8 +1,8 @@ start_server {tags {"introspection"}} { test "PING" { assert_equal {PONG} [r ping] - assert_equal {redis} [r ping redis] - assert_error {*wrong number of arguments for 'ping' command} {r ping hello redis} + assert_equal {valkey} [r ping valkey] + assert_error {*wrong number of arguments for 'ping' command} {r ping hello valkey} } test {CLIENT LIST} { @@ -739,7 +739,7 @@ start_server {tags {"introspection"}} { catch {exec src/valkey-server --shutdown-on-sigint "now force" --shutdown-on-sigterm} err assert_match {*'shutdown-on-sigterm'*argument(s) must be one of the following*} $err - # Something like `redis-server --some-config --config-value1 --config-value2 --loglevel debug` would break, + # Something like `valkey-server --some-config --config-value1 --config-value2 --loglevel debug` would break, # because if you want to pass a value to a config starting with `--`, it can only be a single value. catch {exec src/valkey-server --replicaof 127.0.0.1 abc} err assert_match {*'replicaof "127.0.0.1" "abc"'*Invalid master port*} $err diff --git a/tests/unit/maxmemory.tcl b/tests/unit/maxmemory.tcl index 19b05cc9c..92e68ac1e 100644 --- a/tests/unit/maxmemory.tcl +++ b/tests/unit/maxmemory.tcl @@ -225,7 +225,7 @@ start_server {tags {"maxmemory external:skip"}} { } # If we add the same number of keys already added again and # the policy is allkeys-* we should still be under the limit. - # Otherwise we should see an error reported by Redis. + # Otherwise we should see an error reported by Server. set err 0 for {set j 0} {$j < $numkeys} {incr j} { if {[catch {r set [randomKey] x} e]} { diff --git a/tests/unit/moduleapi/moduleauth.tcl b/tests/unit/moduleapi/moduleauth.tcl index d7399d0ff..745285312 100644 --- a/tests/unit/moduleapi/moduleauth.tcl +++ b/tests/unit/moduleapi/moduleauth.tcl @@ -286,7 +286,7 @@ start_server {tags {"modules"}} { assert_match {*calls=2,*,rejected_calls=0,failed_calls=1} [cmdstat auth] } - test {Disabling Redis User during blocking module auth} { + test {Disabling Valkey User during blocking module auth} { r config resetstat r acl setuser foo >pwd on ~* &* +@all set rd [valkey_deferring_client] diff --git a/tests/unit/other.tcl b/tests/unit/other.tcl index 72781838d..503ee391a 100644 --- a/tests/unit/other.tcl +++ b/tests/unit/other.tcl @@ -466,7 +466,7 @@ start_server {tags {"other external:skip"}} { start_cluster 1 0 {tags {"other external:skip cluster slow"}} { r config set dynamic-hz no hz 500 - test "Redis can trigger resizing" { + test "Server can trigger resizing" { r flushall # hashslot(foo) is 12182 for {set j 1} {$j <= 128} {incr j} { @@ -496,7 +496,7 @@ start_cluster 1 0 {tags {"other external:skip cluster slow"}} { } } {} {needs:debug} - test "Redis can rewind and trigger smaller slot resizing" { + test "Server can rewind and trigger smaller slot resizing" { # hashslot(foo) is 12182 # hashslot(alice) is 749, smaller than hashslot(foo), # attempt to trigger a resize on it, see details in #12802. @@ -526,7 +526,7 @@ start_cluster 1 0 {tags {"other external:skip cluster slow"}} { } start_server {tags {"other external:skip"}} { - test "Redis can resize empty dict" { + test "Server can resize empty dict" { # Write and then delete 128 keys, creating an empty dict r flushall for {set j 1} {$j <= 128} {incr j} { diff --git a/tests/unit/printver.tcl b/tests/unit/printver.tcl index c80f45144..53cfed8ee 100644 --- a/tests/unit/printver.tcl +++ b/tests/unit/printver.tcl @@ -2,5 +2,5 @@ start_server {} { set i [r info] regexp {redis_version:(.*?)\r\n} $i - version regexp {redis_git_sha1:(.*?)\r\n} $i - sha1 - puts "Testing Redis version $version ($sha1)" + puts "Testing Valkey version $version ($sha1)" } diff --git a/tests/unit/protocol.tcl b/tests/unit/protocol.tcl index cd7dbe9e5..f3a2b8e1a 100644 --- a/tests/unit/protocol.tcl +++ b/tests/unit/protocol.tcl @@ -94,7 +94,7 @@ start_server {tags {"protocol network"}} { set elapsed [expr {[clock seconds]-$test_start}] if {$elapsed > $test_time_limit} { close $s - error "assertion:Redis did not closed connection after protocol desync" + error "assertion:Valkey did not closed connection after protocol desync" } } } diff --git a/tests/unit/pubsub.tcl b/tests/unit/pubsub.tcl index 5a2439814..2f336dfcb 100644 --- a/tests/unit/pubsub.tcl +++ b/tests/unit/pubsub.tcl @@ -483,7 +483,7 @@ start_server {tags {"pubsub network"}} { r hello 3 # Note: SUBSCRIBE and UNSUBSCRIBE with multiple channels in the same command, - # breaks the multi response, see https://github.com/redis/redis/issues/12207 + # breaks the multi response, see Redis OSS issue: https://github.com/redis/redis/issues/12207 # this is just a temporary sanity test to detect unintended breakage. # subscribe for 3 channels actually emits 3 "responses" diff --git a/tests/unit/scripting.tcl b/tests/unit/scripting.tcl index 126f4eba4..ea5c1b899 100644 --- a/tests/unit/scripting.tcl +++ b/tests/unit/scripting.tcl @@ -1055,7 +1055,7 @@ start_server {tags {"scripting"}} { test "Try trick global protection 3" { catch { run_script { - valkey = function() return 1 end + redis = function() return 1 end } 0 } e set _ $e