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

Catch Celery forget() not implemented error #44

Merged
merged 1 commit into from
Aug 7, 2016

Conversation

samuelfekete
Copy link
Contributor

When using AMQP as the backend, result.forget() raises an error, which causes the test to fail even though Celery is working perfectly fine. In such cases this error should be caught and ignored.

@codingjoe
Copy link
Collaborator

Hi @samuelfekete, thanks for the contribution.
I know it's much to task, since there are not many tests yet, but can you add a test, that exploits this behavior?
If you are having trouble with that, let me know what setup you need to recreate the bug? Which celery backend and so on.

Thanks,
Joe

@codingjoe codingjoe added the bug label Jul 8, 2016
@codingjoe codingjoe merged commit 104fa15 into revsys:master Aug 7, 2016
@codingjoe
Copy link
Collaborator

Release in 1.1.2

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

Successfully merging this pull request may close these issues.

None yet

2 participants