Fixed a "wrong" description for the ImproperlyConfigured exception in django_celery_results.settings

diegocastrum · AllexVeldman · diegocastrum · commit d72b7e83b094 · 2025-06-19T14:12:45.000+02:00
Fixed a "wrong" description for the `ImproperlyConfigured` exception raised when `task_props_extension` doesn't complies with the Mapping protocol. At this point `task_props_extension` is just a dict that inherits from Mapping, not an explicit instance. Thanks @AllexVeldman Co-authored-by: Allex <a.veldman@chain-stock.com>
diff --git a/django_celery_results/settings.py b/django_celery_results/settings.py
@@ -29,8 +29,7 @@ def get_task_props_extension(request, task_props):
     task_props_extension = extend_task_props_callback(request, task_props) or {}
     if not isinstance(task_props_extension, Mapping):
         raise ImproperlyConfigured(
-            "CELERY_RESULTS_EXTEND_TASK_PROPS_CALLBACK must return a Mapping "
-            "instance."
+            "CELERY_RESULTS_EXTEND_TASK_PROPS_CALLBACK must return a Mapping."
         )
 
     return task_props_extension

Original file line number	Diff line number	Diff line change
`@@ -29,8 +29,7 @@ def get_task_props_extension(request, task_props):`
`29`	`29`	`task_props_extension = extend_task_props_callback(request, task_props) or {}`
`30`	`30`	`if not isinstance(task_props_extension, Mapping):`
`31`	`31`	`raise ImproperlyConfigured(`
`32`		`- "CELERY_RESULTS_EXTEND_TASK_PROPS_CALLBACK must return a Mapping "`
`33`		`- "instance."`
	`32`	`+ "CELERY_RESULTS_EXTEND_TASK_PROPS_CALLBACK must return a Mapping."`
`34`	`33`	`)`
`35`	`34`
`36`	`35`	`return task_props_extension`