intuit / oauth-jsclient / 177

91%

master: 91%

push

travis-ci

Fix: handle not JSON content in response parsing (#59)

* Coveralls Badge Fix

* Coveralls Badge Fix:Final

* Coveralls Badge Fix : istanbul package added

* Coveralls Badge Fix + snyk added

* Coveralls Badge Fix + snyk added

* Snyk removed from Makefile

* Snyk removed for timebeing

* Add more code coverage for OAuthClient (#54)

* Add better code coverage in OAuthClient

* Fix: ValidateIdToken method and unit tests (#58)

* Fix validateIdToken tests

* Pointing README Badge to Develop

* Pointing README Badge to Develop

* Update Develop Coverage Badge

* Fix: handle not JSON content in response parsing

Some intuit API as invoice download return not JSON content (PDF in this
case). `makeApiCall` wasn't working with it because of mandatory
response body parsing.

So if the response is not JSON, we don't want to parse the body.
And simply let the caller decide what to do with it.

Co-authored-by: abisalehalliprasan <anil_kumar3@intuit.com>
Co-authored-by: Oscar Rabasa <oscargrp@gmail.com>
Co-authored-by: Kevin Tang <kevin.hm.tang@gmail.com>
Co-authored-by: abisalehalliprasan <38014312+abisalehalliprasan@users.noreply.github.com>

161 of 187 branches covered (86.1%)

Branch coverage included in aggregate %.

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

309 of 327 relevant lines covered (94.5%)

16.18 hits per line