From 9118ef11a43c7146cc8950e602f59f197ba6c28f Mon Sep 17 00:00:00 2001
From: Antoine Pitrou <antoine@python.org>
Date: Mon, 2 Sep 2024 17:59:54 +0200
Subject: [PATCH] Address review comments

---
 ci/scripts/install_gcs_testbench.sh | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/ci/scripts/install_gcs_testbench.sh b/ci/scripts/install_gcs_testbench.sh
index 426fd536453ad..48a5858a358c9 100755
--- a/ci/scripts/install_gcs_testbench.sh
+++ b/ci/scripts/install_gcs_testbench.sh
@@ -47,13 +47,13 @@ fi
 export PIP_BREAK_SYSTEM_PACKAGES=1
 ${PIPX_BASE_PYTHON} -m pip install -U pipx
 
-pipx_flags="--verbose --python ${PIPX_PYTHON}"
+pipx_flags=(--verbose --python ${PIPX_PYTHON})
 if [[ $(id -un) == "root" ]]; then
   # Install globally as /root/.local/bin is typically not in $PATH
-  pipx_flags="${pipx_flags} --global"
+  pipx_flags+=(--global)
 fi
-if [[ ! -z "${PIPX_PIP_ARGS}" ]]; then
-  pipx_flags="${pipx_flags} --pip-args '${PIPX_PIP_ARGS}'"
+if [[ -n "${PIPX_PIP_ARGS}" ]]; then
+  pipx_flags+=(--pip-args "'${PIPX_PIP_ARGS}'")
 fi
-${PIPX_BASE_PYTHON} -m pipx install ${pipx_flags} \
+${PIPX_BASE_PYTHON} -m pipx install ${pipx_flags[@]} \
   "https://github.com/googleapis/storage-testbench/archive/${version}.tar.gz"