Re: [PATCH] kunit: tool: Fix bug in parsing test plan

2025-03-15 Thread Brendan Jackman
Hi Rae, On Thu, Mar 06, 2025 at 11:02:13AM -0500, Rae Moar wrote: > On Thu, Mar 6, 2025 at 7:26 AM Brendan Jackman wrote: > > Since I reported this bug, I discovered that the example above is in > > fact valid TAP: > > > > > The plan [...] must appear once, whether at the beginning or end of the

Re: [PATCH] kunit: tool: Fix bug in parsing test plan

2025-03-11 Thread Rae Moar
On Thu, Mar 6, 2025 at 4:00 AM David Gow wrote: > > On Thu, 6 Mar 2025 at 08:29, Rae Moar 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 p

Re: [PATCH] kunit: tool: Fix bug in parsing test plan

2025-03-06 Thread Rae Moar
On Thu, Mar 6, 2025 at 7:26 AM Brendan Jackman wrote: > > On Thu, 6 Mar 2025 at 10:00, David Gow wrote: > > > > On Thu, 6 Mar 2025 at 08:29, Rae Moar wrote: > > > > > > A bug was identified where the KTAP below caused an infinite loop: > > > > > > TAP version 13 > > > ok 4 test_case > > > 1..

Re: [PATCH] kunit: tool: Fix bug in parsing test plan

2025-03-06 Thread Brendan Jackman
On Thu, 6 Mar 2025 at 10:00, David Gow wrote: > > On Thu, 6 Mar 2025 at 08:29, Rae Moar 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 pla

Re: [PATCH] kunit: tool: Fix bug in parsing test plan

2025-03-06 Thread David Gow
On Thu, 6 Mar 2025 at 08:29, Rae Moar 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 bug to correctly parse

[PATCH] kunit: tool: Fix bug in parsing test plan

2025-03-05 Thread Rae Moar
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 bug to correctly parse test plan and add error if test plan is missing. Signed-off-by: