TheKevJames / coveralls-python / 503ca376-94db-489a-ac81-70b0e82db497

97%

push

circleci

feat(config): source run_at from COVERALLS_RUN_AT and default to now

run_at was only ever populated when a caller set it explicitly in the config
file or a Coveralls() kwarg; no loader read it from the environment and it was
never auto-generated, so in practice the field went unsent. That diverges from
the official coverallsapp/coverage-reporter (and node-coveralls), which read
COVERALLS_RUN_AT and fall back to the current timestamp.

Read COVERALLS_RUN_AT like every other COVERALLS_* var, and in resolve() default
run_at to the current local time in RFC 3339 when no source provides one. The
default is applied on the upload path only, so a bare Config() stays
deterministic. Precedence: kwarg > config file > COVERALLS_RUN_AT > now.

changelog:
The job timestamp (``run_at``) submitted to Coveralls is now read from the
``COVERALLS_RUN_AT`` environment variable and, when unset, defaults to the
current time (RFC 3339). Previously it was only sent if set explicitly in the
config file or as a ``Coveralls()`` argument.

177 of 184 branches covered (96.2%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 2 files covered. (100.0%)

639 of 660 relevant lines covered (96.82%)

5.8 hits per line