"Derrick Stolee via GitGitGadget" <gitgitgad...@gmail.com> writes:

> From: Derrick Stolee <dsto...@microsoft.com>
>
> The rev-list command is critical to Git's functionality. Ensure it
> works in the three commit-graph environments constructed in
> t6600-test-reach.sh. Here are a few important types of rev-list
> operations:
>
> * Basic: git rev-list --topo-order HEAD
> * Range: git rev-list --topo-order compare..HEAD
> * Ancestry: git rev-list --topo-order --ancestry-path compare..HEAD
> * Symmetric Difference: git rev-list --topo-order compare...HEAD

Could you remind us here which of those operations will be using
generation numbers after this patch series?

> Signed-off-by: Derrick Stolee <dsto...@microsoft.com>
> ---
>  t/t6600-test-reach.sh | 84 +++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 84 insertions(+)
>
> diff --git a/t/t6600-test-reach.sh b/t/t6600-test-reach.sh
> index 9d65b8b946..288f703b7b 100755
> --- a/t/t6600-test-reach.sh
> +++ b/t/t6600-test-reach.sh
> @@ -243,4 +243,88 @@ test_expect_success 'commit_contains:miss' '
>       test_three_modes commit_contains --tag
>  '
>  
> +test_expect_success 'rev-list: basic topo-order' '
> +     git rev-parse \
> +             commit-6-6 commit-5-6 commit-4-6 commit-3-6 commit-2-6 
> commit-1-6 \
> +             commit-6-5 commit-5-5 commit-4-5 commit-3-5 commit-2-5 
> commit-1-5 \
> +             commit-6-4 commit-5-4 commit-4-4 commit-3-4 commit-2-4 
> commit-1-4 \
> +             commit-6-3 commit-5-3 commit-4-3 commit-3-3 commit-2-3 
> commit-1-3 \
> +             commit-6-2 commit-5-2 commit-4-2 commit-3-2 commit-2-2 
> commit-1-2 \
> +             commit-6-1 commit-5-1 commit-4-1 commit-3-1 commit-2-1 
> commit-1-1 \
> +     >expect &&
> +     run_three_modes git rev-list --topo-order commit-6-6
> +'

I wonder if this test could be make easier to write and less error
prone, e.g. creating it from ASCII-art graphics.

But it is good enough.

[...]

--
Jakub Narębski

Reply via email to