Skip to content

Commit 90e1452

Browse files
jlehmanngitster
authored andcommitted
Add tests for the diff.ignoreSubmodules config option
Signed-off-by: Jens Lehmann <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent be4f2b4 commit 90e1452

File tree

2 files changed

+19
-3
lines changed

2 files changed

+19
-3
lines changed

t/t4027-diff-submodule.sh

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,9 @@ test_expect_success 'git diff HEAD with dirty submodule (work tree, refs match)'
115115
'
116116

117117
test_expect_success 'git diff HEAD with dirty submodule (work tree, refs match) [.git/config]' '
118+
git config diff.ignoreSubmodules all &&
119+
git diff HEAD >actual &&
120+
! test -s actual &&
118121
git config submodule.subname.ignore none &&
119122
git config submodule.subname.path sub &&
120123
git diff HEAD >actual &&
@@ -136,10 +139,14 @@ test_expect_success 'git diff HEAD with dirty submodule (work tree, refs match)
136139
sed -e "1,/^@@/d" actual >actual.body &&
137140
expect_from_to >expect.body $subprev $subprev-dirty &&
138141
test_cmp expect.body actual.body &&
139-
git config --remove-section submodule.subname
142+
git config --remove-section submodule.subname &&
143+
git config --unset diff.ignoreSubmodules
140144
'
141145

142146
test_expect_success 'git diff HEAD with dirty submodule (work tree, refs match) [.gitmodules]' '
147+
git config diff.ignoreSubmodules dirty &&
148+
git diff HEAD >actual &&
149+
! test -s actual &&
143150
git config --add -f .gitmodules submodule.subname.ignore none &&
144151
git config --add -f .gitmodules submodule.subname.path sub &&
145152
git diff HEAD >actual &&
@@ -166,6 +173,7 @@ test_expect_success 'git diff HEAD with dirty submodule (work tree, refs match)
166173
test_cmp expect.body actual.body &&
167174
git config --remove-section submodule.subname &&
168175
git config --remove-section -f .gitmodules submodule.subname &&
176+
git config --unset diff.ignoreSubmodules &&
169177
rm .gitmodules
170178
'
171179

t/t7508-status.sh

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -849,11 +849,15 @@ test_expect_success '--ignore-submodules=untracked suppresses submodules with un
849849
'
850850

851851
test_expect_success '.gitmodules ignore=untracked suppresses submodules with untracked content' '
852+
git config diff.ignoreSubmodules dirty &&
853+
git status >output &&
854+
test_cmp expect output &&
852855
git config --add -f .gitmodules submodule.subname.ignore untracked &&
853856
git config --add -f .gitmodules submodule.subname.path sm &&
854857
git status > output &&
855858
test_cmp expect output &&
856-
git config -f .gitmodules --remove-section submodule.subname
859+
git config -f .gitmodules --remove-section submodule.subname &&
860+
git config --unset diff.ignoreSubmodules
857861
'
858862

859863
test_expect_success '.git/config ignore=untracked suppresses submodules with untracked content' '
@@ -873,11 +877,15 @@ test_expect_success '--ignore-submodules=dirty suppresses submodules with untrac
873877
'
874878

875879
test_expect_success '.gitmodules ignore=dirty suppresses submodules with untracked content' '
880+
git config diff.ignoreSubmodules dirty &&
881+
git status >output &&
882+
! test -s actual &&
876883
git config --add -f .gitmodules submodule.subname.ignore dirty &&
877884
git config --add -f .gitmodules submodule.subname.path sm &&
878885
git status > output &&
879886
test_cmp expect output &&
880-
git config -f .gitmodules --remove-section submodule.subname
887+
git config -f .gitmodules --remove-section submodule.subname &&
888+
git config --unset diff.ignoreSubmodules
881889
'
882890

883891
test_expect_success '.git/config ignore=dirty suppresses submodules with untracked content' '

0 commit comments

Comments
 (0)