3 Nov
2022
3 Nov
'22
5:45 p.m.
On Wed, Nov 2, 2022 at 11:38 PM David Gow davidgow@google.com wrote:
On Thu, Nov 3, 2022 at 12:40 AM 'Daniel Latypov' via KUnit Development kunit-dev@googlegroups.com wrote:
Let's verify that the parser isn't reporting any errors for valid inputs.
This change also
- does result.status checking on one line
- makes sure we consistently do it outside of the `with` block
Signed-off-by: Daniel Latypov dlatypov@google.com
Looks good, thanks.
Note that this patch does conflict with "kunit: tool: print summary of failed tests if a few failed out of a lot": https://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git/co... It's only a context line issue, though.
Reviewed-by: David Gow davidgow@google.com
Oh huh, I rebased onto the kunit branch and it managed to merge cleanly. I guess `git am` is more picky than `git rebase`.
I'll send a v2 that's rebased to avoid issues applying them.