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

2021-01-14 Thread Brendan Higgins
On Thu, Jan 14, 2021 at 4:39 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 3

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

2021-01-14 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"