Tatsh / destin / 31219682074

100%

push

github

harmonix.unpacker: wrap threaded next() in a lambda for ty

The `ty` type checker mis-inferred the awaited value of
`asyncio.to_thread(next, ark_bytes, None)` as an unresolved type-variable
union rather than `bytes`, because it could not resolve the overloaded
`next` through `to_thread`'s `ParamSpec`. Wrapping the call in a zero-argument
lambda hands `to_thread` a concrete `Callable[[], bytes | None]`, so `ty`
narrows `data` to `bytes` after the `is not None` guard. Runtime behaviour is
unchanged and mypy already accepted the original form.

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

1428 of 1428 branches covered (100.0%)

Branch coverage included in aggregate %.

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

11220 of 11222 relevant lines covered (99.98%)

5.0 hits per line