From: Alan T. DeKok <aland@freeradius.org>
Date: Sat, 5 Apr 2025 14:42:29 +0000 (-0400)
Subject: more quote arguments
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=bcc56b887ba2834b213c87cd2dbabc4a1a030b0c;p=thirdparty%2Ffreeradius-server.git

more quote arguments
---

diff --git a/src/tests/modules/redis/cluster_key.unlang b/src/tests/modules/redis/cluster_key.unlang
index 38630a573ff..28613142f1d 100644
--- a/src/tests/modules/redis/cluster_key.unlang
+++ b/src/tests/modules/redis/cluster_key.unlang
@@ -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
diff --git a/src/tests/modules/redis/cluster_reset.inc b/src/tests/modules/redis/cluster_reset.inc
index b9b98083e7a..ff48c651f46 100644
--- a/src/tests/modules/redis/cluster_reset.inc
+++ b/src/tests/modules/redis/cluster_reset.inc
@@ -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
 			}
 		}
diff --git a/src/tests/modules/redis/functions.unlang b/src/tests/modules/redis/functions.unlang
index 8ec4096294d..c54bdf129a9 100644
--- a/src/tests/modules/redis/functions.unlang
+++ b/src/tests/modules/redis/functions.unlang
@@ -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
 }