]> git.ipfire.org Git - thirdparty/freeradius-server.git/commitdiff
more quote arguments
authorAlan T. DeKok <aland@freeradius.org>
Sat, 5 Apr 2025 14:42:29 +0000 (10:42 -0400)
committerAlan T. DeKok <aland@freeradius.org>
Sat, 5 Apr 2025 14:42:29 +0000 (10:42 -0400)
src/tests/modules/redis/cluster_key.unlang
src/tests/modules/redis/cluster_reset.inc
src/tests/modules/redis/functions.unlang

index 38630a573ffc209b8cf30a2ad39fd61e064ee08c..28613142f1df379e3610a356b0af5085ac846edf 100644 (file)
@@ -19,7 +19,7 @@ testdata2 := "2-%randstr('aaaaaaaa')"
 testdata3 := "3-%randstr('aaaaaaaa')"
 
 #  Hashes to Redis cluster node master 1 (1)
-if (%redis(SET, b, %{testdata1}) == 'OK') {
+if (%redis('SET', 'b', %{testdata1}) == 'OK') {
        test_pass
 } else {
        test_fail
index b9b98083e7ab94fd95c24492b2393c15b9fb6de7..ff48c651f465da7355e56fc707e86d8c6cc04824 100644 (file)
@@ -53,7 +53,7 @@ foreach i (%range(20)) {
        #
        #  Force a remap as the slaves don't show up in the cluster immediately
        #
-       if (%redis.remap(%{redisserver}:30001) == 'success') {
+       if (%redis.remap("%{redisserver}:30001") == 'success') {
                #  Hashes to Redis cluster node master 0 (1)
                if ((%redis('SET', 'b', %{data1}) == 'OK') && \
                    (%redis('SET', 'c', %{data2}) == 'OK') && \
@@ -63,9 +63,9 @@ foreach i (%range(20)) {
                        #  so we now need to figure out which slave each of those keys
                        #  ended up on.
                        #
-                       if ((%redis("-@%redis.node(b, 1)", 'GET', 'b') == data1) && \
-                           (%redis("-@%redis.node(c, 1)", 'GET', 'c') == data2) && \
-                           (%redis("-@%redis.node(d, 1)", 'GET', 'd') == data3)) {
+                       if ((%redis("-@%redis.node('b', 1)", 'GET', 'b') == data1) && \
+                           (%redis("-@%redis.node('c', 1)", 'GET', 'c') == data2) && \
+                           (%redis("-@%redis.node('d', 1)", 'GET', 'd') == data3)) {
                                break
                        }
                }
index 8ec4096294d2b90175c4c260eee4da007ccacb57..c54bdf129a903efebcad059f7b109a35fe83b4bc 100644 (file)
@@ -20,12 +20,12 @@ if (!(%redis.hello_world(0, %{dummy_string}) == 'hello world')) {
        test_fail
 }
 
-if (!(%redis.concat_args_keys(1, foo, bar, baz) == 'foo,bar,baz')) {
+if (!(%redis.concat_args_keys(1, 'foo', 'bar', 'baz') == 'foo,bar,baz')) {
        test_fail
 }
 
 # Concat with an empty argument.  This is a regression test
-if (!(%redis.concat_args_keys(1, foo, %{dummy_string}, baz) == 'foo,,baz')) {
+if (!(%redis.concat_args_keys(1, 'foo', %{dummy_string}, 'baz') == 'foo,,baz')) {
        test_fail
 }