Skip to content

Commit 386e7a9

Browse files
avargitster
authored andcommitted
tests: add missing double quotes to included library paths
Fix inclusion errors which would occur if the $TEST_DIRECTORY had $IFS whitespace in it. See d42bab4 (core.fsyncmethod: tests for batch mode, 2022-04-04) and a242c15 (vimdiff: integrate layout tests in the unit tests framework ('t' folder), 2022-03-30) for the two relevant commits. Both were first released with v2.37.0-rc0 (and were also part of v2.37.0). Signed-off-by: Ævar Arnfjörð Bjarmason <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 8168d5e commit 386e7a9

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

t/t3700-add.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ test_description='Test of git add, including the -- option.'
88
TEST_PASSES_SANITIZE_LEAK=true
99
. ./test-lib.sh
1010

11-
. $TEST_DIRECTORY/lib-unique-files.sh
11+
. "$TEST_DIRECTORY"/lib-unique-files.sh
1212

1313
# Test the file mode "$1" of the file "$2" in the index.
1414
test_mode_in_index () {

t/t3903-stash.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main
99
export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
1010

1111
. ./test-lib.sh
12-
. $TEST_DIRECTORY/lib-unique-files.sh
12+
. "$TEST_DIRECTORY"/lib-unique-files.sh
1313

1414
test_expect_success 'usage on cmd and subcommand invalid option' '
1515
test_expect_code 129 git stash --invalid-option 2>usage &&

t/t7609-mergetool--lib.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ Testing basic merge tools options'
77
. ./test-lib.sh
88

99
test_expect_success 'mergetool --tool=vimdiff creates the expected layout' '
10-
. $GIT_BUILD_DIR/mergetools/vimdiff &&
10+
. "$GIT_BUILD_DIR"/mergetools/vimdiff &&
1111
run_unit_tests
1212
'
1313

0 commit comments

Comments
 (0)