Re: [GSoC][PATCH v2 3/4] t: move tests from reftable/record_test.c to the new unit test
- From
Patrick Steinhardt <ps@pks.im>
- Date
- May 29, 2024, 09:30 UTC
- Message-ID
- <Zlb1m5cwhW_R5EzP@tanuki>
- In-Reply-To
- <20240529070341.4248-4-chandrapratap3519@gmail.com>
On Wed, May 29, 2024 at 12:25:11PM +0530, Chandra Pratap wrote: [snip]
Show 38 quoted lines
> diff --git a/t/unit-tests/t-reftable-basics.c b/t/unit-tests/t-reftable-basics.c
> index 55fcff12d9..b02ca02040 100644
> --- a/t/unit-tests/t-reftable-basics.c
> +++ b/t/unit-tests/t-reftable-basics.c
> @@ -99,13 +99,38 @@ static void test_parse_names_drop_empty(void)
>
> static void test_common_prefix(void)
> {
> - struct strbuf s1 = STRBUF_INIT;
> - struct strbuf s2 = STRBUF_INIT;
> - strbuf_addstr(&s1, "abcdef");
> - strbuf_addstr(&s2, "abc");
> - check_int(common_prefix_size(&s1, &s2), ==, 3);
> - strbuf_release(&s1);
> - strbuf_release(&s2);
> + struct strbuf a = STRBUF_INIT;
> + struct strbuf b = STRBUF_INIT;
> + struct {
> + const char *a, *b;
> + int want;
> + } cases[] = {
> + {"abcdef", "abc", 3},
> + { "abc", "ab", 2 },
> + { "", "abc", 0 },
> + { "abc", "abd", 2 },
> + { "abc", "pqr", 0 },
> + };
> +
> + for (size_t i = 0; i < ARRAY_SIZE(cases); i++) {
> + strbuf_addstr(&a, cases[i].a);
> + strbuf_addstr(&b, cases[i].b);
> + check_int(common_prefix_size(&a, &b), ==, cases[i].want);
> + strbuf_reset(&a);
> + strbuf_reset(&b);
> + }
> + strbuf_release(&a);
> + strbuf_release(&b);
> +}Oh, so this test was even duplicated. It may make sense to point out details like this in the commit message to prepare the reader. But that's probably not worth a reroll.
Patrick