Refines and mocks tests - Completes #25 #32
Conversation
commands/get.js
Outdated
|
||
return output; | ||
|
||
if( (userInput.options.json === true || config.json === true) && userInput.options.json !== false) { |
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.
if( (
-> if ((
also would be cool to have the same files formats convention across all the projects
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.
#33 addresses it, nevermind
|
||
}); | ||
|
||
}); |
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.
missing empty line at the end
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.
no
package.json
Outdated
@@ -34,7 +34,8 @@ | |||
"mocha": "^3.3.0", | |||
"nyc": "^10.3.2", | |||
"should": "^11.2.1", | |||
"sinon": "^2.3.1" | |||
"sinon": "^2.3.1", | |||
"sinon-as-promised": "^4.0.3" |
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.
please fix versions
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.
please look at comments
empty line in the end is fixed, everything else resolved
Completes #25