Brian Gesiak <modoca...@gmail.com> writes: > No test asserts that "git branch -u refs/heads/my-branch my-branch" > emits a warning. Add a test that does so. > > Signed-off-by: Brian Gesiak <modoca...@gmail.com> > --- > t/t3200-branch.sh | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/t/t3200-branch.sh b/t/t3200-branch.sh > index fcdb867..6164126 100755 > --- a/t/t3200-branch.sh > +++ b/t/t3200-branch.sh > @@ -507,6 +507,14 @@ EOF > test_cmp expected actual > ' > > +test_expect_success '--set-upstream-to shows warning if used to set branch > as own upstream' ' > + git branch --set-upstream-to refs/heads/my13 my13 2>actual && > + cat >expected <<EOF && > +warning: Not setting branch my13 as its own upstream. > +EOF > + test_i18ncmp expected actual > +' > +
Checking the error message is fine, but we are also interested in seeing that we do not leave such a nonsense configuration, if not more. Shouldn't we check the resulting config as well here? > # Keep this test last, as it changes the current branch > cat >expect <<EOF > $_z40 $HEAD $GIT_COMMITTER_NAME <$GIT_COMMITTER_EMAIL> 1117150200 +0000 > branch: Created from master -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
