[PATCH] Skip t1300.70 and 71 on msysGit.

Pat Thoyts· Feb 15, 2010, 23:14 UTC · lore

These two tests fail on msysGit because /dev/null is an alias for nul on Windows and when reading the value back from git config the alias does not match the real filename. Also the HOME environment variable has a unix-style path but git returns a native equivalent path for '~'. As these are platform-dependent equivalent results it seems simplest to skip the test entirely.

Signed-off-by: Pat Thoyts <patthoyts@users.sourceforge.net>
---
 t/t1300-repo-config.sh |    6 +++---
 t/test-lib.sh          |    1 +
 2 files changed, 4 insertions(+), 3 deletions(-)

Show changes to 2 files +4 −3

t/t1300-repo-config.sh, t/test-lib.sh

diff --git a/t/t1300-repo-config.sh b/t/t1300-repo-config.sh
index f11f98c..90541a3 100755
--- a/t/t1300-repo-config.sh
+++ b/t/t1300-repo-config.sh
@@ -701,7 +701,7 @@ cat >expect <<\EOF
 	trailingtilde = foo~
 EOF
 
-test_expect_success 'set --path' '
+test_expect_success NONMINGW 'set --path' '
 	git config --path path.home "~/" &&
 	git config --path path.normal "/dev/null" &&
 	git config --path path.trailingtilde "foo~" &&
@@ -713,14 +713,14 @@ $HOME/
 foo~
 EOF
 
-test_expect_success 'get --path' '
+test_expect_success NONMINGW 'get --path' '
 	git config --get --path path.home > result &&
 	git config --get --path path.normal >> result &&
 	git config --get --path path.trailingtilde >> result &&
 	test_cmp expect result
 '
 
-rm .git/config
+test_have_prereq NONMINGW && rm .git/config
 
 git config quote.leading " test"
 git config quote.ending "test "
diff --git a/t/test-lib.sh b/t/test-lib.sh
index afd3053..f6a1590 100644
--- a/t/test-lib.sh
+++ b/t/test-lib.sh
@@ -776,6 +776,7 @@ case $(uname -s) in
 	test_set_prereq POSIXPERM
 	test_set_prereq BSLASHPSPEC
 	test_set_prereq EXECKEEPSPID
+	test_set_prereq NONMINGW
 	;;
 esac

-- 
1.7.0.rc2.1438.gebbe6.dirty

Johannes Schindelin· Feb 16, 2010, 09:27 UTC · re: Pat Thoyts · lore

Re: [msysGit] [PATCH] Skip t1300.70 and 71 on msysGit.

Hi Pat,

On Mon, 15 Feb 2010, Pat Thoyts wrote:

Show 9 quoted lines

> These two tests fail on msysGit because /dev/null is an alias for nul on 
> Windows and when reading the value back from git config the alias does 
> not match the real filename. Also the HOME environment variable has a 
> unix-style path but git returns a native equivalent path for '~'.  As 
> these are platform-dependent equivalent results it seems simplest to 
> skip the test entirely.
> 
> Signed-off-by: Pat Thoyts <patthoyts@users.sourceforge.net>
> ---

Can you push this to 4msysgit's 'devel' branch?

Thank you, Dscho

patchSkip t1300.70 and 71 on msysGit.