Update the Mercurial unit tests to use build_client() and clean up tests.

Review Request #12508 — Created Aug. 5, 2022 and submitted

Information

RBTools
release-4.x

Reviewers

This updates the Mercurial unit tests to build a MercurialClient in
each test where it's needed, rather than in setUp(). This is in
prepration for new tests that will need to handle client construction
differently.

This also cleans up a lot of the assertions to be less verbose and to be
more comprehensive.

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. We missed things because we weren't
doing these checks.

Given the work being done already in each function, I felt this was the
best time to clean this code up. It's caught some missing state we
weren't checking.

Mercurial unit tests pass.

Summary ID
Update the Mercurial unit tests to use build_client() and clean up tests.
This updates the Mercurial unit tests to build a `MercurialClient` in each test where it's needed, rather than in `setUp()`. This is in prepration for new tests that will need to handle client construction differently. This also cleans up a lot of the assertions to be less verbose and to be more comprehensive. 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. We missed things because we weren't doing these checks. Given the work being done already in each function, I felt this was the best time to clean this code up. It's caught some missing state we weren't checking.
597657ccec6f6de6fa3e3679a826de89238a4c78
chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-4.x (9866ceb)