Tatsh / niquests-cache / 25070412745

100%

push

github

session: return new headers instead of mutating kwargs

Replace _attach_validators(entry, kwargs) -> bool, which mutated the
caller's kwargs in place, with _validator_headers(entry, current_headers)
-> dict[str, Any] | None, which returns a fresh headers dict (or None
when the entry has no validators). Both request() implementations now
assign the returned dict explicitly, matching the project rule about
returning modified copies rather than mutating arguments.

Tighten _select_headers' return annotation from Mapping[str, str] | None
to dict[str, str] | None so it returns a concrete type, again per
project guidance.

Add an async cache_control test that exercises the no-validators branch
to keep branch coverage at 100%.

Signed-off-by: Andrew Udvare <audvare@gmail.com>

60 of 60 branches covered (100.0%)

Branch coverage included in aggregate %.

10 of 10 new or added lines in 1 file covered. (100.0%)

516 of 516 relevant lines covered (100.0%)

5.0 hits per line