Update client tests to enable diff checks and require opting in to diff.
Review Request #12613 — Created Sept. 20, 2022 and submitted — Latest diff uploaded
build_client()
now accepts aneeds_diff=
argument, indicating if the
test needs to perform diff functionality.By default, this is
False
, and attempting to calldiff()
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.
Diff Revision 1 (Latest)
Commits
Files
rbtools/clients/tests/__init__.py |
---|
rbtools/clients/tests/test_bzr.py |
---|
rbtools/clients/tests/test_clearcase.py |
---|
rbtools/clients/tests/test_cvs.py |
---|
rbtools/clients/tests/test_git.py |
---|
rbtools/clients/tests/test_mercurial.py |
---|
rbtools/clients/tests/test_p4.py |
---|
rbtools/clients/tests/test_sos.py |
---|
rbtools/clients/tests/test_svn.py |
---|
rbtools/diffs/tools/errors.py |
---|