TheKevJames / coveralls-python / 29760278946

93%

master: 96%

Pull #754

github

refactor(config)!: resolve every source through one uniform pipeline

Replace the ad-hoc per-service loaders (positional tuples, three different
pull-request parsers, the _coveralls_host/_token_required attrs, and call-time
os.environ reads for host/SSL) with a single resolve() that merges the CI
environment, COVERALLS_* env vars, the config file, and explicit overrides in
one documented precedence order. CI detection returns a uniform dict and
pull-request numbers share one trailing-integer helper.

token_required is not one of those merged config values: it is a calculated
guard against accidental tokenless uploads. resolve() takes it as an explicit
argument (the CLI derives it from --debug/--output) and waives it only on a CI
service that authenticates uploads itself (TOKENLESS_CI_SERVICES, e.g.
TravisCI). It is never read from the config file or env vars, so a committed
.coveralls.yml cannot silently disable the check.

changelog:
Configuration from every source -- CI detection, ``COVERALLS_*`` env vars, the
config file, and explicit overrides -- is now resolved through one pipeline with
a single documented precedence order (later wins), replacing the per-service
special cases. **Breaking (library API):** ``Coveralls()`` no longer accepts
``service_name`` as a dedicated argument; pass it as a keyword override like any
other field (e.g. ``Coveralls(service_name="travis-pro")``). Positional calls
such as ``Coveralls(True, "travis-pro")`` must be updated.

129 of 140 branches covered (92.14%)

Branch coverage included in aggregate %.

115 of 119 new or added lines in 2 files covered. (96.64%)

415 of 443 relevant lines covered (93.68%)

4.68 hits per line