-
Notifications
You must be signed in to change notification settings - Fork 20
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(tests): remove swagger-codegen dependecy #230
Conversation
The style in this PR agrees with This formatting comment was generated automatically by a script in uc-cdis/wool. |
1984970
to
171fc86
Compare
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.
very cool! 80% coverage 🎉
a general comment: adding assert res.status_code == 200
after each get/post/put/patch/delete operation is good practice. especially after the post
steps in your case, since the rest of the test is counting on the record being there
please also have a look at what Codacy is complaining about!
fa8931b
to
6fdb91d
Compare
…d into test/improve-test-coverage
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
Pull request for swagger-codegen language removal from testing to get more accurate reading on coveralls. Conftest has been stripped of swagger_client fixtures. Tests previously dependent on swagger_client fixtures now depend on client and user fixtures.
Improvements