summaryrefslogtreecommitdiffstats
path: root/tests
diff options
context:
space:
mode:
authorPablo Neira Ayuso <pablo@netfilter.org>2023-11-21 11:27:02 +0100
committerPablo Neira Ayuso <pablo@netfilter.org>2023-11-21 15:44:04 +0100
commit25b0ac520234cb3326c5cdadbea6b0b80666a338 (patch)
tree88c684f670fb92b6077b2844a42c8aa2fd620669 /tests
parent566182cd8259cfc22f9fd949e9ad3c15ed6a7945 (diff)
tests: shell: quote reference to array to iterate over empty string
This patch restores coverage for non-interval set backend. Use "${FLAGS[@]}" in loop, otherwise empty string is skipped in the iteration. This snippet: FLAGS=("") available_flags FLAGS "single" for flags in "${FLAGS[@]}" ; do echo $flags done ... now shows the empty string: # bash test.sh interval Fixes: ed927baa4fd8 ("tests: shell: skip pipapo set backend in transactions/30s-stress") Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'tests')
-rwxr-xr-xtests/shell/testcases/transactions/30s-stress2
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/shell/testcases/transactions/30s-stress b/tests/shell/testcases/transactions/30s-stress
index ef3e1d3e..e92b9226 100755
--- a/tests/shell/testcases/transactions/30s-stress
+++ b/tests/shell/testcases/transactions/30s-stress
@@ -324,7 +324,7 @@ random_elem()
FLAGS=("")
available_flags FLAGS $key
- for flags in ${FLAGS[@]} ; do
+ for flags in "${FLAGS[@]}" ; do
cnt=$((cnt+1))
if [ $f -ne fkr ] ;then
f=$((f+1))