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

Integration testing #47

Open
pbrisbin opened this issue Dec 3, 2018 · 0 comments
Open

Integration testing #47

pbrisbin opened this issue Dec 3, 2018 · 0 comments
Labels
enhancement New feature or request help wanted Extra attention is needed maintainability

Comments

@pbrisbin
Copy link
Owner

pbrisbin commented Dec 3, 2018

We lack any good way to test things end-to-end: e.g. to assert on the result of some notify(With) call given some scenario. This makes it hard to get a regression test on #31 for example, which led to us not really fixing it.

I'm thinking about something like sticking an optional IORef in Settings that, when present, gets the Events appended to it instead of actually send to Bugsnag. This could even be useful to end-users who want to test Bugsnag-related paths in their own applications.

@pbrisbin pbrisbin added enhancement New feature or request help wanted Extra attention is needed maintainability labels Dec 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed maintainability
Projects
None yet
Development

No branches or pull requests

1 participant