diff options
author | Daniel Latypov <dlatypov@google.com> | 2022-02-24 20:20:34 +0100 |
---|---|---|
committer | Shuah Khan <skhan@linuxfoundation.org> | 2022-04-04 23:22:15 +0200 |
commit | 6bd0f52ee8f400a558f1c0f33e1f3fd3ef4922a8 (patch) | |
tree | 8210d2242ae24a20470156e0af67a700e2c5fe90 /scripts/makelst | |
parent | apparmor: test: Use NULL macros (diff) | |
download | linux-6bd0f52ee8f400a558f1c0f33e1f3fd3ef4922a8.tar.xz linux-6bd0f52ee8f400a558f1c0f33e1f3fd3ef4922a8.zip |
kunit: tool: readability tweaks in KernelCI json generation logic
Use a more idiomatic check that a list is non-empty (`if mylist:`) and
simplify the function body by dedenting and using a dict to map between
the kunit TestStatus enum => KernelCI json status string.
The dict hopefully makes it less likely to have bugs like commit
9a6bb30a8830 ("kunit: tool: fix --json output for skipped tests").
Signed-off-by: Daniel Latypov <dlatypov@google.com>
Reviewed-by: David Gow <davidgow@google.com>
Reviewed-by: Brendan Higgins <brendanhiggins@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'scripts/makelst')
0 files changed, 0 insertions, 0 deletions