On Sat, 13 Jul 2019 00:23:57 +0530 Rohit Ashiwal <rohit.ashiwal...@gmail.com> 
wrote:
>
> rebase am already has this flag to "lie" about the committer date
> by changing it to the author date. Let's add the same for
> interactive machinery.
>
> Signed-off-by: Rohit Ashiwal <rohit.ashiwal...@gmail.com>
> [...]
>
> +test_expect_success '--committer-date-is-author-date works with interactive 
> backend' '
> +     git rebase -f HEAD^ &&
> +     git cat-file commit HEAD | sed -e "/^\$/q" >head &&
> +     sed -ne "/^author /s/.*> //p" head >authortime &&
> +     sed -ne "/^committer /s/.*> //p" head >committertime &&
> +     ! test_cmp at ct &&
                   ^~~^~~~~
Slight mistake in changing names, I'll change it after a full review,
please ignore till then.

> +     git rebase -i --committer-date-is-author-date HEAD^ &&
> +     git cat-file commit HEAD | sed -e "/^\$/q" >head &&
> +     sed -ne "/^committer /s/.*> //p" head >committertime &&
> +     test_cmp authortime committertime
> +'

We should also include a test for am backend since the flag is not
directly passed to the am after the application of this patch.

> +
>  test_done
> --
> 2.21.0

Best
Rohit

Reply via email to