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

Call #handle_exception when Worker.perform_one is called #1944

Closed
bogdan opened this issue Sep 11, 2014 · 1 comment
Closed

Call #handle_exception when Worker.perform_one is called #1944

bogdan opened this issue Sep 11, 2014 · 1 comment

Comments

@bogdan
Copy link

bogdan commented Sep 11, 2014

I am testing Sidekiq exception_handlers and it is not trivial task to do because no one of testing methods (inline or fake) runs handle_exception.

I think Woker.perform_one should call it.

If not tell me a way to test exception handlers in 'integration' way so that I ensure that technology stack works correctly together rather than testing exception_handlers block directly.

@mperham
Copy link
Collaborator

mperham commented Sep 11, 2014

This is a conscious choice. I don't supply integration testing infrastructure because it follows the law of diminishing returns. It gets more and more complex but there's always some semantic that you can't reproduce in a test environment.

Test your functionality, not Sidekiq's.

@mperham mperham closed this as completed Sep 11, 2014
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