Update the Bazaar unit tests to use build_client() and clean up asserts.
Review Request #12502 — Created Aug. 5, 2022 and submitted
This updates the Bazaar unit tests to build a
BazaarClient
in each
test where it's needed, rather than insetUp()
. This is in prepration
for new tests that will need to handle client construction differently.The test suite temporarily implements a wrapping
build_client()
function that sets the appropriatebzr
/brz
binary as the default for
the client. When the new dependency code is ready, this function will be
removed.This also cleans up a lot of the assertions to be less verbose, and
more specific. We had a lot of combinations of checking instance type,
key presence, and then values, but this is far less useful (less
accurate) than just checking dictionaries for equality. Given the work
being done already in each function, I felt this was the best time to
clean this code up.
All unit tests pass.
Summary | ID |
---|---|
a88d1dbac6dd6cccb3962ab2134aa2854ca06813 |
- Change Summary:
-
Updated to target RBTools 4.0.
- Commits:
-
Summary ID 5653ffd516af0b282bc5d5904607eb564b6a6190 a88d1dbac6dd6cccb3962ab2134aa2854ca06813 - Branch:
-
release-3.xrelease-4.x
- Diff:
-
Revision 2 (+404 -278)