• 
      

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

    Review Request #12502 — Created Aug. 5, 2022 and submitted — Latest diff uploaded

    Information

    RBTools
    release-4.x

    Reviewers

    This updates the Bazaar unit tests to build a BazaarClient 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.

    The test suite temporarily implements a wrapping build_client()
    function that sets the appropriate bzr/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.

    Commits

    Files