Re: [GSoC][PATCH v7 0/5] t: port reftable/tree_test.c to the unit testing framework
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Aug 6, 2024, 06:30 UTC
- Message-ID
- <ZrHDB1-yTv_gD1Mx@tanuki>
- In-Reply-To
- <xmqqr0b33r16.fsf@gitster.g>
On Mon, Aug 05, 2024 at 08:53:09AM -0700, Junio C Hamano wrote:
Show 28 quoted lines
> Patrick Steinhardt <ps@pks.im> writes: > > > On Sun, Aug 04, 2024 at 07:36:44PM +0530, Chandra Pratap wrote: > >> The reftable library comes with self tests, which are exercised > >> as part of the usual end-to-end tests and are designed to > >> observe the end-user visible effects of Git commands. What it > >> exercises, however, is a better match for the unit-testing > >> framework, merged at 8bf6fbd0 (Merge branch 'js/doc-unit-tests', > >> 2023-12-09), which is designed to observe how low level > >> implementation details, at the level of sequences of individual > >> function calls, behave. > >> > >> Hence, port reftable/tree_test.c to the unit testing framework and > >> improve upon the ported test. The first patch in the series is > >> preparatory cleanup, the second patch moves the test to the unit > >> testing framework, and the rest of the patches improve upon the > >> ported test. > >> > >> Mentored-by: Patrick Steinhardt <ps@pks.im> > >> Mentored-by: Christian Couder <chriscool@tuxfamily.org> > >> Signed-off-by: Chandra Pratap <chandrapratap3519@gmail.com> > > > > Only a single change compared to v6, addressing the only feedback on > > that version. So this looks good to me, thanks! > > FWIW, I didn't have other feedback not because I found the rest > perfect, but because I didn't read the series myself carefully, > hoping others are sharing the burden.
Oh, yes. I didn't mean to say that I relied on your feedback being addressed exclusively. I already reviewed v5/v6 of this patch series and found it to be good, and given that there was only a single change proposed by you that I'm happy with it translates into me being in favor of v7, as well.
Patrick