-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[cli] Add CLI login unit test #6866
Conversation
@@ -100,7 +100,7 @@ export default class Client extends EventEmitter { | |||
let body; | |||
if (isJSONObject(opts.body)) { | |||
body = JSON.stringify(opts.body); | |||
headers.set('content-type', 'application/json; charset=utf8'); | |||
headers.set('content-type', 'application/json; charset=utf-8'); |
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.
Needed because express' body-parser
is strict about this (which we use for the API test mocks - this is the first test that does a POST
apparently).
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.
This PR currently has a merge conflict. Please resolve this and then re-add the |
Codecov Report
@@ Coverage Diff @@
## main #6866 +/- ##
==========================================
+ Coverage 49.26% 50.71% +1.45%
==========================================
Files 126 126
Lines 4882 4886 +4
Branches 1166 1166
==========================================
+ Hits 2405 2478 +73
+ Misses 2466 2398 -68
+ Partials 11 10 -1
Continue to review full report at Codecov.
|
This has just been sitting in my working tree for a while, might as well push it up.