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

Remove meck in favor of an API stub #7

Closed
rbishop opened this issue Aug 6, 2015 · 2 comments
Closed

Remove meck in favor of an API stub #7

rbishop opened this issue Aug 6, 2015 · 2 comments

Comments

@rbishop
Copy link
Contributor

rbishop commented Aug 6, 2015

Using meck for mocking is kind of weird and unnatural. It's easier and cleaner to just stand up a simple Plug app with a notify route and change the origin config for the test environment. We will probably also need to add Cowboy as a test dependency.

@joshuap
Copy link
Member

joshuap commented Oct 8, 2015

👍

@joshuap joshuap self-assigned this Jan 4, 2016
@dpehrson
Copy link

dpehrson commented Jun 15, 2016

@rbishop @joshuap It may be worth looking into Bypass

I've used it a bit and while there is room for improvement, if you're not doing anything that requires coordinating lots of outgoing requests, it's probably a lighter weight solution.

Edit: Just realized this was a pretty old issue, I just saw it and figured I'd offer a suggestion.

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

4 participants