mbox series

[0/2] unicode: kunit: refactor selftest to kunit tests

Message ID 20240922201631.179925-1-gbittencourt@lkcamp.dev (mailing list archive)
Headers show
Series unicode: kunit: refactor selftest to kunit tests | expand

Message

Gabriela Bittencourt Sept. 22, 2024, 8:16 p.m. UTC
Hey all,

We are making these changes as part of a KUnit Hackathon at LKCamp [1].

This patch sets out to refactor fs/unicode/utf8-selftest.c to KUnit tests.

The first commit is the refactoring itself from self test into KUnit, while
the second one applies the naming style conventions.

We appreciate any feedback and suggestions. :)

[1] https://lkcamp.dev/about/

Co-developed-by: Pedro Orlando <porlando@lkcamp.dev>
Co-developed-by: Danilo Pereira <dpereira@lkcamp.dev>
Signed-off-by: Pedro Orlando <porlando@lkcamp.dev>
Signed-off-by: Danilo Pereira <dpereira@lkcamp.dev>
Signed-off-by: Gabriela Bittencourt <gbittencourt@lkcamp.dev>

Gabriela Bittencourt (2):
  unicode: kunit: refactor selftest to kunit tests
  unicode: kunit: change tests filename and path

 fs/unicode/Kconfig                            |   5 +-
 fs/unicode/Makefile                           |   2 +-
 fs/unicode/tests/.kunitconfig                 |   3 +
 .../{utf8-selftest.c => tests/utf8_kunit.c}   | 152 ++++++++----------
 4 files changed, 76 insertions(+), 86 deletions(-)
 create mode 100644 fs/unicode/tests/.kunitconfig
 rename fs/unicode/{utf8-selftest.c => tests/utf8_kunit.c} (63%)

Comments

Pedro Orlando Sept. 23, 2024, 3:42 p.m. UTC | #1
+CC linux-kselftest

-------

On 22/09/2024 17:16, Gabriela Bittencourt wrote:
> Hey all,
> 
> We are making these changes as part of a KUnit Hackathon at LKCamp [1].
> 
> This patch sets out to refactor fs/unicode/utf8-selftest.c to KUnit tests.
> 
> The first commit is the refactoring itself from self test into KUnit, while
> the second one applies the naming style conventions.
> 
> We appreciate any feedback and suggestions. :)
> 
> [1] https://lkcamp.dev/about/
> 
> Co-developed-by: Pedro Orlando <porlando@lkcamp.dev>
> Co-developed-by: Danilo Pereira <dpereira@lkcamp.dev>
> Signed-off-by: Pedro Orlando <porlando@lkcamp.dev>
> Signed-off-by: Danilo Pereira <dpereira@lkcamp.dev>
> Signed-off-by: Gabriela Bittencourt <gbittencourt@lkcamp.dev>
> 
> Gabriela Bittencourt (2):
>    unicode: kunit: refactor selftest to kunit tests
>    unicode: kunit: change tests filename and path
> 
>   fs/unicode/Kconfig                            |   5 +-
>   fs/unicode/Makefile                           |   2 +-
>   fs/unicode/tests/.kunitconfig                 |   3 +
>   .../{utf8-selftest.c => tests/utf8_kunit.c}   | 152 ++++++++----------
>   4 files changed, 76 insertions(+), 86 deletions(-)
>   create mode 100644 fs/unicode/tests/.kunitconfig
>   rename fs/unicode/{utf8-selftest.c => tests/utf8_kunit.c} (63%)
>
Shuah Khan Sept. 23, 2024, 4:41 p.m. UTC | #2
On 9/23/24 09:42, Pedro Orlando wrote:
> +CC linux-kselftest
> 
> -------
> 
> On 22/09/2024 17:16, Gabriela Bittencourt wrote:
>> Hey all,
>>
>> We are making these changes as part of a KUnit Hackathon at LKCamp [1].
>>
>> This patch sets out to refactor fs/unicode/utf8-selftest.c to KUnit tests.
>>
>> The first commit is the refactoring itself from self test into KUnit, while
>> the second one applies the naming style conventions.
>>
>> We appreciate any feedback and suggestions. :)
>>
>> [1] https://lkcamp.dev/about/
>>
>> Co-developed-by: Pedro Orlando <porlando@lkcamp.dev>
>> Co-developed-by: Danilo Pereira <dpereira@lkcamp.dev>
>> Signed-off-by: Pedro Orlando <porlando@lkcamp.dev>
>> Signed-off-by: Danilo Pereira <dpereira@lkcamp.dev>
>> Signed-off-by: Gabriela Bittencourt <gbittencourt@lkcamp.dev>
>>
>> Gabriela Bittencourt (2):
>>    unicode: kunit: refactor selftest to kunit tests
>>    unicode: kunit: change tests filename and path
>>
>>   fs/unicode/Kconfig                            |   5 +-
>>   fs/unicode/Makefile                           |   2 +-
>>   fs/unicode/tests/.kunitconfig                 |   3 +
>>   .../{utf8-selftest.c => tests/utf8_kunit.c}   | 152 ++++++++----------
>>   4 files changed, 76 insertions(+), 86 deletions(-)
>>   create mode 100644 fs/unicode/tests/.kunitconfig
>>   rename fs/unicode/{utf8-selftest.c => tests/utf8_kunit.c} (63%)
>>
> 

Please resend the series with linux-kselftest on the cc.

thanks,
-- Shuah