From efecf5113bda17cef370155616c8de4f33be6f19 Mon Sep 17 00:00:00 2001 From: Adarnof Date: Thu, 22 Feb 2018 18:00:03 -0500 Subject: [PATCH] Correct celery eagerness during tests. I have no idea why this setting name has to be changed. The docs for Celery 4.1.0 (installed) indicate it should be called CELERY_TASK_ALWAYS_EAGER - even with namespace removal TASK_ALWAYS_EAGER doesn't work, but the "old" name of CELERY_ALWAYS_EAGER does. --- tests/celery.py | 2 +- tests/settings.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/celery.py b/tests/celery.py index d2a57085..6492058a 100644 --- a/tests/celery.py +++ b/tests/celery.py @@ -10,7 +10,7 @@ app = Celery('devauth') # Using a string here means the worker don't have to serialize # the configuration object to child processes. -app.config_from_object('django.conf:settings', namespace='CELERY') +app.config_from_object('django.conf:settings') # Load task modules from all registered Django app configs. app.autodiscover_tasks(lambda: settings.INSTALLED_APPS) diff --git a/tests/settings.py b/tests/settings.py index b87f1020..b0304290 100644 --- a/tests/settings.py +++ b/tests/settings.py @@ -14,7 +14,7 @@ NOSE_ARGS = [ ] # Celery configuration -CELERY_TASK_ALWAYS_EAGER = True # Forces celery to run locally for testing +CELERY_ALWAYS_EAGER = True # Forces celery to run locally for testing INSTALLED_APPS += [ 'allianceauth.eveonline.autogroups',