Re: [PATCH v2 2/2] push: suggest <remote> <branch> for a slash slip
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 24, 2026, 22:42 UTC
- Message-ID
- <xmqqa4sjh85o.fsf@gitster.g>
- In-Reply-To
- <49de5a925de506ed9a141eb72927b2548b73af22.1782338114.git.gitgitgadget@gmail.com>
"Harald Nordgren via GitGitGadget" <gitgitgadget@gmail.com> writes:
Show 30 quoted lines
> diff --git a/t/t5529-push-errors.sh b/t/t5529-push-errors.sh
> index 80b06a0cd2..cfb294305d 100755
> --- a/t/t5529-push-errors.sh
> +++ b/t/t5529-push-errors.sh
> @@ -54,6 +54,37 @@ test_expect_success 'detect empty remote with targeted refspec' '
> grep "fatal: bad repository ${SQ}${SQ}" stderr
> '
>
> +test_expect_success 'suggest <remote> <branch> for a <remote>/<branch> slip' '
> + test_must_fail git push origin/main 2>stderr &&
> + grep "${SQ}origin/main${SQ} is not a valid push target" stderr &&
> + grep "hint: Did you mean to use: git push origin main?" stderr &&
> + test_must_fail git -c advice.pushRepoLooksLikeRef=false push origin/main 2>stderr &&
> + ! grep "Did you mean" stderr
> +'
> +
> +test_expect_success 'suggest <remote> <branch> when the branch has slashes' '
> + test_must_fail git push origin/feature/x 2>stderr &&
> + grep "hint: Did you mean to use: git push origin feature/x?" stderr
> +'
> +
> +test_expect_success 'no suggestion when prefix is not a configured remote' '
> + test_must_fail git push not-a-remote/main 2>stderr &&
> + ! grep "Did you mean" stderr
> +'
> +
> +test_expect_success 'no suggestion for a trailing slash with no branch' '
> + test_must_fail git push origin/ 2>stderr &&
> + ! grep "Did you mean" stderr
> +'t5529-push-errors.sh:59: error: bare grep outside pipeline (use test_grep) t5529-push-errors.sh:60: error: bare grep outside pipeline (use test_grep) t5529-push-errors.sh:62: error: bare grep outside pipeline (use test_grep) t5529-push-errors.sh:67: error: bare grep outside pipeline (use test_grep) t5529-push-errors.sh:72: error: bare grep outside pipeline (use test_grep) t5529-push-errors.sh:77: error: bare grep outside pipeline (use test_grep) t5529-push-errors.sh:84: error: bare grep outside pipeline (use test_grep)