mirror of
https://github.com/git/git.git
synced 2026-03-08 00:07:41 +01:00
t-reftable-stack: use reftable_ref_record_equal() to compare ref records
In the current stack tests, ref records are compared for equality by sometimes using the dedicated function for ref-record comparison, reftable_ref_record_equal(), and sometimes by explicity comparing contents of the ref records. The latter method is undesired because there can exist unequal ref records with some of the contents being equal. Replace the latter instances of ref-record comparison with the former. This has the added benefit of preserving uniformity throughout the test file. Mentored-by: Patrick Steinhardt <ps@pks.im> Mentored-by: Christian Couder <chriscool@tuxfamily.org> Signed-off-by: Chandra Pratap <chandrapratap3519@gmail.com> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
committed by
Junio C Hamano
parent
476abc39ba
commit
1052280136
@@ -174,7 +174,7 @@ static void t_reftable_stack_add_one(void)
|
||||
|
||||
err = reftable_stack_read_ref(st, ref.refname, &dest);
|
||||
check(!err);
|
||||
check_str("master", dest.value.symref);
|
||||
check(reftable_ref_record_equal(&ref, &dest, GIT_SHA1_RAWSZ));
|
||||
check_int(st->readers_len, >, 0);
|
||||
|
||||
#ifndef GIT_WINDOWS_NATIVE
|
||||
@@ -285,7 +285,7 @@ static void t_reftable_stack_transaction_api(void)
|
||||
err = reftable_stack_read_ref(st, ref.refname, &dest);
|
||||
check(!err);
|
||||
check_int(REFTABLE_REF_SYMREF, ==, dest.value_type);
|
||||
check_str("master", dest.value.symref);
|
||||
check(reftable_ref_record_equal(&ref, &dest, GIT_SHA1_RAWSZ));
|
||||
|
||||
reftable_ref_record_release(&dest);
|
||||
reftable_stack_destroy(st);
|
||||
|
||||
Reference in New Issue
Block a user