-
Notifications
You must be signed in to change notification settings - Fork 5
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
Automated tests #30
base: master
Are you sure you want to change the base?
Automated tests #30
Conversation
@gd-tmagrys the first 2 commits here are outdated, is that right? |
Yes, because this work was created on top of #29 . |
Implement automated tests for plugin API HTTP resource Add Mockito and Power Mock dependency Implement automated tests for ConfigurationsManager. Implement additional tests for HTTP Client and HTTP resource Exclude artifacts forbidden by maven enforce plugin
@ctapobep, I rebased work regarding this PR on master. Feel free to review. |
Hm. But I see classes like Also, couple of test design comments:
Can be placed in method: |
@ctapobep, I hope that I fix things you've mentioned in my last commit. |
Implementing story:
#16
This PR overrides:
#24
Please review and merge if possible.