Re: Tests in Cygwin
From: Johannes Sixt <hidden>
Date: 2016-06-15 22:46:43
Subsystem:
the rest · Maintainer:
Linus Torvalds
Possibly related (same subject, not in this thread)
- 2016-06-15 · Re: Tests in Cygwin · Don Slutz <hidden>
- 2016-06-15 · Re: Tests in Cygwin · Johannes Sixt <hidden>
- 2016-06-15 · Re: Tests in Cygwin · Alex Riesen <hidden>
- 2016-06-15 · Re: Tests in Cygwin · Don Slutz <hidden>
- 2016-06-15 · Re: Tests in Cygwin · Junio C Hamano <hidden>
Junio C Hamano schrieb:
Don Slutz [off-list ref] writes:quoted
A lot of these failures are do to running as root (member of Administrators)....Perhaps we can do this to reduce the noise level.
Thank you for this patch. Unfortunately, it does not work as intended because the call to test_have_prereq in test_skip does not split the prereq tokens so that tests with space-separated prerequisite tokens are always skipped. My first attempt to solve that is to remove the quotes from the test_have_prereq call in test_skip; but I think that this simple solution is rather unintuitive, and will look like a bug if you inspect the code again in two months. You could squash in the patch below, where I chose to concatenate tokens with '+' because I dislike the extra quoting that is otherwise necessary at the call site. The downside is that it plays games with IFS, which you might find too ugly and not worth it. It's your call. BTW, I didn't dare test this as root on my production system; I'll do that later today elsewhere.
diff --git a/t/README b/t/README
index a532c32..49c3a51 100644
--- a/t/README
+++ b/t/README@@ -200,7 +200,7 @@ library for your script to use. <script>. If it yields success, test is considered successful. <message> should state what it is testing. - An optional parameter <prereq> is a space separated list of prerequisite + An optional parameter <prereq> is a '+' separated list of prerequisite tokens. The test will be skipped if the test environment lacks any of the prerequiste feature listed (see below for commonly used prerequisites).
@@ -260,7 +260,8 @@ Prerequisites ------------- POSIXPERM The filesystem supports POSIX style permission bits - BSLASHPSPEC ??? + BSLASHPSPEC Backslashes in pathspec are not directory separators + (i.e. they are not converted to forward-slash) EXECKEEPSPID The process retains the same pid across exec(2) SANITY Test is not run by root user, and an attempt to an
diff --git a/t/t1004-read-tree-m-u-wf.sh b/t/t1004-read-tree-m-u-wf.sh
index eb60f80..8683189 100755
--- a/t/t1004-read-tree-m-u-wf.sh
+++ b/t/t1004-read-tree-m-u-wf.sh@@ -177,7 +177,7 @@ test_expect_success SYMLINKS 'funny symlink in work tree' ' ' -test_expect_success 'SYMLINKS SANITY' 'funny symlink in work tree, un-unlink-able' ' +test_expect_success SYMLINKS+SANITY 'funny symlink in work tree, un-unlink-able' ' rm -fr a b && git reset --hard &&
diff --git a/t/test-lib.sh b/t/test-lib.sh
index 1922e0b..2d12799 100644
--- a/t/test-lib.sh
+++ b/t/test-lib.sh@@ -263,7 +263,12 @@ test_set_prereq () { } satisfied=" " +# prerequisites can be concatenated with '+' test_have_prereq () { + save_IFS=$IFS + IFS=+ + set -- $* + IFS=$save_IFS for prerequisite do case $satisfied in