search

pmclanahan / django-celery-email

A Django email backend that uses a celery task for sending the email.
BSD 3-Clause "New" or "Revised" License
477 stars 110 forks source link

Missing coverage for celery_queue_pop, is it dead code? #62

Open GabLeRoux opened 5 years ago

GabLeRoux commented 5 years ago

Hey there, I just sent #61 and from there, we can see a few things covered and some that are missing coverage.

In the tests, there seem to be a celery_queue_pop method that doesn't seem to be used, should it be removed?

There is no coverage for it as shown here:
https://codecov.io/gh/GabLeRoux/django-celery-email/src/codecov-integration/tests/tests.py