Re: [PATCH v4 12/12] t5318: use test_oid for HASH_LEN

From: Derrick Stolee <stolee@gmail.com>
Date: Sep 4, 2018, 12:36 UTC
Message-ID: <8af735dc-99cd-f297-415f-24e53ea77344@gmail.com>
In-Reply-To: <20180903232515.336397-13-sandals@crustytoothpaste.net>

On 9/3/2018 7:25 PM, brian m. carlson wrote:

Show 20 quoted lines

> From: Derrick Stolee <stolee@gmail.com>
>
> Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
> Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
> ---
>   t/t5318-commit-graph.sh | 5 +++--
>   1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/t/t5318-commit-graph.sh b/t/t5318-commit-graph.sh
> index 3c1ffad491..d286516f0e 100755
> --- a/t/t5318-commit-graph.sh
> +++ b/t/t5318-commit-graph.sh
> @@ -8,7 +8,8 @@ test_expect_success 'setup full repo' '
>   	cd "$TRASH_DIRECTORY/full" &&
>   	git init &&
>   	git config core.commitGraph true &&
> -	objdir=".git/objects"
> +	objdir=".git/objects" &&
> +	test_oid_init
>   '

Ah yes, thanks for adding this.

Show 11 quoted lines

>   
>   test_expect_success 'verify graph with no graph file' '
> @@ -273,7 +274,7 @@ test_expect_success 'git commit-graph verify' '
>   
>   NUM_COMMITS=9
>   NUM_OCTOPUS_EDGES=2
> -HASH_LEN=20
> +HASH_LEN="$(test_oid rawsz)"
>   GRAPH_BYTE_VERSION=4
>   GRAPH_BYTE_HASH=5
>   GRAPH_BYTE_CHUNK_COUNT=6