{"thread":{"id":"44267","subject":"[PATCH] t3700-add.sh: Avoid filename collission between tests which could lead to test failure","startedAt":"2016-10-10T03:58:13Z","lastAt":"2016-10-10T17:09:13Z","messageCount":3,"participants":["Jeremy Huddleston Sequoia","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"303696","messageId":"20161010035756.38408-1-jeremyhu@apple.com","threadId":"44267","inReplyTo":null,"subject":"[PATCH] t3700-add.sh: Avoid filename collission between tests which could lead to test failure","fromName":"Jeremy Huddleston Sequoia","fromEmail":"jeremyhu@apple.com","sentAt":"2016-10-10T03:57:56Z","receivedAt":"2016-10-10T03:58:13Z","isPatch":true,"sender":{"key":"jeremyhu@apple.com","avatar":"https://avatars.githubusercontent.com/u/1258676?v=4"},"body":"Regressed-in: 610d55af0f082f6b866dc858e144c03d8ed4424c\nSigned-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>\nCC: Thomas Gummerer <t.gummerer@gmail.com>\nCC: Junio C Hamano <gitster@pobox.com>\n---\n t/t3700-add.sh | 10 +++++-----\n 1 file changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/t/t3700-add.sh b/t/t3700-add.sh\nindex 924a266..3ccb19b 100755\n--- a/t/t3700-add.sh\n+++ b/t/t3700-add.sh\n@@ -354,11 +354,11 @@ test_expect_success 'git add --chmod=[+-]x changes index with already added file\n \tgit add foo3 &&\n \tgit add --chmod=+x foo3 &&\n \ttest_mode_in_index 100755 foo3 &&\n-\techo foo >xfoo3 &&\n-\tchmod 755 xfoo3 &&\n-\tgit add xfoo3 &&\n-\tgit add --chmod=-x xfoo3 &&\n-\ttest_mode_in_index 100644 xfoo3\n+\techo foo >xfoo4 &&\n+\tchmod 755 xfoo4 &&\n+\tgit add xfoo4 &&\n+\tgit add --chmod=-x xfoo4 &&\n+\ttest_mode_in_index 100644 xfoo4\n '\n \n test_expect_success POSIXPERM 'git add --chmod=[+-]x does not change the working tree' '\n-- \n2.10.1\n\n"},{"id":"303732","messageId":"xmqqlgxwgnmu.fsf@gitster.mtv.corp.google.com","threadId":"44267","inReplyTo":"20161010035756.38408-1-jeremyhu@apple.com","subject":"Re: [PATCH] t3700-add.sh: Avoid filename collission between tests which could lead to test failure","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2016-10-10T16:44:09Z","receivedAt":"2016-10-10T16:50:56Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Jeremy Huddleston Sequoia <jeremyhu@apple.com> writes:\n\n> Regressed-in: 610d55af0f082f6b866dc858e144c03d8ed4424c\n> Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>\n> CC: Thomas Gummerer <t.gummerer@gmail.com>\n> CC: Junio C Hamano <gitster@pobox.com>\n\nThis is also under-explained.  Was the test broken for everybody and\nyou are the first to report, or was your environment somehow different\nthat the existing problem was revealed there first?\n\n"},{"id":"303736","messageId":"D7AF680C-1BED-431C-980E-5895B3848815@apple.com","threadId":"44267","inReplyTo":"xmqqlgxwgnmu.fsf@gitster.mtv.corp.google.com","subject":"Re: [PATCH] t3700-add.sh: Avoid filename collission between tests which could lead to test failure","fromName":"Jeremy Huddleston Sequoia","fromEmail":"jeremyhu@apple.com","sentAt":"2016-10-10T17:09:04Z","receivedAt":"2016-10-10T17:09:13Z","isPatch":true,"sender":{"key":"jeremyhu@apple.com","avatar":"https://avatars.githubusercontent.com/u/1258676?v=4"},"body":"\n> On Oct 10, 2016, at 09:44, Junio C Hamano <gitster@pobox.com> wrote:\n> \n> Jeremy Huddleston Sequoia <jeremyhu@apple.com> writes:\n> \n>> Regressed-in: 610d55af0f082f6b866dc858e144c03d8ed4424c\n>> Signed-off-by: Jeremy Huddleston Sequoia <jeremyhu@apple.com>\n>> CC: Thomas Gummerer <t.gummerer@gmail.com>\n>> CC: Junio C Hamano <gitster@pobox.com>\n> \n> This is also under-explained.  Was the test broken for everybody and\n> you are the first to report, or was your environment somehow different\n> that the existing problem was revealed there first?\n\nThere is nothing obviously different about my environment.  I'd expect this test to fail for everyone that runs the entire test because xfoo3 is not cleaned up after this earlier test:\n\ntest_expect_success \\\n        'git update-index --add: Test that executable bit is not used...' \\\n        'git config core.filemode 0 &&\n         test_ln_s_add xfoo2 xfoo3 &&   # runs git update-index --add\n         test_mode_in_index 120000 xfoo3'\n\nIf anyone ran the test by itself or otherwise skipped the 'git update-index --add: Test that executable bit is not used...' test, they would not encounter the issue.  However, the normal case is to run all of the tests, so I would expect everyone to be tripping over this issue.  If they're not, then that itself might be indicative of another bug because clearly that previous test just added xfoo3 as a symlink.\n\n--Jeremy"}]}