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
Jkmarx/node api v2 unit test #2913
Conversation
jkmarx
commented
Jul 30, 2018
•
edited
edited
- Add unit tests for node v2 api - patch
Codecov Report
@@ Coverage Diff @@
## develop #2913 +/- ##
===========================================
- Coverage 59.38% 59.32% -0.06%
===========================================
Files 435 435
Lines 27321 27259 -62
Branches 1273 1273
===========================================
- Hits 16224 16172 -52
+ Misses 11097 11087 -10
Continue to review full report at Codecov.
|
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.
Looks good. One question is if it would be better to still send { file_uuid: null }
from the client instead of ''
?
@@ -1,5 +1,6 @@ | |||
import mock | |||
import uuid | |||
from urlparse import urljoin |
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.
Alphabetical import ordering
@scottx611x No, empty string is fine. I want to be explicit and avoid the null conversion. |
* Refactor unit tests. * Fix imports. * Jkmarx/node api v2 unit test (#2913) * Move node related to data_set_manager app * Move node v1 api to data_set_manager. * Update api v1 url dependency. * Move node v1 api tests to correct directory. * Remove unused get. * Add unit tests. * Add unit tests and add more responses to api. * Update client call. * Update text. * Fix import order.