On Sun, May 12, 2013 at 12:41:31PM +0100, John Keeping wrote:
> diff --git a/t/t6007-rev-list-cherry-pick-file.sh 
> b/t/t6007-rev-list-cherry-pick-file.sh
> index 28d4f6b..378cf3e 100755
> --- a/t/t6007-rev-list-cherry-pick-file.sh
> +++ b/t/t6007-rev-list-cherry-pick-file.sh
> @@ -207,4 +207,20 @@ test_expect_success '--count --left-right' '
>       test_cmp expect actual
>  '
>  
> +cat >expect <<EOF
> +3    2       0
> +EOF
> +
> +test_expect_success 'rev-list --cherry-mark caches patch ids' '
> +     test_config patchid.cacheref patchids &&
> +     git rev-list --cherry-mark --left-right --count F...E >actual &&
> +     test_cmp expect actual &&
> +     git notes --ref patchids show master >cached_master &&

I forgot to update this test, it needs a "| sed -e 1q" in the middle of
this line to make sure that we're only checking the patch ID and not the
diffopts hash and Git version.

> +     git log -p -1 master | git patch-id | sed -e "s/ .*//" >patch-id_master 
> &&
> +     test_cmp patch-id_master cached_master &&
> +     # Get the patch IDs again, now they should be cached
> +     git rev-list --cherry-mark --left-right --count F...E >actual &&
> +     test_cmp expect actual
> +'
> +
>  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