Attention is currently required from: Paul Menzel, Paul Fagerburg, Jan Dabros. Jakub Czapiga has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57555 )
Change subject: tests: Add lib/lzma-test test case ......................................................................
Patch Set 6:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/57555/comment/0bc2159b_d292c096 PS1, Line 8:
It would show, that two(?) new tests are added
This CL adds one test (tests/lib/lzma-test) with six test cases. It's output is (after changing cmocka_run_group_tests() to cb_run_group_tests()):
[==========] tests/lib/lzma-test(tests): Running 6 test(s). [ RUN ] test_ulzman_correct_file(data.1) [ OK ] test_ulzman_correct_file(data.1) [ RUN ] test_ulzman_correct_file(data.2) [ OK ] test_ulzman_correct_file(data.2) [ RUN ] test_ulzman_correct_file(data.3) [ OK ] test_ulzman_correct_file(data.3) [ RUN ] test_ulzman_correct_file(data.4) [ OK ] test_ulzman_correct_file(data.4) [ RUN ] test_ulzman_input_too_small [ OK ] test_ulzman_input_too_small [ RUN ] test_ulzman_zero_buffer [ OK ] test_ulzman_zero_buffer [==========] tests/lib/lzma-test(tests): 6 test(s) run.
I do not see, why I should add this output to the commit message. Format is same for all tests.