test(AuthResponse) Add Additional unit tests for AuthResponse

intuit / oauth-jsclient

Intuit's NodeJS OAuth client provides a set of methods to make it easier to work with OAuth2.0 and Open ID

https://developer.intuit.com/

Apache License 2.0

121 stars 156 forks source link

test(AuthResponse) Add Additional unit tests for AuthResponse #47

Closed edgarsherman closed 4 years ago

edgarsherman commented 4 years ago

Added tests for AuthResponse to bring it to 100%
modified nyc config to exclude test directory - incorrectly affecting the coverage numbers
modified nyc config to report per-file coverage at the end of a test run
reduced coverage threshold so we still meet standards (Note: I'll continue to add tests for OAuthClient.js to bring it up)

Reference issue #39, but only partially bring code coverage up. I'll continue to add more in a different PR.

abisalehalliprasan commented 4 years ago

Added tests for AuthResponse to bring it to 100%

modified nyc config to exclude test directory - incorrectly affecting the coverage numbers

modified nyc config to report per-file coverage at the end of a test run

reduced coverage threshold so we still meet standards (Note: I'll continue to add tests for OAuthClient.js to bring it up)

Reference issue #39, but only partially bring code coverage up. I'll continue to add more in a different PR.

Sounds good !

abisalehalliprasan commented 4 years ago

🚀 PR was released in v2.0.0 hacktoberfest