| Message ID | 1579862044-29847-1-git-send-email-peter.kaestle@nokia.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [v3,1/2] Testcase for submodule status on empty dirs | expand |
Peter Kaestle <peter.kaestle@nokia.com> writes: > Subject: Re: [PATCH v3 1/2] Testcase for submodule status on empty dirs > > Test that submodule status reports initialized but not cloned > submodules as missing. Let's not waste the findings you made and the description you wrote during the review of the previous round. After all, the reason why reviewers ask questions and you write responses is not because we want to satisify curiosity of the reviewers, but to polish the history we leave to the future readers, both in the patch text and its explanations. How about explaining this step more like this? Subject: t7400: add a testcase for submodule status on empty dirs We have test coverage for "git submodule status" output in various cases, i.e. 1) not-init, not-cloned: status should initially be "missing" 2) init, not-cloned: status should be "missing" 3) not-init, cloned: 4) init, cloned:'status should be "up-to-date" after update 4.1) + modified: status should be "modified" after submodule commit 4.2) + modified, committed: status should be "up-to-date" after update but the cases 2) and 3) are not covered. Test that submodule status reports initialized but not cloned submodules as missing to fill the gap in test coverage. Thanks. > > Signed-off-by: Peter Kaestle <peter.kaestle@nokia.com> > --- > t/t7400-submodule-basic.sh | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/t/t7400-submodule-basic.sh b/t/t7400-submodule-basic.sh > index 7f75bb1..2e84914 100755 > --- a/t/t7400-submodule-basic.sh > +++ b/t/t7400-submodule-basic.sh > @@ -399,6 +399,14 @@ test_expect_success 'init should register submodule url in .git/config' ' > test_cmp expect url > ' > > +test_expect_success 'status should still be "missing" after initializing' ' > + rm -fr init && > + mkdir init && > + git submodule status >lines && > + rm -fr init && > + grep "^-$rev1" lines > +' > + > test_failure_with_unknown_submodule () { > test_must_fail git submodule $1 no-such-submodule 2>output.err && > test_i18ngrep "^error: .*no-such-submodule" output.err
diff --git a/t/t7400-submodule-basic.sh b/t/t7400-submodule-basic.sh index 7f75bb1..2e84914 100755 --- a/t/t7400-submodule-basic.sh +++ b/t/t7400-submodule-basic.sh @@ -399,6 +399,14 @@ test_expect_success 'init should register submodule url in .git/config' ' test_cmp expect url ' +test_expect_success 'status should still be "missing" after initializing' ' + rm -fr init && + mkdir init && + git submodule status >lines && + rm -fr init && + grep "^-$rev1" lines +' + test_failure_with_unknown_submodule () { test_must_fail git submodule $1 no-such-submodule 2>output.err && test_i18ngrep "^error: .*no-such-submodule" output.err
Test that submodule status reports initialized but not cloned submodules as missing. Signed-off-by: Peter Kaestle <peter.kaestle@nokia.com> --- t/t7400-submodule-basic.sh | 8 ++++++++ 1 file changed, 8 insertions(+)