Chandra Pratap <chandrapratap3519@xxxxxxxxx> writes:
[snip]
> @@ -369,14 +357,15 @@ static void test_reftable_index_record_roundtrip(void)
> strbuf_release(&in.u.idx.last_key);
> }
>
> -int record_test_main(int argc, const char *argv[])
> +int cmd_main(int argc, const char *argv[])
> {
> - RUN_TEST(test_reftable_log_record_equal);
> - RUN_TEST(test_reftable_log_record_roundtrip);
> - RUN_TEST(test_reftable_ref_record_roundtrip);
> - RUN_TEST(test_varint_roundtrip);
> - RUN_TEST(test_key_roundtrip);
> - RUN_TEST(test_reftable_obj_record_roundtrip);
> - RUN_TEST(test_reftable_index_record_roundtrip);
> - return 0;
> + TEST(test_reftable_log_record_equal(), "reftable_log_record_equal works");
> + TEST(test_reftable_log_record_roundtrip(), "record operations work on log record");
> + TEST(test_reftable_ref_record_roundtrip(), "record operations work on ref record");
> + TEST(test_varint_roundtrip(), "put_var_int and get_var_int work");
> + TEST(test_key_roundtrip(), "reftable_encode_key and reftable_decode_key work");
> + TEST(test_reftable_obj_record_roundtrip(), "record operations work on obj record");
> + TEST(test_reftable_index_record_roundtrip(), "record operations work on index record");
> +
> + return test_done();
> }
> --
> 2.45.2.404.g9eaef5822c
I think it would be nice to be consistent with the naming of the tests
with other unit-tests as mentioned in the previous version too [1].
[1]: https://lore.kernel.org/r/CAOLa=ZSmnMLMoKKWMiM7M4Jw8CJ0Jvrs0oMLy18FHaLv_6s6yA@xxxxxxxxxxxxxx
Attachment:
signature.asc
Description: PGP signature
