craigcitro / apitools

88%

master: 89%

push

travis-ci

Fix a bad bounds check for downloads.

Previously, if we created a download with `auto_transfer=True` with the full
download smaller than the default `chunksize`, and then called
`StreamInChunks`, we'd end up downloading the full content twice. This was due
to a bad bounds check in setting a range header -- we'd end up with a header
like `Range: bytes=100-99`, and the server would ignore the start value,
giving us all the bytes again.

This adds a fix and a test.

1722 of 2307 branches covered (74.64%)

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

8266 of 9418 relevant lines covered (87.77%)

0.88 hits per line