Swapping `git add <submodule>` to `git submodule add <submodule>` in a previous patch created a .gitmodules file with multiple submodules pointing to the same path in certain tests. Fix tests so that they are run on the original added submodule rather than a separate manually configured submodule. Signed-off-by: Calvin Wan <calvinwan@xxxxxxxxxx> --- t/t4027-diff-submodule.sh | 41 ++++++-------- t/t7508-status.sh | 114 +++++++++++++++----------------------- 2 files changed, 62 insertions(+), 93 deletions(-) diff --git a/t/t4027-diff-submodule.sh b/t/t4027-diff-submodule.sh index 2ee9f18b38..ce335534b9 100755 --- a/t/t4027-diff-submodule.sh +++ b/t/t4027-diff-submodule.sh @@ -127,36 +127,33 @@ test_expect_success 'git diff HEAD with dirty submodule (work tree, refs match) git config diff.ignoreSubmodules dirty && git diff HEAD >actual && test_must_be_empty actual && - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sub && + git config --add -f .gitmodules submodule.sub.ignore none && git commit -m "Update .gitmodules" .gitmodules && git diff HEAD >actual && sed -e "1,/^@@/d" actual >actual.body && expect_from_to >expect.body $subprev $subprev-dirty && test_cmp expect.body actual.body && - git config -f .gitmodules submodule.subname.ignore all && - git config -f .gitmodules submodule.subname.path sub && + git config -f .gitmodules submodule.sub.ignore all && git commit -m "Update .gitmodules" .gitmodules && git diff HEAD >actual2 && test_must_be_empty actual2 && - git config -f .gitmodules submodule.subname.ignore untracked && + git config -f .gitmodules submodule.sub.ignore untracked && git commit -m "Update .gitmodules" .gitmodules && git diff HEAD >actual3 && sed -e "1,/^@@/d" actual3 >actual3.body && expect_from_to >expect.body $subprev $subprev-dirty && test_cmp expect.body actual3.body && - git config -f .gitmodules submodule.subname.ignore dirty && + git config -f .gitmodules submodule.sub.ignore dirty && git commit -m "Update .gitmodules" .gitmodules && git diff HEAD >actual4 && test_must_be_empty actual4 && - git config submodule.subname.ignore none && - git config submodule.subname.path sub && + git config submodule.sub.ignore none && + git config submodule.sub.path sub && git diff HEAD >actual && sed -e "1,/^@@/d" actual >actual.body && expect_from_to >expect.body $subprev $subprev-dirty && test_cmp expect.body actual.body && - git config --remove-section submodule.subname && - git config --remove-section -f .gitmodules submodule.subname && + git config --unset submodule.sub.ignore && git config --unset diff.ignoreSubmodules && git reset --hard pristine-gitmodules ' @@ -194,27 +191,24 @@ test_expect_success 'git diff HEAD with dirty submodule (untracked, refs match)' ' test_expect_success 'git diff HEAD with dirty submodule (untracked, refs match) [.gitmodules]' ' - git config --add -f .gitmodules submodule.subname.ignore all && - git config --add -f .gitmodules submodule.subname.path sub && + git config --add -f .gitmodules submodule.sub.ignore all && git commit -m "Update .gitmodules" .gitmodules && git diff HEAD >actual2 && test_must_be_empty actual2 && - git config -f .gitmodules submodule.subname.ignore untracked && + git config -f .gitmodules submodule.sub.ignore untracked && git commit -m "Update .gitmodules" .gitmodules && git diff HEAD >actual3 && test_must_be_empty actual3 && - git config -f .gitmodules submodule.subname.ignore dirty && + git config -f .gitmodules submodule.sub.ignore dirty && git commit -m "Update .gitmodules" .gitmodules && git diff HEAD >actual4 && test_must_be_empty actual4 && - git config submodule.subname.ignore none && - git config submodule.subname.path sub && + git config submodule.sub.ignore none && git diff HEAD >actual && sed -e "1,/^@@/d" actual >actual.body && expect_from_to >expect.body $subprev $subprev-dirty && test_cmp expect.body actual.body && - git config --remove-section submodule.subname && - git config --remove-section -f .gitmodules submodule.subname && + git config --unset submodule.sub.ignore && git reset --hard pristine-gitmodules ' @@ -236,22 +230,19 @@ test_expect_success 'git diff between submodule commits [.gitmodules]' ' sed -e "1,/^@@/d" actual >actual.body && expect_from_to >expect.body $subtip $subprev && test_cmp expect.body actual.body && - git config --add -f .gitmodules submodule.subname.ignore dirty && - git config --add -f .gitmodules submodule.subname.path sub && + git config --add -f .gitmodules submodule.sub.ignore dirty && git diff HEAD^..HEAD >actual && sed -e "1,/^@@/d" actual >actual.body && expect_from_to >expect.body $subtip $subprev && test_cmp expect.body actual.body && - git config -f .gitmodules submodule.subname.ignore all && + git config -f .gitmodules submodule.sub.ignore all && git diff HEAD^..HEAD >actual && test_must_be_empty actual && - git config submodule.subname.ignore dirty && - git config submodule.subname.path sub && + git config submodule.sub.ignore dirty && git diff HEAD^..HEAD >actual && sed -e "1,/^@@/d" actual >actual.body && expect_from_to >expect.body $subtip $subprev && - git config --remove-section submodule.subname && - git config --remove-section -f .gitmodules submodule.subname && + git config --unset submodule.sub.ignore && git reset --hard pristine-gitmodules ' diff --git a/t/t7508-status.sh b/t/t7508-status.sh index 5808339997..3d934bfb86 100755 --- a/t/t7508-status.sh +++ b/t/t7508-status.sh @@ -1154,8 +1154,7 @@ test_expect_success '.gitmodules ignore=untracked suppresses submodules with unt test_config diff.ignoreSubmodules dirty && git status >output && test_cmp expect output && - git config --add -f .gitmodules submodule.subname.ignore untracked && - git config --add -f .gitmodules submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore untracked && cat > expect-modified-gitmodules << EOF && On branch main Your branch and '\''upstream'\'' have diverged, @@ -1187,18 +1186,16 @@ Untracked files: EOF git status >output && test_cmp expect-modified-gitmodules output && - git config -f .gitmodules --remove-section submodule.subname + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success '.git/config ignore=untracked suppresses submodules with untracked content' ' - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore untracked && - git config --add submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore none && + git config --add submodule.sm.ignore untracked && git status >output && test_cmp expect-modified-gitmodules output && - git config --remove-section submodule.subname && - git config --remove-section -f .gitmodules submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success '--ignore-submodules=dirty suppresses submodules with untracked content' ' @@ -1210,22 +1207,19 @@ test_expect_success '.gitmodules ignore=dirty suppresses submodules with untrack test_config diff.ignoreSubmodules dirty && git status >output && ! test -s actual && - git config --add -f .gitmodules submodule.subname.ignore dirty && - git config --add -f .gitmodules submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore dirty && git status >output && test_cmp expect-modified-gitmodules output && - git config -f .gitmodules --remove-section submodule.subname + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success '.git/config ignore=dirty suppresses submodules with untracked content' ' - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore dirty && - git config --add submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore none && + git config --add submodule.sm.ignore dirty && git status >output && test_cmp expect-modified-gitmodules output && - git config --remove-section submodule.subname && - git config -f .gitmodules --remove-section submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success '--ignore-submodules=dirty suppresses submodules with modified content' ' @@ -1235,22 +1229,19 @@ test_expect_success '--ignore-submodules=dirty suppresses submodules with modifi ' test_expect_success '.gitmodules ignore=dirty suppresses submodules with modified content' ' - git config --add -f .gitmodules submodule.subname.ignore dirty && - git config --add -f .gitmodules submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore dirty && git status >output && test_cmp expect-modified-gitmodules output && - git config -f .gitmodules --remove-section submodule.subname + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success '.git/config ignore=dirty suppresses submodules with modified content' ' - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore dirty && - git config --add submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore none && + git config --add submodule.sm.ignore dirty && git status >output && test_cmp expect-modified-gitmodules output && - git config --remove-section submodule.subname && - git config -f .gitmodules --remove-section submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success "--ignore-submodules=untracked doesn't suppress submodules with modified content" ' @@ -1289,8 +1280,7 @@ EOF ' test_expect_success ".gitmodules ignore=untracked doesn't suppress submodules with modified content" ' - git config --add -f .gitmodules submodule.subname.ignore untracked && - git config --add -f .gitmodules submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore untracked && cat > expect-modified-gitmodules << EOF && On branch main Your branch and '\''upstream'\'' have diverged, @@ -1324,18 +1314,16 @@ Untracked files: EOF git status >output && test_cmp expect-modified-gitmodules output && - git config -f .gitmodules --remove-section submodule.subname + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success ".git/config ignore=untracked doesn't suppress submodules with modified content" ' - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore untracked && - git config --add submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore none && + git config --add submodule.sm.ignore untracked && git status >output && test_cmp expect-modified-gitmodules output && - git config --remove-section submodule.subname && - git config -f .gitmodules --remove-section submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' head2=$(cd sm && git commit -q -m "2nd commit" foo && git rev-parse --short=7 --verify HEAD) @@ -1415,22 +1403,19 @@ Untracked files: untracked EOF - git config --add -f .gitmodules submodule.subname.ignore untracked && - git config --add -f .gitmodules submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore untracked && git status >output && test_cmp expect-modified-gitmodules output && - git config -f .gitmodules --remove-section submodule.subname + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success ".git/config ignore=untracked doesn't suppress submodule summary" ' - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore untracked && - git config --add submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore none && + git config --add submodule.sm.ignore untracked && git status >output && test_cmp expect-modified-gitmodules output && - git config --remove-section submodule.subname && - git config -f .gitmodules --remove-section submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success "--ignore-submodules=dirty doesn't suppress submodule summary" ' @@ -1438,22 +1423,19 @@ test_expect_success "--ignore-submodules=dirty doesn't suppress submodule summar test_cmp expect output ' test_expect_success ".gitmodules ignore=dirty doesn't suppress submodule summary" ' - git config --add -f .gitmodules submodule.subname.ignore dirty && - git config --add -f .gitmodules submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore dirty && git status >output && test_cmp expect-modified-gitmodules output && - git config -f .gitmodules --remove-section submodule.subname + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success ".git/config ignore=dirty doesn't suppress submodule summary" ' - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore dirty && - git config --add submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore none && + git config --add submodule.sm.ignore dirty && git status >output && test_cmp expect-modified-gitmodules output && - git config --remove-section submodule.subname && - git config -f .gitmodules --remove-section submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' cat > expect << EOF @@ -1552,22 +1534,19 @@ Untracked files: untracked EOF - git config --add -f .gitmodules submodule.subname.ignore all && - git config --add -f .gitmodules submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore all && git status > output && test_cmp expect output && - git config -f .gitmodules --remove-section submodule.subname + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success '.git/config ignore=all suppresses unstaged submodule summary' ' - git config --add -f .gitmodules submodule.subname.ignore none && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore all && - git config --add submodule.subname.path sm && + git config --add -f .gitmodules submodule.sm.ignore none && + git config --add submodule.sm.ignore all && git status > output && test_cmp expect output && - git config --remove-section submodule.subname && - git config -f .gitmodules --remove-section submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success 'setup of test environment' ' @@ -1633,9 +1612,8 @@ test_expect_success 'Restore default test environment' ' ' test_expect_success 'git commit will commit a staged but ignored submodule' ' - git config --add -f .gitmodules submodule.subname.ignore all && - git config --add -f .gitmodules submodule.subname.path sm && - git config --add submodule.subname.ignore all && + git config --add -f .gitmodules submodule.sm.ignore all && + git config --add submodule.sm.ignore all && git status -s --ignore-submodules=dirty >output && test_i18ngrep "^M. sm" output && GIT_EDITOR="echo hello >>\"\$1\"" && @@ -1676,8 +1654,8 @@ test_expect_success 'git commit -m will commit a staged but ignored submodule' ' git commit -uno -m message && git status -s --ignore-submodules=dirty >output && test_i18ngrep ! "^M. sm" output && - git config --remove-section submodule.subname && - git config -f .gitmodules --remove-section submodule.subname + git config --unset submodule.sm.ignore && + git config -f .gitmodules --unset submodule.sm.ignore ' test_expect_success 'show stash info with "--show-stash"' ' -- 2.39.2.722.g9855ee24e9-goog
