On Fri, 7 Mar 2025 at 02:20, Rae Moar <rmoar@xxxxxxxxxx> wrote: > > A bug was identified where the KTAP below caused an infinite loop: > > TAP version 13 > ok 4 test_case > 1..4 > > The infinite loop was caused by the parser not parsing a test plan > if following a test result line. > > Fix this bug to correctly parse test plan line. > > Signed-off-by: Rae Moar <rmoar@xxxxxxxxxx> > --- Looks good. I agree with Brendan that this probably should have a test, but I'm happy to take that as a follow-up if you prefer. Reviewed-by: David Gow <davidgow@xxxxxxxxxx> Cheers, -- David > Changes since v1: > - Remove error reported when test plan is missing. > > tools/testing/kunit/kunit_parser.py | 9 ++++----- > 1 file changed, 4 insertions(+), 5 deletions(-) > > diff --git a/tools/testing/kunit/kunit_parser.py b/tools/testing/kunit/kunit_parser.py > index 29fc27e8949b..da53a709773a 100644 > --- a/tools/testing/kunit/kunit_parser.py > +++ b/tools/testing/kunit/kunit_parser.py > @@ -759,7 +759,7 @@ def parse_test(lines: LineStream, expected_num: int, log: List[str], is_subtest: > # If parsing the main/top-level test, parse KTAP version line and > # test plan > test.name = "main" > - ktap_line = parse_ktap_header(lines, test, printer) > + parse_ktap_header(lines, test, printer) > test.log.extend(parse_diagnostic(lines)) > parse_test_plan(lines, test) > parent_test = True > @@ -768,13 +768,12 @@ def parse_test(lines: LineStream, expected_num: int, log: List[str], is_subtest: > # the KTAP version line and/or subtest header line > ktap_line = parse_ktap_header(lines, test, printer) > subtest_line = parse_test_header(lines, test) > + test.log.extend(parse_diagnostic(lines)) > + parse_test_plan(lines, test) > parent_test = (ktap_line or subtest_line) > if parent_test: > - # If KTAP version line and/or subtest header is found, attempt > - # to parse test plan and print test header > - test.log.extend(parse_diagnostic(lines)) > - parse_test_plan(lines, test) > print_test_header(test, printer) > + > expected_count = test.expected_count > subtests = [] > test_num = 1 > > base-commit: 0619a4868fc1b32b07fb9ed6c69adc5e5cf4e4b2 > -- > 2.49.0.rc0.332.g42c0ae87b1-goog >
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature