Re: [PATCH] t3411: Fix test 1 for case-insensitive file systems

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Feb 3, 2009, at 12:18 PM, Johannes Schindelin wrote:

On Tue, 3 Feb 2009, Brian Gernhardt wrote:

On Feb 3, 2009, at 11:53 AM, Johannes Schindelin wrote:

Besides, I think that my latest comment still stands there: testing is not good enough, code inspection is required if something expects the file
names as they used to be.

As far as I can tell, no test relies on the auto-generated name of the test file. In fact, only t3411 uses that feature at all and it only performs operations on commits. All other uses of test_commit give a filename (even
though many of them don't appear to use the file).

You did not look far.

If there's something I missed, could you perhaps say what it is instead of being cryptic? I have a day job and am attempting to squeeze in searching for this between tasks. This fix (or similar) needs to make it into master so that the tests can run on case- insensitive file systems.

I used git grep to find uses of test_commit in next, and the only uses that did not provide a file name argument were in t3411. I read through every test in the file, and the only operations I saw were test_commit, test_merge, checkout -b, reset without filenames, rebase, and rev-parse. All operations on commits, not files.

The tests all run properly, and I've run t3411 individually (since it's apparently the only one using this feature) using -v to ensure that it was actually performing work and it seems to be.

So, based on the above, every usage of test_commit either provides a file name or does not care about the names of the files. Unless you have something that says it isn't, I'd like to see the fix Junio gave applied (repeated here as a reminder):

On Jan 29, 2009, at 12:19 PM, Junio C Hamano wrote:

diff --git c/t/test-lib.sh w/t/test-lib.sh
index c1839f7..8066c25 100644
--- c/t/test-lib.sh
+++ w/t/test-lib.sh
@@ -201,7 +201,7 @@ test_tick () {
# Both <file> and <contents> default to <message>.

test_commit () {
-	file=${2:-$(echo "$1" | tr 'A-Z' 'a-z')}
+	file=${2:-$(echo "$1" | tr 'A-Z' 'a-z').t}
	echo "${3-$1}" > "$file" &&
	git add "$file" &&
	test_tick &&

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux