mbox series

[0/5] string: Merge separate tests into string_kunit.c

Message ID 20240419140017.work.012-kees@kernel.org (mailing list archive)
Headers show
Series string: Merge separate tests into string_kunit.c | expand

Message

Kees Cook April 19, 2024, 2:01 p.m. UTC
Hi,

We have a few lone function tests (strscpy and strcat) that are better
collected into the common string_kunit.c test suite. Perform various
renamings, merge everything, and clean up after them.

-Kees

Kees Cook (5):
  string: Prepare to merge strscpy_kunit.c into string_kunit.c
  string: Merge strscpy KUnit tests into string_kunit.c
  string: Prepare to merge strcat KUnit tests into string_kunit.c
  string: Merge strcat KUnit tests into string_kunit.c
  string: Convert KUnit test names to standard convention

 MAINTAINERS         |   2 -
 lib/Kconfig.debug   |  10 --
 lib/Makefile        |   2 -
 lib/strcat_kunit.c  | 104 ------------------
 lib/string_kunit.c  | 258 +++++++++++++++++++++++++++++++++++++++-----
 lib/strscpy_kunit.c | 142 ------------------------
 6 files changed, 230 insertions(+), 288 deletions(-)
 delete mode 100644 lib/strcat_kunit.c
 delete mode 100644 lib/strscpy_kunit.c

Comments

Andy Shevchenko April 19, 2024, 2:12 p.m. UTC | #1
On Fri, Apr 19, 2024 at 5:02 PM Kees Cook <keescook@chromium.org> wrote:
>
> Hi,
>
> We have a few lone function tests (strscpy and strcat) that are better
> collected into the common string_kunit.c test suite. Perform various
> renamings, merge everything, and clean up after them.

All makes sense to me, thanks for doing this!

Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Ivan Orlov April 19, 2024, 4:28 p.m. UTC | #2
On 4/19/24 15:01, Kees Cook wrote:
> Hi,
> 
> We have a few lone function tests (strscpy and strcat) that are better
> collected into the common string_kunit.c test suite. Perform various
> renamings, merge everything, and clean up after them.
> 
> -Kees
> 
> Kees Cook (5):
>    string: Prepare to merge strscpy_kunit.c into string_kunit.c
>    string: Merge strscpy KUnit tests into string_kunit.c
>    string: Prepare to merge strcat KUnit tests into string_kunit.c
>    string: Merge strcat KUnit tests into string_kunit.c
>    string: Convert KUnit test names to standard convention
> 
>   MAINTAINERS         |   2 -
>   lib/Kconfig.debug   |  10 --
>   lib/Makefile        |   2 -
>   lib/strcat_kunit.c  | 104 ------------------
>   lib/string_kunit.c  | 258 +++++++++++++++++++++++++++++++++++++++-----
>   lib/strscpy_kunit.c | 142 ------------------------
>   6 files changed, 230 insertions(+), 288 deletions(-)
>   delete mode 100644 lib/strcat_kunit.c
>   delete mode 100644 lib/strscpy_kunit.c
> 

I tested all of the patches in the series with kunit_tool, and 
everything works correctly.

Thank you for doing this!

Tested-by: Ivan Orlov <ivan.orlov0322@gmail.com>