Re: [PATCH 0/5] t/unit-tests: convert unit-tests to use clar

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Seyi

On 20/02/2025 08:29, Seyi Kuforiji wrote:
Hello,

This small patch series transitions a couple more of our existing unit
test files to the Clar testing framework. This change is part of our
ongoing effort to standardize our testing framework to enhance
maintainability.

Overall I think the conversions are correct, but we'd be better to keep the diagnostic messages by using cl_failf().

Best Wishes

Phillip

Thanks
Seyi

Mentored-by: Patrick Steinhardt <ps@xxxxxx>
Signed-off-by: Seyi Kuforiji <kuforiji98@xxxxxxxxx>

Seyi Kuforiji (5):
   t/unit-tests: implement oid helper functions in unit-tests.{c,h}
   t/unit-tests: convert oid-array test to use clar test framework
   t/unit-tests: convert oidmap test to use clar test framework
   t/unit-tests: convert oidtree test to use clar test framework
   t/unit-tests: remove lib-oid.{c,h,o}

  Makefile                                      |   7 +-
  t/meson.build                                 |   7 +-
  t/unit-tests/lib-oid.c                        |  52 ------
  t/unit-tests/lib-oid.h                        |  25 ---
  t/unit-tests/{t-oid-array.c => u-oid-array.c} | 123 +++++++-------
  t/unit-tests/{t-oidmap.c => u-oidmap.c}       | 153 ++++++------------
  t/unit-tests/{t-oidtree.c => u-oidtree.c}     |  78 ++++-----
  t/unit-tests/unit-test.c                      |  42 +++++
  t/unit-tests/unit-test.h                      |  19 +++
  9 files changed, 212 insertions(+), 294 deletions(-)
  delete mode 100644 t/unit-tests/lib-oid.c
  delete mode 100644 t/unit-tests/lib-oid.h
  rename t/unit-tests/{t-oid-array.c => u-oid-array.c} (35%)
  rename t/unit-tests/{t-oidmap.c => u-oidmap.c} (32%)
  rename t/unit-tests/{t-oidtree.c => u-oidtree.c} (44%)






[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux