Add dependency checks for ClearCase.

Review Request #12546 — Created Aug. 18, 2022 and submitted — Latest diff uploaded

Information

RBTools
release-4.x

Reviewers

This updates ClearCaseClient to add formal support for dependency
checks. ClearCaseClient.check_dependencies() now determines if
cleartool is installed, and no longer invokes it upon instantiation of
the class.

The ClearCaseClient.host_properties attribute used to be set when the
client was instantiated. Now it's a property that retrieves the
information upon first access, performing a dependency check if needed
(and emitting a warning if one was not already performed).

Unit tests have been updated to include new dependency-related tests.
Since the existing ClearCase tests are designed to use spies to simulate
results, rather than depending on cleartool for test runs, those have
been updated to turn off dependency checks when setting up a client.

Unit tests pass on Python 3.7-3.11.

Diff Revision 2 (Latest)

orig
1
2

Commits

First Last Summary ID Author
Add dependency checks for ClearCase.
This updates `ClearCaseClient` to add formal support for dependency checks. `ClearCaseClient.check_dependencies()` now determines if `cleartool` is installed, and no longer invokes it upon instantiation of the class. The `ClearCaseClient.host_properties` attribute used to be set when the client was instantiated. Now it's a property that retrieves the information upon first access, performing a dependency check if needed (and emitting a warning if one was not already performed). Unit tests have been updated to include new dependency-related tests. Since the existing ClearCase tests are designed to use spies to simulate results, rather than depending on `cleartool` for test runs, those have been updated to turn off dependency checks when setting up a client.
202d156df859997e34ed229df16d6d3d90cbc67a Christian Hammond
rbtools/clients/clearcase.py
rbtools/clients/tests/test_clearcase.py
Loading...