Remove OptionsStub and support default client options in BaseSCMClient.

Review Request #12531 — Created Aug. 16, 2022 and submitted — Latest diff uploaded

Information

RBTools
release-4.x

Reviewers

The SCMClient unit tests have been using an old OptionsStub in
rbtools.tests (the only thing in that module) to define default
options for a while. This hasn't been kept up-to-date, missing a lot of
defaults that specific tests have had to provide. It also doesn't play
well with type annotations, and isn't worth fixing to do so.

SCMClientTestCase now supports default options that are used in calls
to build_client(). There are set of global defaults
(DEFAULT_SCMCLIENT_OPTIONS) that will always be applied, and a
per-suite set of defaults (default_scmclient_options) that can be set.

These can then be overridden via build_client() as before.

OptionsStub is no longer being used, and will emit a deprecation
warning. We'll remove it in 5.0.

All unit tests pass.

Commits

Files

    Loading...