diff options
| author | Daniel Latypov <dlatypov@google.com> | 2020-12-02 20:08:22 +0100 |
|---|---|---|
| committer | Shuah Khan <skhan@linuxfoundation.org> | 2021-02-08 23:34:50 +0100 |
| commit | 0b3e68076bb9a8e1b1bd448994b9c57828173d8e (patch) | |
| tree | 0b5ec6c2edef6fd01a68083c8b44f5c5a76dcc84 /virt | |
| parent | kunit: tool: fix unit test cleanup handling (diff) | |
| download | linux-0b3e68076bb9a8e1b1bd448994b9c57828173d8e.tar.xz linux-0b3e68076bb9a8e1b1bd448994b9c57828173d8e.zip | |
kunit: tool: stop using bare asserts in unit test
Use self.assertEqual/assertNotEqual() instead.
Besides being more appropriate in a unit test, it'll also give a better
error message by show the unexpected values.
Also
* Delete redundant check of exception types. self.assertRaises does this.
* s/kall/call. There's no reason to name it this way.
* This is probably a misunderstanding from the docs which uses it
since `mock.call` is in scope as `call`.
Signed-off-by: Daniel Latypov <dlatypov@google.com>
Reviewed-by: David Gow <davidgow@google.com>
Tested-by: Brendan Higgins <brendanhiggins@google.com>
Acked-by: Brendan Higgins <brendanhiggins@google.com>
Signed-off-by: Shuah Khan <skhan@linuxfoundation.org>
Diffstat (limited to 'virt')
0 files changed, 0 insertions, 0 deletions