Re: [PATCH v4 3/8] fetch: add a test to exercise invalid output formats
- From
Junio C Hamano <gitster@pobox.com>
- Date
- May 9, 2023, 17:58 UTC
- Message-ID
- <xmqqv8h1xu7q.fsf@gitster.g>
- In-Reply-To
- <006ea93afbef8742a823a9444d2b71785a9b4ec3.1683636885.git.ps@pks.im>
Patrick Steinhardt <ps@pks.im> writes:
Show 7 quoted lines
> Add a testcase that exercises the logic when an invalid output format is > passed via the `fetch.output` configuration. > > Signed-off-by: Patrick Steinhardt <ps@pks.im> > --- > t/t5574-fetch-output.sh | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+)
It makes perfect sense to make sure that invalid input gets rejected and the command exits with non-zero status, and it is probably a good thing that the end-user sees a message that explains why the particular input is rejected (even though it adds one more thing that needs to be updated when the message gets reworded).
But do we need to insist on no output to the standard output stream when the command errors out?
Other than that, looking good.
THanks.
Show 30 quoted lines
> diff --git a/t/t5574-fetch-output.sh b/t/t5574-fetch-output.sh
> index f91b654d38..a09750d225 100755
> --- a/t/t5574-fetch-output.sh
> +++ b/t/t5574-fetch-output.sh
> @@ -7,6 +7,25 @@ export GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME
>
> . ./test-lib.sh
>
> +test_expect_success 'fetch with invalid output format configuration' '
> + test_when_finished "rm -rf clone" &&
> + git clone . clone &&
> +
> + test_must_fail git -C clone -c fetch.output= fetch origin >actual.out 2>actual.err &&
> + cat >expect <<-EOF &&
> + fatal: invalid value for ${SQ}fetch.output${SQ}: ${SQ}${SQ}
> + EOF
> + test_must_be_empty actual.out &&
> + test_cmp expect actual.err &&
> +
> + test_must_fail git -C clone -c fetch.output=garbage fetch origin >actual.out 2>actual.err &&
> + cat >expect <<-EOF &&
> + fatal: invalid value for ${SQ}fetch.output${SQ}: ${SQ}garbage${SQ}
> + EOF
> + test_must_be_empty actual.out &&
> + test_cmp expect actual.err
> +'
> +
> test_expect_success 'fetch aligned output' '
> git clone . full-output &&
> test_commit looooooooooooong-tag &&