From 230da1cbe73800524d3fd3373b3eecb988d9b435 Mon Sep 17 00:00:00 2001
From: Farid Zakaria <farid.m.zakaria@gmail.com>
Date: Thu, 25 Sep 2025 10:36:12 -0700
Subject: [PATCH] shellcheck fix: tests/functional/export.sh

---
 maintainers/flake-module.nix |  1 -
 tests/functional/export.sh   | 16 +++++++++-------
 2 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/maintainers/flake-module.nix b/maintainers/flake-module.nix
index 7d593d26e..0a46dc57f 100644
--- a/maintainers/flake-module.nix
+++ b/maintainers/flake-module.nix
@@ -107,7 +107,6 @@
             excludes = [
               # We haven't linted these files yet
               ''^tests/functional/dump-db\.sh$''
-              ''^tests/functional/export\.sh$''
               ''^tests/functional/fetchClosure\.sh$''
               ''^tests/functional/fetchGit\.sh$''
               ''^tests/functional/fetchGitRefs\.sh$''
diff --git a/tests/functional/export.sh b/tests/functional/export.sh
index 3e895a540..53bbdd9ac 100755
--- a/tests/functional/export.sh
+++ b/tests/functional/export.sh
@@ -8,11 +8,12 @@ clearStore
 
 outPath=$(nix-build dependencies.nix --no-out-link)
 
-nix-store --export $outPath > $TEST_ROOT/exp
+nix-store --export "$outPath" > "$TEST_ROOT"/exp
 
-nix-store --export $(nix-store -qR $outPath) > $TEST_ROOT/exp_all
+# shellcheck disable=SC2046
+nix-store --export $(nix-store -qR "$outPath") > "$TEST_ROOT"/exp_all
 
-if nix-store --export $outPath >/dev/full ; then
+if nix-store --export "$outPath" >/dev/full ; then
     echo "exporting to a bad file descriptor should fail"
     exit 1
 fi
@@ -20,7 +21,7 @@ fi
 
 clearStore
 
-if nix-store --import < $TEST_ROOT/exp; then
+if nix-store --import < "$TEST_ROOT"/exp; then
     echo "importing a non-closure should fail"
     exit 1
 fi
@@ -28,13 +29,14 @@ fi
 
 clearStore
 
-nix-store --import < $TEST_ROOT/exp_all
+nix-store --import < "$TEST_ROOT"/exp_all
 
-nix-store --export $(nix-store -qR $outPath) > $TEST_ROOT/exp_all2
+# shellcheck disable=SC2046
+nix-store --export $(nix-store -qR "$outPath") > "$TEST_ROOT"/exp_all2
 
 
 clearStore
 
 # Regression test: the derivers in exp_all2 are empty, which shouldn't
 # cause a failure.
-nix-store --import < $TEST_ROOT/exp_all2
+nix-store --import < "$TEST_ROOT"/exp_all2