Benoit Pierre <benoit.pie...@gmail.com> writes:

> Add (failing) tests: with commit changing the environment to let hooks
> know that no editor will be used (by setting GIT_EDITOR to ":"), the
> "edit hunk" functionality does not work (no editor is launched and the
> whole hunk is committed).
>
> Signed-off-by: Benoit Pierre <benoit.pie...@gmail.com>
> ---
>  t/t7513-commit-patch.sh | 32 ++++++++++++++++++++++++++++++++
>  1 file changed, 32 insertions(+)
>  create mode 100755 t/t7513-commit-patch.sh
>
> diff --git a/t/t7513-commit-patch.sh b/t/t7513-commit-patch.sh

Again, as I said, I'll rename this to t7514-commit.patch.sh while I
queue this.

Thanks.

> new file mode 100755
> index 0000000..9311b0c
> --- /dev/null
> +++ b/t/t7513-commit-patch.sh
> @@ -0,0 +1,32 @@
> +#!/bin/sh
> +
> +test_description='hunk edit with "commit -p -m"'
> +. ./test-lib.sh
> +
> +if ! test_have_prereq PERL
> +then
> +     skip_all="skipping '$test_description' tests, perl not available"
> +     test_done
> +fi
> +
> +test_expect_success 'setup (initial)' '
> +     echo line1 >file &&
> +     git add file &&
> +     git commit -m commit1
> +'
> +
> +test_expect_failure 'edit hunk "commit -p -m message"' '
> +     test_when_finished "rm -f editor_was_started" &&
> +     echo more >>file &&
> +     echo e | env GIT_EDITOR="touch editor_was_started" git commit -p -m 
> commit2 file &&
> +     test -r editor_was_started
> +'
> +
> +test_expect_failure 'edit hunk "commit --dry-run -p -m message"' '
> +     test_when_finished "rm -f editor_was_started" &&
> +     echo more >>file &&
> +     echo e | env GIT_EDITOR="touch editor_was_started" git commit -p -m 
> commit3 file &&
> +     test -r editor_was_started
> +'
> +
> +test_done
--
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

Reply via email to