Anders Kaseorg <[email protected]> writes: > Signed-off-by: Anders Kaseorg <[email protected]> > --- > t/t5516-fetch-push.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/t/t5516-fetch-push.sh b/t/t5516-fetch-push.sh > index 85c7fec..594d7a6 100755 > --- a/t/t5516-fetch-push.sh > +++ b/t/t5516-fetch-push.sh > @@ -238,7 +238,7 @@ test_expect_success 'push with pushInsteadOf' ' > test_expect_success 'push with pushInsteadOf and explicit pushurl > (pushInsteadOf should not rewrite)' ' > mk_empty testrepo && > test_config "url.trash2/.pushInsteadOf" testrepo/ && > - test_config "url.trash3/.pusnInsteadOf" trash/wrong && > + test_config "url.trash3/.pushInsteadOf" trash/wrong && > test_config remote.r.url trash/wrong && > test_config remote.r.pushurl "testrepo/" && > git push r refs/heads/master:refs/remotes/origin/master &&
Interesting. Now an obvious and natural question after seeing this change is how the original test passed with misspelled configuration. Is a test that pushes into "trash/wrong" checking the right outcome? If the reason why the existing tests passed without this patch is because they do not test the right thing, then shouldn't they be corrected together with the above fix? -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
