fixes issue with renewTokens that would drop scopes passed to getToken

[aarongranick-okta]

Fixes issue: #942

scopes can be passed in the constructor or into the getToken function (getWithRedirect, getWithoutPrompt, etc.). The scopes are saved with the token and are passed along when renewing a single token or renewing using a refresh token. However when all tokens are renewed together (default action when autoRenew is true), the scopes passed to getToken were being dropped and the scopes passed in constructor were used instead.

To reproduce the issue clearly, some "integration" tests which run against a live environment but are not using a test app or a web browser. Rewrite unit tests for renewTokens to cover the specific logic of this function. The previous unit tests (oauthUtil.setupFrame) are superseded by the new integration tests.

OKTA-429429

[codecov-commenter]

Codecov Report

Merging #985 (5cce60f) into master (2775ad8) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #985      +/-   ##
==========================================
+ Coverage   92.27%   92.30%   +0.02%     
==========================================
  Files         121      121              
  Lines        3497     3510      +13     
  Branches      726      735       +9     
==========================================
+ Hits         3227     3240      +13     
  Misses        270      270              

Impacted Files	Coverage Δ
lib/oidc/renewTokens.ts	100.00% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2775ad8...5cce60f. Read the comment docs.

[shuowu]

Thanks for adding the integration test suite!