Re: [PATCH v4 2/4] kunit: tool: Report an error if any test has no subtests

From: Brendan Higgins
Date: Tue Dec 07 2021 - 15:23:06 EST


On Tue, Nov 2, 2021 at 3:30 AM David Gow <davidgow@xxxxxxxxxx> wrote:
>
> It's possible for a test to have a subtest header, but zero valid
> subtests. We used to error on this if the test plan had no subtests
> listed, but it's possible to have subtests without a test plan (indeed,
> this is how parameterised tests work).
>
> Tests with 0 subtests now have the result NO_TESTS, and will report an
> error (which does not halt test execution, but is printed in a scary red
> colour and is noted in the results summary).
>
> Signed-off-by: David Gow <davidgow@xxxxxxxxxx>
> Reviewed-by: Daniel Latypov <dlatypov@xxxxxxxxxx>

Reviewed-by: Brendan Higgins <brendanhiggins@xxxxxxxxxx>