Messages in this thread | | | From | Brendan Higgins <> | Date | Wed, 6 Jul 2022 15:04:39 -0400 | Subject | Re: [PATCH 2/3] kunit: tool: redo how we construct and mock LinuxSourceTree |
| |
On Mon, May 16, 2022 at 3:48 PM 'Daniel Latypov' via KUnit Development <kunit-dev@googlegroups.com> wrote: > > Our main function currently has an optional `linux` argument which is > used to by our unit tests to inject a mock. > We currently have the same code copy-pasted several times to do > if not linux: > linux = MakeRealInstance(cli_args.foo, cli_args.bar, ...) > > But in python, dependency injection isn't necessary or idiomatic when we > can just use mock.patch() to mock things out. > > This change > 1. adds a helper to create a LinuxSourceTree from the cli_args > 2. drops the `linux` parameter in favor of mocking the __init__ func. > > Signed-off-by: Daniel Latypov <dlatypov@google.com>
Reviewed-by: Brendan Higgins <brendanhiggins@google.com>
| |