-
Notifications
You must be signed in to change notification settings - Fork 371
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: update code to support typescript@2.6.2 #209
Conversation
@alexander-fenster for review as well |
@JustinBeckwith is the version of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM if nits are addressed.
test/test.oauth2.ts
Outdated
assert.equal(query.client_id, CLIENT_ID); | ||
assert.equal(query.redirect_uri, REDIRECT_URI); | ||
done(); | ||
if (typeof parsed.query === 'string') { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
test/test.oauth2.ts
Outdated
|
||
assert.equal(query.scope, SCOPE_ARRAY.join(' ')); | ||
done(); | ||
if (typeof parsed.query === 'string') { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
test/test.oauth2.ts
Outdated
|
||
assert.equal(query.response_type, 'code'); | ||
done(); | ||
if (typeof parsed.query === 'string') { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
test/test.oauth2.ts
Outdated
@@ -793,17 +803,25 @@ describe('OAuth2 client', () => { | |||
new OAuth2Client(CLIENT_ID, CLIENT_SECRET, REDIRECT_URI); | |||
const generated = oauth2client.generateAuthUrl({}); | |||
const parsed = url.parse(generated); | |||
const query = qs.parse(parsed.query); | |||
assert.equal(query.redirect_uri, REDIRECT_URI); | |||
if (typeof parsed.query === 'string') { |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
The tests for Node.js 4 and Node.js 6 randomly started failing due to what seems like a compiler change. Our
package-lock.json
is pinning totypescript@2.6.1
, which covers Node.js 8. The~2.6.0
tag in ourpackage.json
is letting new patch versions of typescript come through as designed, resulting intypescript@2.6.2
getting installed. I suspect TypeScript made a breaking change in a patch release :(