handling for 304 not modified responses

[joesarre]

Added a fix for response = None

[ardydedase]

Hi @joesarre,

Unit tests are broken for this pull request: https://travis-ci.org/Skyscanner/skyscanner-python-sdk/builds/81211741 and coverage have decreased.

Looks like we'll need to update the unit tests for this pull request.

Thanks,
Ardy

[joesarre]

Cheers, yeah I'll have another go at it when I can. I think the tests are
failing because of the rate limit, but agree on coverage. Hard to test for
as that code path requires the underlying API to return a 304 which only
happens occasionally. I stopped short of introducing mocks into the tests.
On 19 Sep 2015 21:47, "Ardy Dedase" [email protected] wrote:

Hi @joesarre https://github.com/joesarre,

Unit tests are broken for this pull request:
https://travis-ci.org/Skyscanner/skyscanner-python-sdk/builds/81211741
and coverage have decreased.

Looks like we'll need to update the unit tests for this pull request.

Thanks,
Ardy

—
Reply to this email directly or view it on GitHub
#11 (comment)
.