Re: [PATCH v3 2/3] kunit: tool: fix minor typing issue with None status

2021-01-11 Thread Brendan Higgins
On Thu, Jan 7, 2021 at 3:48 PM Daniel Latypov wrote: > > The code to handle aggregating statuses didn't check that the status > actually got set to some non-None value. > Default the value to SUCCESS instead of adding a bunch of `is None` > checks. > > This sorta follows the precedent in commit 3f

[PATCH v3 2/3] kunit: tool: fix minor typing issue with None status

2021-01-07 Thread Daniel Latypov
The code to handle aggregating statuses didn't check that the status actually got set to some non-None value. Default the value to SUCCESS instead of adding a bunch of `is None` checks. This sorta follows the precedent in commit 3fc48259d525 ("kunit: Don't fail test suites if one of them is empty"