{"thread":{"id":"55139","subject":"[PATCH v3 00/10] Modernizing the t7001 test script","startedAt":"2021-02-11T19:48:38Z","lastAt":"2021-02-23T17:34:32Z","messageCount":14,"participants":["Shubham Verma","Junio C Hamano"],"isPatch":true,"patchVersion":3,"patchTotal":10},"messages":[{"id":"416760","messageId":"20210211194704.28913-1-shubhunic@gmail.com","threadId":"55139","inReplyTo":null,"subject":"[PATCH v3 00/10] Modernizing the t7001 test script","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:46:54Z","receivedAt":"2021-02-11T19:48:38Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"In this patch series modernize the t7001 test script by changing the\nstyle of its tests from an old one to the modern one and by cleaning\nup the test script.\n\nChanges since v2:\n\n- Correct the author name spelling\n\nThanks everyone for your feedback and suggestions.\n\nShubham Verma (10):\n  t7001: modernize test formatting\n  t7001: indent with TABs instead of spaces\n  t7001: remove unnecessary blank lines\n  t7001: modernize subshell formatting\n  t7001: remove whitespace after redirect operators\n  t7001: avoid using `cd` outside of subshells\n  t7001: use '>' rather than 'touch'\n  t7001: put each command on a separate line\n  t7001: use here-docs instead of echo\n  t7001: use `test` rather than `[`\n\n t/t7001-mv.sh | 420 ++++++++++++++++++++++++--------------------------\n 1 file changed, 199 insertions(+), 221 deletions(-)\n\n-- \n2.25.1\n\n"},{"id":"416761","messageId":"20210211194704.28913-2-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 01/10] t7001: modernize test formatting","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:46:55Z","receivedAt":"2021-02-11T19:48:38Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Some tests in this script are formatted using a very old style:\n\n        test_expect_success \\\n            'title' \\\n            'body line 1 &&\n            body line 2'\n\nUpdate the formatting to the modern style:\n\n        test_expect_success 'title' '\n            body line 1 &&\n            body line 2\n        '\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 192 +++++++++++++++++++++++++-------------------------\n 1 file changed, 96 insertions(+), 96 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 63d5f41a12..4bbb51e578 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -3,74 +3,74 @@\n test_description='git mv in subdirs'\n . ./test-lib.sh\n \n-test_expect_success \\\n-    'prepare reference tree' \\\n-    'mkdir path0 path1 &&\n+test_expect_success 'prepare reference tree' '\n+     mkdir path0 path1 &&\n      cp \"$TEST_DIRECTORY\"/../COPYING path0/COPYING &&\n      git add path0/COPYING &&\n-     git commit -m add -a'\n+     git commit -m add -a\n+'\n \n-test_expect_success \\\n-    'moving the file out of subdirectory' \\\n-    'cd path0 && git mv COPYING ../path1/COPYING'\n+test_expect_success 'moving the file out of subdirectory' '\n+     cd path0 && git mv COPYING ../path1/COPYING\n+'\n \n # in path0 currently\n-test_expect_success \\\n-    'commiting the change' \\\n-    'cd .. && git commit -m move-out -a'\n+test_expect_success 'commiting the change' '\n+     cd .. && git commit -m move-out -a\n+'\n \n-test_expect_success \\\n-    'checking the commit' \\\n-    'git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n-    grep \"^R100..*path0/COPYING..*path1/COPYING\" actual'\n+test_expect_success 'checking the commit' '\n+     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+     grep \"^R100..*path0/COPYING..*path1/COPYING\" actual\n+'\n \n-test_expect_success \\\n-    'moving the file back into subdirectory' \\\n-    'cd path0 && git mv ../path1/COPYING COPYING'\n+test_expect_success 'moving the file back into subdirectory' '\n+     cd path0 && git mv ../path1/COPYING COPYING\n+'\n \n # in path0 currently\n-test_expect_success \\\n-    'commiting the change' \\\n-    'cd .. && git commit -m move-in -a'\n-\n-test_expect_success \\\n-    'checking the commit' \\\n-    'git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n-    grep \"^R100..*path1/COPYING..*path0/COPYING\" actual'\n-\n-test_expect_success \\\n-    'mv --dry-run does not move file' \\\n-    'git mv -n path0/COPYING MOVED &&\n+test_expect_success 'commiting the change' '\n+     cd .. && git commit -m move-in -a\n+'\n+\n+test_expect_success 'checking the commit' '\n+     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+     grep \"^R100..*path1/COPYING..*path0/COPYING\" actual\n+'\n+\n+test_expect_success 'mv --dry-run does not move file' '\n+     git mv -n path0/COPYING MOVED &&\n      test -f path0/COPYING &&\n-     test ! -f MOVED'\n+     test ! -f MOVED\n+'\n \n-test_expect_success \\\n-    'checking -k on non-existing file' \\\n-    'git mv -k idontexist path0'\n+test_expect_success 'checking -k on non-existing file' '\n+     git mv -k idontexist path0\n+'\n \n-test_expect_success \\\n-    'checking -k on untracked file' \\\n-    'touch untracked1 &&\n+test_expect_success 'checking -k on untracked file' '\n+     touch untracked1 &&\n      git mv -k untracked1 path0 &&\n      test -f untracked1 &&\n-     test ! -f path0/untracked1'\n+     test ! -f path0/untracked1\n+'\n \n-test_expect_success \\\n-    'checking -k on multiple untracked files' \\\n-    'touch untracked2 &&\n+test_expect_success 'checking -k on multiple untracked files' '\n+     touch untracked2 &&\n      git mv -k untracked1 untracked2 path0 &&\n      test -f untracked1 &&\n      test -f untracked2 &&\n      test ! -f path0/untracked1 &&\n-     test ! -f path0/untracked2'\n+     test ! -f path0/untracked2\n+'\n \n-test_expect_success \\\n-    'checking -f on untracked file with existing target' \\\n-    'touch path0/untracked1 &&\n+test_expect_success 'checking -f on untracked file with existing target' '\n+     touch path0/untracked1 &&\n      test_must_fail git mv -f untracked1 path0 &&\n      test ! -f .git/index.lock &&\n      test -f untracked1 &&\n-     test -f path0/untracked1'\n+     test -f path0/untracked1\n+'\n \n # clean up the mess in case bad things happen\n rm -f idontexist untracked1 untracked2 \\\n@@ -78,79 +78,79 @@ rm -f idontexist untracked1 untracked2 \\\n      .git/index.lock\n rmdir path1\n \n-test_expect_success \\\n-    'moving to absent target with trailing slash' \\\n-    'test_must_fail git mv path0/COPYING no-such-dir/ &&\n+test_expect_success 'moving to absent target with trailing slash' '\n+     test_must_fail git mv path0/COPYING no-such-dir/ &&\n      test_must_fail git mv path0/COPYING no-such-dir// &&\n      git mv path0/ no-such-dir/ &&\n-     test_path_is_dir no-such-dir'\n+     test_path_is_dir no-such-dir\n+'\n \n-test_expect_success \\\n-    'clean up' \\\n-    'git reset --hard'\n+test_expect_success 'clean up' '\n+     git reset --hard\n+'\n \n-test_expect_success \\\n-    'moving to existing untracked target with trailing slash' \\\n-    'mkdir path1 &&\n+test_expect_success 'moving to existing untracked target with trailing slash' '\n+     mkdir path1 &&\n      git mv path0/ path1/ &&\n-     test_path_is_dir path1/path0/'\n+     test_path_is_dir path1/path0/\n+'\n \n-test_expect_success \\\n-    'moving to existing tracked target with trailing slash' \\\n-    'mkdir path2 &&\n+test_expect_success 'moving to existing tracked target with trailing slash' '\n+     mkdir path2 &&\n      >path2/file && git add path2/file &&\n      git mv path1/path0/ path2/ &&\n-     test_path_is_dir path2/path0/'\n+     test_path_is_dir path2/path0/\n+'\n \n-test_expect_success \\\n-    'clean up' \\\n-    'git reset --hard'\n+test_expect_success 'clean up' '\n+     git reset --hard\n+'\n \n-test_expect_success \\\n-    'adding another file' \\\n-    'cp \"$TEST_DIRECTORY\"/../README.md path0/README &&\n+test_expect_success 'adding another file' '\n+     cp \"$TEST_DIRECTORY\"/../README.md path0/README &&\n      git add path0/README &&\n-     git commit -m add2 -a'\n+     git commit -m add2 -a\n+'\n \n-test_expect_success \\\n-    'moving whole subdirectory' \\\n-    'git mv path0 path2'\n+test_expect_success 'moving whole subdirectory' '\n+     git mv path0 path2\n+'\n \n-test_expect_success \\\n-    'commiting the change' \\\n-    'git commit -m dir-move -a'\n+test_expect_success 'commiting the change' '\n+     git commit -m dir-move -a\n+'\n \n-test_expect_success \\\n-    'checking the commit' \\\n-    'git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+test_expect_success 'checking the commit' '\n+     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n      grep \"^R100..*path0/COPYING..*path2/COPYING\" actual &&\n-     grep \"^R100..*path0/README..*path2/README\" actual'\n+     grep \"^R100..*path0/README..*path2/README\" actual\n+'\n \n-test_expect_success \\\n-    'succeed when source is a prefix of destination' \\\n-    'git mv path2/COPYING path2/COPYING-renamed'\n+test_expect_success 'succeed when source is a prefix of destination' '\n+     git mv path2/COPYING path2/COPYING-renamed\n+'\n \n-test_expect_success \\\n-    'moving whole subdirectory into subdirectory' \\\n-    'git mv path2 path1'\n+test_expect_success 'moving whole subdirectory into subdirectory' '\n+     git mv path2 path1\n+'\n \n-test_expect_success \\\n-    'commiting the change' \\\n-    'git commit -m dir-move -a'\n+test_expect_success 'commiting the change' '\n+     git commit -m dir-move -a\n+'\n \n-test_expect_success \\\n-    'checking the commit' \\\n-    'git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+test_expect_success 'checking the commit' '\n+     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n      grep \"^R100..*path2/COPYING..*path1/path2/COPYING\" actual &&\n-     grep \"^R100..*path2/README..*path1/path2/README\" actual'\n+     grep \"^R100..*path2/README..*path1/path2/README\" actual\n+'\n \n-test_expect_success \\\n-    'do not move directory over existing directory' \\\n-    'mkdir path0 && mkdir path0/path2 && test_must_fail git mv path2 path0'\n+test_expect_success 'do not move directory over existing directory' '\n+     mkdir path0 && mkdir path0/path2 && test_must_fail git mv path2 path0\n+'\n \n-test_expect_success \\\n-    'move into \".\"' \\\n-    'git mv path1/path2/ .'\n+test_expect_success 'move into \".\"' '\n+     git mv path1/path2/ .\n+'\n \n test_expect_success \"Michael Cassar's test case\" '\n \trm -fr .git papers partA &&\n-- \n2.25.1\n\n"},{"id":"416762","messageId":"20210211194704.28913-4-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 03/10] t7001: remove unnecessary blank lines","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:46:57Z","receivedAt":"2021-02-11T19:49:02Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Some tests use a deprecated style in which there are unnecessary\nblank lines after the opening quote of the test body and before the\nclosing quote. So we should remove these unnecessary blank lines.\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 12 ------------\n 1 file changed, 12 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 0f74c6339c..e98b44f7a1 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -182,7 +182,6 @@ test_expect_success \"Sergey Vlasov's test case\" '\n '\n \n test_expect_success 'absolute pathname' '(\n-\n \trm -fr mine &&\n \tmkdir mine &&\n \tcd mine &&\n@@ -196,12 +195,9 @@ test_expect_success 'absolute pathname' '(\n \t! test -d sub &&\n \ttest -d in &&\n \tgit ls-files --error-unmatch in/file\n-\n-\n )'\n \n test_expect_success 'absolute pathname outside should fail' '(\n-\n \trm -fr mine &&\n \tmkdir mine &&\n \tcd mine &&\n@@ -216,7 +212,6 @@ test_expect_success 'absolute pathname outside should fail' '(\n \ttest -d sub &&\n \t! test -d ../in &&\n \tgit ls-files --error-unmatch sub/file\n-\n )'\n \n test_expect_success 'git mv to move multiple sources into a directory' '\n@@ -232,7 +227,6 @@ test_expect_success 'git mv to move multiple sources into a directory' '\n '\n \n test_expect_success 'git mv should not change sha1 of moved cache entry' '\n-\n \trm -fr .git &&\n \tgit init &&\n \techo 1 >dirty &&\n@@ -243,7 +237,6 @@ test_expect_success 'git mv should not change sha1 of moved cache entry' '\n \techo 2 >dirty2 &&\n \tgit mv dirty2 dirty &&\n \t[ \"$entry\" = \"$(git ls-files --stage dirty | cut -f 1)\" ]\n-\n '\n \n rm -f dirty dirty2\n@@ -266,7 +259,6 @@ test_expect_success 'git mv error on conflicted file' '\n '\n \n test_expect_success 'git mv should overwrite symlink to a file' '\n-\n \trm -fr .git &&\n \tgit init &&\n \techo 1 >moved &&\n@@ -279,13 +271,11 @@ test_expect_success 'git mv should overwrite symlink to a file' '\n \ttest \"$(cat symlink)\" = 1 &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n-\n '\n \n rm -f moved symlink\n \n test_expect_success 'git mv should overwrite file with a symlink' '\n-\n \trm -fr .git &&\n \tgit init &&\n \techo 1 >moved &&\n@@ -296,11 +286,9 @@ test_expect_success 'git mv should overwrite file with a symlink' '\n \t! test -e symlink &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n-\n '\n \n test_expect_success SYMLINKS 'check moved symlink' '\n-\n \ttest -h moved\n '\n \n-- \n2.25.1\n\n"},{"id":"416763","messageId":"20210211194704.28913-3-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 02/10] t7001: indent with TABs instead of spaces","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:46:56Z","receivedAt":"2021-02-11T19:49:02Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Signed-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 120 +++++++++++++++++++++++++-------------------------\n 1 file changed, 60 insertions(+), 60 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 4bbb51e578..0f74c6339c 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -4,72 +4,72 @@ test_description='git mv in subdirs'\n . ./test-lib.sh\n \n test_expect_success 'prepare reference tree' '\n-     mkdir path0 path1 &&\n-     cp \"$TEST_DIRECTORY\"/../COPYING path0/COPYING &&\n-     git add path0/COPYING &&\n-     git commit -m add -a\n+\tmkdir path0 path1 &&\n+\tcp \"$TEST_DIRECTORY\"/../COPYING path0/COPYING &&\n+\tgit add path0/COPYING &&\n+\tgit commit -m add -a\n '\n \n test_expect_success 'moving the file out of subdirectory' '\n-     cd path0 && git mv COPYING ../path1/COPYING\n+\tcd path0 && git mv COPYING ../path1/COPYING\n '\n \n # in path0 currently\n test_expect_success 'commiting the change' '\n-     cd .. && git commit -m move-out -a\n+\tcd .. && git commit -m move-out -a\n '\n \n test_expect_success 'checking the commit' '\n-     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n-     grep \"^R100..*path0/COPYING..*path1/COPYING\" actual\n+\tgit diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+\tgrep \"^R100..*path0/COPYING..*path1/COPYING\" actual\n '\n \n test_expect_success 'moving the file back into subdirectory' '\n-     cd path0 && git mv ../path1/COPYING COPYING\n+\tcd path0 && git mv ../path1/COPYING COPYING\n '\n \n # in path0 currently\n test_expect_success 'commiting the change' '\n-     cd .. && git commit -m move-in -a\n+\tcd .. && git commit -m move-in -a\n '\n \n test_expect_success 'checking the commit' '\n-     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n-     grep \"^R100..*path1/COPYING..*path0/COPYING\" actual\n+\tgit diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+\tgrep \"^R100..*path1/COPYING..*path0/COPYING\" actual\n '\n \n test_expect_success 'mv --dry-run does not move file' '\n-     git mv -n path0/COPYING MOVED &&\n-     test -f path0/COPYING &&\n-     test ! -f MOVED\n+\tgit mv -n path0/COPYING MOVED &&\n+\ttest -f path0/COPYING &&\n+\ttest ! -f MOVED\n '\n \n test_expect_success 'checking -k on non-existing file' '\n-     git mv -k idontexist path0\n+\tgit mv -k idontexist path0\n '\n \n test_expect_success 'checking -k on untracked file' '\n-     touch untracked1 &&\n-     git mv -k untracked1 path0 &&\n-     test -f untracked1 &&\n-     test ! -f path0/untracked1\n+\ttouch untracked1 &&\n+\tgit mv -k untracked1 path0 &&\n+\ttest -f untracked1 &&\n+\ttest ! -f path0/untracked1\n '\n \n test_expect_success 'checking -k on multiple untracked files' '\n-     touch untracked2 &&\n-     git mv -k untracked1 untracked2 path0 &&\n-     test -f untracked1 &&\n-     test -f untracked2 &&\n-     test ! -f path0/untracked1 &&\n-     test ! -f path0/untracked2\n+\ttouch untracked2 &&\n+\tgit mv -k untracked1 untracked2 path0 &&\n+\ttest -f untracked1 &&\n+\ttest -f untracked2 &&\n+\ttest ! -f path0/untracked1 &&\n+\ttest ! -f path0/untracked2\n '\n \n test_expect_success 'checking -f on untracked file with existing target' '\n-     touch path0/untracked1 &&\n-     test_must_fail git mv -f untracked1 path0 &&\n-     test ! -f .git/index.lock &&\n-     test -f untracked1 &&\n-     test -f path0/untracked1\n+\ttouch path0/untracked1 &&\n+\ttest_must_fail git mv -f untracked1 path0 &&\n+\ttest ! -f .git/index.lock &&\n+\ttest -f untracked1 &&\n+\ttest -f path0/untracked1\n '\n \n # clean up the mess in case bad things happen\n@@ -79,77 +79,77 @@ rm -f idontexist untracked1 untracked2 \\\n rmdir path1\n \n test_expect_success 'moving to absent target with trailing slash' '\n-     test_must_fail git mv path0/COPYING no-such-dir/ &&\n-     test_must_fail git mv path0/COPYING no-such-dir// &&\n-     git mv path0/ no-such-dir/ &&\n-     test_path_is_dir no-such-dir\n+\ttest_must_fail git mv path0/COPYING no-such-dir/ &&\n+\ttest_must_fail git mv path0/COPYING no-such-dir// &&\n+\tgit mv path0/ no-such-dir/ &&\n+\ttest_path_is_dir no-such-dir\n '\n \n test_expect_success 'clean up' '\n-     git reset --hard\n+\tgit reset --hard\n '\n \n test_expect_success 'moving to existing untracked target with trailing slash' '\n-     mkdir path1 &&\n-     git mv path0/ path1/ &&\n-     test_path_is_dir path1/path0/\n+\tmkdir path1 &&\n+\tgit mv path0/ path1/ &&\n+\ttest_path_is_dir path1/path0/\n '\n \n test_expect_success 'moving to existing tracked target with trailing slash' '\n-     mkdir path2 &&\n-     >path2/file && git add path2/file &&\n-     git mv path1/path0/ path2/ &&\n-     test_path_is_dir path2/path0/\n+\tmkdir path2 &&\n+\t>path2/file && git add path2/file &&\n+\tgit mv path1/path0/ path2/ &&\n+\ttest_path_is_dir path2/path0/\n '\n \n test_expect_success 'clean up' '\n-     git reset --hard\n+\tgit reset --hard\n '\n \n test_expect_success 'adding another file' '\n-     cp \"$TEST_DIRECTORY\"/../README.md path0/README &&\n-     git add path0/README &&\n-     git commit -m add2 -a\n+\tcp \"$TEST_DIRECTORY\"/../README.md path0/README &&\n+\tgit add path0/README &&\n+\tgit commit -m add2 -a\n '\n \n test_expect_success 'moving whole subdirectory' '\n-     git mv path0 path2\n+\tgit mv path0 path2\n '\n \n test_expect_success 'commiting the change' '\n-     git commit -m dir-move -a\n+\tgit commit -m dir-move -a\n '\n \n test_expect_success 'checking the commit' '\n-     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n-     grep \"^R100..*path0/COPYING..*path2/COPYING\" actual &&\n-     grep \"^R100..*path0/README..*path2/README\" actual\n+\tgit diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+\tgrep \"^R100..*path0/COPYING..*path2/COPYING\" actual &&\n+\tgrep \"^R100..*path0/README..*path2/README\" actual\n '\n \n test_expect_success 'succeed when source is a prefix of destination' '\n-     git mv path2/COPYING path2/COPYING-renamed\n+\tgit mv path2/COPYING path2/COPYING-renamed\n '\n \n test_expect_success 'moving whole subdirectory into subdirectory' '\n-     git mv path2 path1\n+\tgit mv path2 path1\n '\n \n test_expect_success 'commiting the change' '\n-     git commit -m dir-move -a\n+\tgit commit -m dir-move -a\n '\n \n test_expect_success 'checking the commit' '\n-     git diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n-     grep \"^R100..*path2/COPYING..*path1/path2/COPYING\" actual &&\n-     grep \"^R100..*path2/README..*path1/path2/README\" actual\n+\tgit diff-tree -r -M --name-status  HEAD^ HEAD >actual &&\n+\tgrep \"^R100..*path2/COPYING..*path1/path2/COPYING\" actual &&\n+ \tgrep \"^R100..*path2/README..*path1/path2/README\" actual\n '\n \n test_expect_success 'do not move directory over existing directory' '\n-     mkdir path0 && mkdir path0/path2 && test_must_fail git mv path2 path0\n+\tmkdir path0 && mkdir path0/path2 && test_must_fail git mv path2 path0\n '\n \n test_expect_success 'move into \".\"' '\n-     git mv path1/path2/ .\n+\tgit mv path1/path2/ .\n '\n \n test_expect_success \"Michael Cassar's test case\" '\n-- \n2.25.1\n\n"},{"id":"416764","messageId":"20210211194704.28913-6-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 05/10] t7001: remove whitespace after redirect operators","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:46:59Z","receivedAt":"2021-02-11T19:49:27Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"According to Documentation/CodingGuidelines, there should be no\nwhitespace after redirect operators. So, we should remove these\nwhitespaces after redirect operators.\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 6 +++---\n 1 file changed, 3 insertions(+), 3 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 7fd4341988..2e49e9af81 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -156,9 +156,9 @@ test_expect_success \"Michael Cassar's test case\" '\n \trm -fr .git papers partA &&\n \tgit init &&\n \tmkdir -p papers/unsorted papers/all-papers partA &&\n-\techo a > papers/unsorted/Thesis.pdf &&\n-\techo b > partA/outline.txt &&\n-\techo c > papers/unsorted/_another &&\n+\techo a >papers/unsorted/Thesis.pdf &&\n+\techo b >partA/outline.txt &&\n+\techo c >papers/unsorted/_another &&\n \tgit add papers partA &&\n \tT1=$(git write-tree) &&\n \n-- \n2.25.1\n\n"},{"id":"416765","messageId":"20210211194704.28913-8-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 07/10] t7001: use '>' rather than 'touch'","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:47:01Z","receivedAt":"2021-02-11T19:49:28Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Use `>` rather than `touch` to create an empty file when the\ntimestamp isn't relevant to the test.\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 10 +++++-----\n 1 file changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 6412c4f882..a56580b26d 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -49,14 +49,14 @@ test_expect_success 'checking -k on non-existing file' '\n '\n \n test_expect_success 'checking -k on untracked file' '\n-\ttouch untracked1 &&\n+\t>untracked1 &&\n \tgit mv -k untracked1 path0 &&\n \ttest -f untracked1 &&\n \ttest ! -f path0/untracked1\n '\n \n test_expect_success 'checking -k on multiple untracked files' '\n-\ttouch untracked2 &&\n+\t>untracked2 &&\n \tgit mv -k untracked1 untracked2 path0 &&\n \ttest -f untracked1 &&\n \ttest -f untracked2 &&\n@@ -65,7 +65,7 @@ test_expect_success 'checking -k on multiple untracked files' '\n '\n \n test_expect_success 'checking -f on untracked file with existing target' '\n-\ttouch path0/untracked1 &&\n+\t>path0/untracked1 &&\n \ttest_must_fail git mv -f untracked1 path0 &&\n \ttest ! -f .git/index.lock &&\n \ttest -f untracked1 &&\n@@ -488,7 +488,7 @@ test_expect_success 'moving nested submodules' '\n \tmkdir sub_nested_nested &&\n \t(\n \t\tcd sub_nested_nested &&\n-\t\ttouch nested_level2 &&\n+\t\t>nested_level2 &&\n \t\tgit init &&\n \t\tgit add . &&\n \t\tgit commit -m \"nested level 2\"\n@@ -496,7 +496,7 @@ test_expect_success 'moving nested submodules' '\n \tmkdir sub_nested &&\n \t(\n \t\tcd sub_nested &&\n-\t\ttouch nested_level1 &&\n+\t\t>nested_level1 &&\n \t\tgit init &&\n \t\tgit add . &&\n \t\tgit commit -m \"nested level 1\" &&\n-- \n2.25.1\n\n"},{"id":"416766","messageId":"20210211194704.28913-5-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 04/10] t7001: modernize subshell formatting","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:46:58Z","receivedAt":"2021-02-11T19:49:28Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Some test use an old style for formatting subshells:\n\n        (command &&\n            ...\n\nUpdate them to the modern style:\n\n        (\n            command &&\n            ...\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 74 ++++++++++++++++++++++++++++-----------------------\n 1 file changed, 40 insertions(+), 34 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex e98b44f7a1..7fd4341988 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -181,38 +181,42 @@ test_expect_success \"Sergey Vlasov's test case\" '\n \tgit mv ab a\n '\n \n-test_expect_success 'absolute pathname' '(\n-\trm -fr mine &&\n-\tmkdir mine &&\n-\tcd mine &&\n-\ttest_create_repo one &&\n-\tcd one &&\n-\tmkdir sub &&\n-\t>sub/file &&\n-\tgit add sub/file &&\n-\n-\tgit mv sub \"$(pwd)/in\" &&\n-\t! test -d sub &&\n-\ttest -d in &&\n-\tgit ls-files --error-unmatch in/file\n-)'\n-\n-test_expect_success 'absolute pathname outside should fail' '(\n-\trm -fr mine &&\n-\tmkdir mine &&\n-\tcd mine &&\n-\tout=$(pwd) &&\n-\ttest_create_repo one &&\n-\tcd one &&\n-\tmkdir sub &&\n-\t>sub/file &&\n-\tgit add sub/file &&\n-\n-\ttest_must_fail git mv sub \"$out/out\" &&\n-\ttest -d sub &&\n-\t! test -d ../in &&\n-\tgit ls-files --error-unmatch sub/file\n-)'\n+test_expect_success 'absolute pathname' '\n+\t(\n+\t\trm -fr mine &&\n+\t\tmkdir mine &&\n+\t\tcd mine &&\n+\t\ttest_create_repo one &&\n+\t\tcd one &&\n+\t\tmkdir sub &&\n+\t\t>sub/file &&\n+\t\tgit add sub/file &&\n+\n+\t\tgit mv sub \"$(pwd)/in\" &&\n+\t\t! test -d sub &&\n+\t\ttest -d in &&\n+\t\tgit ls-files --error-unmatch in/file\n+\t)\n+'\n+\n+test_expect_success 'absolute pathname outside should fail' '\n+\t(\n+\t\trm -fr mine &&\n+\t\tmkdir mine &&\n+\t\tcd mine &&\n+\t\tout=$(pwd) &&\n+\t\ttest_create_repo one &&\n+\t\tcd one &&\n+\t\tmkdir sub &&\n+\t\t>sub/file &&\n+\t\tgit add sub/file &&\n+\n+\t\ttest_must_fail git mv sub \"$out/out\" &&\n+\t\ttest -d sub &&\n+\t\t! test -d ../in &&\n+\t\tgit ls-files --error-unmatch sub/file\n+\t)\n+'\n \n test_expect_success 'git mv to move multiple sources into a directory' '\n \trm -fr .git && git init &&\n@@ -503,14 +507,16 @@ test_expect_success 'moving a submodule in nested directories' '\n test_expect_success 'moving nested submodules' '\n \tgit commit -am \"cleanup commit\" &&\n \tmkdir sub_nested_nested &&\n-\t(cd sub_nested_nested &&\n+\t(\n+\t\tcd sub_nested_nested &&\n \t\ttouch nested_level2 &&\n \t\tgit init &&\n \t\tgit add . &&\n \t\tgit commit -m \"nested level 2\"\n \t) &&\n \tmkdir sub_nested &&\n-\t(cd sub_nested &&\n+\t(\n+\t\tcd sub_nested &&\n \t\ttouch nested_level1 &&\n \t\tgit init &&\n \t\tgit add . &&\n-- \n2.25.1\n\n"},{"id":"416767","messageId":"20210211194704.28913-7-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 06/10] t7001: avoid using `cd` outside of subshells","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:47:00Z","receivedAt":"2021-02-11T19:49:55Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Avoid using `cd` outside of subshells since, if the test fails,\nthere is no guarantee that the current working directory is the\nexpected one, which may cause subsequent tests to run in the wrong\ndirectory.\n\nWhile at it, make some other tests more concise by replacing\nsimple subshells with `git -C`.\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 43 +++++++++++--------------------------------\n 1 file changed, 11 insertions(+), 32 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 2e49e9af81..6412c4f882 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -11,12 +11,12 @@ test_expect_success 'prepare reference tree' '\n '\n \n test_expect_success 'moving the file out of subdirectory' '\n-\tcd path0 && git mv COPYING ../path1/COPYING\n+\tgit -C path0 mv COPYING ../path1/COPYING\n '\n \n # in path0 currently\n test_expect_success 'commiting the change' '\n-\tcd .. && git commit -m move-out -a\n+\tgit commit -m move-out -a\n '\n \n test_expect_success 'checking the commit' '\n@@ -25,12 +25,12 @@ test_expect_success 'checking the commit' '\n '\n \n test_expect_success 'moving the file back into subdirectory' '\n-\tcd path0 && git mv ../path1/COPYING COPYING\n+\tgit -C path0 mv ../path1/COPYING COPYING\n '\n \n # in path0 currently\n test_expect_success 'commiting the change' '\n-\tcd .. && git commit -m move-in -a\n+\tgit commit -m move-in -a\n '\n \n test_expect_success 'checking the commit' '\n@@ -328,10 +328,7 @@ test_expect_success 'git mv moves a submodule with a .git directory and no .gitm\n \tgit mv sub mod/sub &&\n \t! test -e sub &&\n \t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n-\t(\n-\t\tcd mod/sub &&\n-\t\tgit status\n-\t) &&\n+\tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n '\n@@ -351,10 +348,7 @@ test_expect_success 'git mv moves a submodule with a .git directory and .gitmodu\n \tgit mv sub mod/sub &&\n \t! test -e sub &&\n \t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n-\t(\n-\t\tcd mod/sub &&\n-\t\tgit status\n-\t) &&\n+\tgit -C mod/sub status &&\n \techo mod/sub >expected &&\n \tgit config -f .gitmodules submodule.sub.path >actual &&\n \ttest_cmp expected actual &&\n@@ -368,16 +362,10 @@ test_expect_success 'git mv moves a submodule with gitfile' '\n \tgit submodule update &&\n \tentry=\"$(git ls-files --stage sub | cut -f 1)\" &&\n \tmkdir mod &&\n-\t(\n-\t\tcd mod &&\n-\t\tgit mv ../sub/ .\n-\t) &&\n+\tgit -C mod mv ../sub/ . &&\n \t! test -e sub &&\n \t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n-\t(\n-\t\tcd mod/sub &&\n-\t\tgit status\n-\t) &&\n+\tgit -C mod/sub status &&\n \techo mod/sub >expected &&\n \tgit config -f .gitmodules submodule.sub.path >actual &&\n \ttest_cmp expected actual &&\n@@ -396,10 +384,7 @@ test_expect_success 'mv does not complain when no .gitmodules file is found' '\n \ttest_must_be_empty actual.err &&\n \t! test -e sub &&\n \t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n-\t(\n-\t\tcd mod/sub &&\n-\t\tgit status\n-\t) &&\n+\tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n '\n@@ -420,10 +405,7 @@ test_expect_success 'mv will error out on a modified .gitmodules file unless sta\n \ttest_must_be_empty actual.err &&\n \t! test -e sub &&\n \t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n-\t(\n-\t\tcd mod/sub &&\n-\t\tgit status\n-\t) &&\n+\tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n '\n@@ -441,10 +423,7 @@ test_expect_success 'mv issues a warning when section is not found in .gitmodule\n \ttest_i18ncmp expect.err actual.err &&\n \t! test -e sub &&\n \t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n-\t(\n-\t\tcd mod/sub &&\n-\t\tgit status\n-\t) &&\n+\tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n '\n-- \n2.25.1\n\n"},{"id":"416768","messageId":"20210211194704.28913-11-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 10/10] t7001: use `test` rather than `[`","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:47:04Z","receivedAt":"2021-02-11T19:49:55Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"According to Documentation/CodingGuidelines, we should use \"test\"\nrather than \"[ ... ]\" in shell scripts, so let's replace the\n\"[ ... ]\" with \"test\" in the t7001 test script.\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 16 ++++++++--------\n 1 file changed, 8 insertions(+), 8 deletions(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 36a274e96d..65807e4873 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -242,10 +242,10 @@ test_expect_success 'git mv should not change sha1 of moved cache entry' '\n \tgit add dirty &&\n \tentry=\"$(git ls-files --stage dirty | cut -f 1)\" &&\n \tgit mv dirty dirty2 &&\n-\t[ \"$entry\" = \"$(git ls-files --stage dirty2 | cut -f 1)\" ] &&\n+\ttest \"$entry\" = \"$(git ls-files --stage dirty2 | cut -f 1)\" &&\n \techo 2 >dirty2 &&\n \tgit mv dirty2 dirty &&\n-\t[ \"$entry\" = \"$(git ls-files --stage dirty | cut -f 1)\" ]\n+\ttest \"$entry\" = \"$(git ls-files --stage dirty | cut -f 1)\"\n '\n \n rm -f dirty dirty2\n@@ -332,7 +332,7 @@ test_expect_success 'git mv moves a submodule with a .git directory and no .gitm\n \tmkdir mod &&\n \tgit mv sub mod/sub &&\n \t! test -e sub &&\n-\t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n+\ttest \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" &&\n \tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n@@ -352,7 +352,7 @@ test_expect_success 'git mv moves a submodule with a .git directory and .gitmodu\n \tmkdir mod &&\n \tgit mv sub mod/sub &&\n \t! test -e sub &&\n-\t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n+\ttest \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" &&\n \tgit -C mod/sub status &&\n \techo mod/sub >expected &&\n \tgit config -f .gitmodules submodule.sub.path >actual &&\n@@ -369,7 +369,7 @@ test_expect_success 'git mv moves a submodule with gitfile' '\n \tmkdir mod &&\n \tgit -C mod mv ../sub/ . &&\n \t! test -e sub &&\n-\t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n+\ttest \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" &&\n \tgit -C mod/sub status &&\n \techo mod/sub >expected &&\n \tgit config -f .gitmodules submodule.sub.path >actual &&\n@@ -388,7 +388,7 @@ test_expect_success 'mv does not complain when no .gitmodules file is found' '\n \tgit mv sub mod/sub 2>actual.err &&\n \ttest_must_be_empty actual.err &&\n \t! test -e sub &&\n-\t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n+\ttest \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" &&\n \tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n@@ -409,7 +409,7 @@ test_expect_success 'mv will error out on a modified .gitmodules file unless sta\n \tgit mv sub mod/sub 2>actual.err &&\n \ttest_must_be_empty actual.err &&\n \t! test -e sub &&\n-\t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n+\ttest \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" &&\n \tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n@@ -427,7 +427,7 @@ test_expect_success 'mv issues a warning when section is not found in .gitmodule\n \tgit mv sub mod/sub 2>actual.err &&\n \ttest_i18ncmp expect.err actual.err &&\n \t! test -e sub &&\n-\t[ \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" ] &&\n+\ttest \"$entry\" = \"$(git ls-files --stage mod/sub | cut -f 1)\" &&\n \tgit -C mod/sub status &&\n \tgit update-index --refresh &&\n \tgit diff-files --quiet\n-- \n2.25.1\n\n"},{"id":"416769","messageId":"20210211194704.28913-9-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 08/10] t7001: put each command on a separate line","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:47:02Z","receivedAt":"2021-02-11T19:50:07Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Modern practice is to avoid multiple commands per line, and\ninstead place each command on its own line.\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 4 +++-\n 1 file changed, 3 insertions(+), 1 deletion(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex a56580b26d..93d6767951 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -145,7 +145,9 @@ test_expect_success 'checking the commit' '\n '\n \n test_expect_success 'do not move directory over existing directory' '\n-\tmkdir path0 && mkdir path0/path2 && test_must_fail git mv path2 path0\n+\tmkdir path0 &&\n+\tmkdir path0/path2 &&\n+\ttest_must_fail git mv path2 path0\n '\n \n test_expect_success 'move into \".\"' '\n-- \n2.25.1\n\n"},{"id":"416770","messageId":"20210211194704.28913-10-shubhunic@gmail.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"[PATCH v3 09/10] t7001: use here-docs instead of echo","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-11T19:47:03Z","receivedAt":"2021-02-11T19:50:27Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Change from old style to current style by taking advantage of\nhere-docs instead of echo commands.\n\nSigned-off-by: Shubham Verma <shubhunic@gmail.com>\n---\n t/t7001-mv.sh | 5 ++++-\n 1 file changed, 4 insertions(+), 1 deletion(-)\n\ndiff --git a/t/t7001-mv.sh b/t/t7001-mv.sh\nindex 93d6767951..36a274e96d 100755\n--- a/t/t7001-mv.sh\n+++ b/t/t7001-mv.sh\n@@ -228,7 +228,10 @@ test_expect_success 'git mv to move multiple sources into a directory' '\n \tgit add dir/?.txt &&\n \tgit mv dir/a.txt dir/b.txt other &&\n \tgit ls-files >actual &&\n-\t{ echo other/a.txt; echo other/b.txt; } >expect &&\n+\tcat >expect <<-\\EOF &&\n+\tother/a.txt\n+\tother/b.txt\n+\tEOF\n \ttest_cmp expect actual\n '\n \n-- \n2.25.1\n\n"},{"id":"416773","messageId":"xmqqk0rewa82.fsf@gitster.c.googlers.com","threadId":"55139","inReplyTo":"20210211194704.28913-1-shubhunic@gmail.com","subject":"Re: [PATCH v3 00/10] Modernizing the t7001 test script","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-02-11T21:44:29Z","receivedAt":"2021-02-11T21:45:15Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Shubham Verma <shubhunic@gmail.com> writes:\n\n> In this patch series modernize the t7001 test script by changing the\n> style of its tests from an old one to the modern one and by cleaning\n> up the test script.\n>\n> Changes since v2:\n>\n> - Correct the author name spelling\n>\n> Thanks everyone for your feedback and suggestions.\n\nI've compared between v2 and v3 and verified the only change is the\nname spelling, so I'll port over Eric's reviewed-by's from the\nprevious round.\n\nThanks.\n\n\n\n"},{"id":"417551","messageId":"CAD=kLptZS28NH53qEQ48eD7yT-GZ4Xx4Uc4oGAJD-g_UwQFRUg@mail.gmail.com","threadId":"55139","inReplyTo":"xmqqk0rewa82.fsf@gitster.c.googlers.com","subject":"Re: [PATCH v3 00/10] Modernizing the t7001 test script","fromName":"Shubham Verma","fromEmail":"shubhunic@gmail.com","sentAt":"2021-02-23T08:26:15Z","receivedAt":"2021-02-23T08:27:29Z","isPatch":true,"sender":{"key":"shubhunic@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45103446?v=4"},"body":"Hello Junio,\n\nI already corrected the author-name spelling in this v3 series\nbut I looking at the last \"What's Cooking\" message and saw that\nModernizing the t7001 test script series described as\n\"Expecting the final version with corrected author attribution\".\nPlease once look at my patch series.\n\nThank You!\n\n\nOn Fri, Feb 12, 2021 at 3:14 AM Junio C Hamano <gitster@pobox.com> wrote:\n>\n> Shubham Verma <shubhunic@gmail.com> writes:\n>\n> > In this patch series modernize the t7001 test script by changing the\n> > style of its tests from an old one to the modern one and by cleaning\n> > up the test script.\n> >\n> > Changes since v2:\n> >\n> > - Correct the author name spelling\n> >\n> > Thanks everyone for your feedback and suggestions.\n>\n> I've compared between v2 and v3 and verified the only change is the\n> name spelling, so I'll port over Eric's reviewed-by's from the\n> previous round.\n>\n> Thanks.\n>\n>\n>\n"},{"id":"417569","messageId":"xmqqft1m4rkf.fsf@gitster.g","threadId":"55139","inReplyTo":"CAD=kLptZS28NH53qEQ48eD7yT-GZ4Xx4Uc4oGAJD-g_UwQFRUg@mail.gmail.com","subject":"Re: [PATCH v3 00/10] Modernizing the t7001 test script","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-02-23T17:33:36Z","receivedAt":"2021-02-23T17:34:32Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Shubham Verma <shubhunic@gmail.com> writes:\n\n> Hello Junio,\n>\n> I already corrected the author-name spelling in this v3 series\n> but I looking at the last \"What's Cooking\" message and saw that\n> Modernizing the t7001 test script series described as\n> \"Expecting the final version with corrected author attribution\".\n\nI looked at it on Feb 12 and requeued already, but the entry in the\nwhat's cooking report was left stale by mistake.\n\nThanks for pinging.\n"}]}