Re: [PATCH] Fix wrong failures in config test
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:50:21
Subsystem:
the rest · Maintainer:
Linus Torvalds
Possibly related (same subject, not in this thread)
- 2016-06-15 · [PATCH] Fix wrong failures in config test · Ingo Brückl <hidden>
- 2016-06-15 · Re: [PATCH] Fix wrong failures in config test · Jonathan Nieder <hidden>
- 2016-06-15 · Re: [PATCH] Fix wrong failures in config test · Ingo Brückl <hidden>
- 2016-06-15 · Re: [PATCH] Fix wrong failures in config test · Jonathan Nieder <hidden>
- 2016-06-15 · Re: [PATCH] Fix wrong failures in config test · Junio C Hamano <hidden>
Ingo Brückl [off-list ref] writes:
The tests after '--set in alternative GIT_CONFIG' failed because variable GIT_CONFIG was still set. Signed-off-by: Ingo Brückl <redacted> --- Is it only me (bash 3.2.48(1)-release) experiencing these failures?
t/t1300-repo-config.sh | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-)
t1300 first sources test-lib.sh that explicitly unsets GIT_CONFIG and the tests that might touch GIT_CONFIG all do so by a single-shot assignment to be exported, i.e. GIT_CONFIG=other-config git config anwohner.park ausweis that shouldn't affect the later test, unless the shell is broken. With this patch, can you check which one of the new tests barf on you? t/t1300-repo-config.sh | 21 +++++++++++++++++++++ 1 files changed, 21 insertions(+), 0 deletions(-)
diff --git a/t/t1300-repo-config.sh b/t/t1300-repo-config.sh
index d0e5546..c91d166 100755
--- a/t/t1300-repo-config.sh
+++ b/t/t1300-repo-config.sh@@ -7,6 +7,10 @@ test_description='Test git config in different settings' . ./test-lib.sh +test_expect_success 'is GIT_CONFIG set (0)?' ' + test "z${GIT_CONFIG+set}" = z +' + test -f .git/config && rm .git/config git config core.penguin "little blue"
@@ -399,8 +403,17 @@ cat > expect << EOF ein.bahn=strasse EOF + +test_expect_success 'is GIT_CONFIG set (1)?' ' + test "z${GIT_CONFIG+set}" = z +' + GIT_CONFIG=other-config git config -l > output +test_expect_success 'is GIT_CONFIG set (2)?' ' + test "z${GIT_CONFIG+set}" = z +' + test_expect_success 'alternative GIT_CONFIG' 'cmp output expect' test_expect_success 'alternative GIT_CONFIG (--file)' \
@@ -419,6 +432,10 @@ test_expect_success 'refer config from subdirectory' ' GIT_CONFIG=other-config git config anwohner.park ausweis +test_expect_success 'is GIT_CONFIG set (3)?' ' + test "z${GIT_CONFIG+set}" = z +' + cat > expect << EOF [ein] bahn = strasse
@@ -426,6 +443,10 @@ cat > expect << EOF park = ausweis EOF +test_expect_success 'is GIT_CONFIG set (4)?' ' + test "z${GIT_CONFIG+set}" = z +' + test_expect_success '--set in alternative GIT_CONFIG' 'cmp other-config expect' cat > .git/config << EOF