Skip to content
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 test for using external services #11

Closed
maZahaca opened this issue Sep 29, 2015 · 3 comments
Closed

Fix test for using external services #11

maZahaca opened this issue Sep 29, 2015 · 3 comments

Comments

@maZahaca
Copy link
Member

Do NOT use external services in test, need to mock them to add more stability to tests

@maZahaca
Copy link
Member Author

One more provider is still using external service.

@remedge Could you look on this? And do almost the same as you did before + a bit refactoring of CbrProvider

@remedge
Copy link
Contributor

remedge commented Nov 10, 2015

@maZahaca i'll do my best)

@remedge
Copy link
Contributor

remedge commented Nov 17, 2015

Here is the PR - #14

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants