Update client tests to enable diff checks and require opting in to diff.

Review Request #12613 — Created Sept. 20, 2022 and submitted

Information

RBTools
release-4.x

Reviewers

build_client() now accepts a needs_diff= argument, indicating if the
test needs to perform diff functionality.

By default, this is False, and attempting to call diff() will result
in an error.

By setting to True, diff tool calculations will be performed and
cached, preventing the tests from having to account for any diff checks
in any spies. A missing compatible tool will cause the test to be
skipped.

Unit tests pass.

Tested with a client failing a diff tool check. Saw the test was skipped.

Summary ID
Update client tests to enable diff checks and require opting in to diff.
`build_client()` now accepts a `needs_diff=` argument, indicating if the test needs to perform diff functionality. By default, this is `False`, and attempting to call `diff()` will result in an error. By setting to `True`, diff tool calculations will be performed and cached, preventing the tests from having to account for any diff checks in any spies. A missing compatible tool will cause the test to be skipped.
d2a94007a83d09a81cf84f7c6914f5533570a9fe
maubin
  1. Ship It!
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-4.x (835f6da)
Loading...