mbox series

[v2,0/1] t/unit-tests: convert hash to use clar test framework

Message ID 20250108120339.225596-1-kuforiji98@gmail.com (mailing list archive)
Headers show
Series t/unit-tests: convert hash to use clar test framework | expand

Message

Seyi Kuforiji Jan. 8, 2025, 12:03 p.m. UTC
Hello,

This small patch series transitions the existing unit test file t-hash.c
to the Clar testing framework. This change is part of our ongoing effort
to standardize our testing approach and enhance maintainability.

changes in v2:
- Some small fixes were made to the macros and test functions
- Link to v1: https://public-inbox.org/git/20250107091932.126673-1-kuforiji98@gmail.com/T/#t

Thanks
Seyi

Seyi Kuforiji (1):
  t/unit-tests: convert hash to use clar test framework

 Makefile                            |  2 +-
 t/meson.build                       |  2 +-
 t/unit-tests/{t-hash.c => u-hash.c} | 75 +++++++++++++++++++----------
 3 files changed, 52 insertions(+), 27 deletions(-)
 rename t/unit-tests/{t-hash.c => u-hash.c} (79%)

Range-diff against v1:
-:  ---------- > 1:  6fde57893d t/unit-tests: convert hash to use clar test framework

Comments

Junio C Hamano Jan. 8, 2025, 3:28 p.m. UTC | #1
Seyi Kuforiji <kuforiji98@gmail.com> writes:

> Hello,
>
> This small patch series transitions the existing unit test file t-hash.c
> to the Clar testing framework. This change is part of our ongoing effort
> to standardize our testing approach and enhance maintainability.

Thanks; this is no longer a series but a single patch ;-)
Seyi Kuforiji Jan. 9, 2025, 7:21 a.m. UTC | #2
On Wed, 8 Jan 2025 at 16:28, Junio C Hamano <gitster@pobox.com> wrote:
>
> Seyi Kuforiji <kuforiji98@gmail.com> writes:
>
> > Hello,
> >
> > This small patch series transitions the existing unit test file t-hash.c
> > to the Clar testing framework. This change is part of our ongoing effort
> > to standardize our testing approach and enhance maintainability.
>
> Thanks; this is no longer a series but a single patch ;-)

You're absolutely right.Thank you for pointing that out :), I'll make
sure to update my phrasing in future submissions.

Thanks
Seyi