During testing do not postpone execution of the tasks

[ticosax]

No description provided.

[nickbruun]

We should probably have test cases verifying that this works as expected depending on the configuration of ALWAYS_EAGER. @ticosax, will you add those?

[ticosax]

sure

[ticosax]

@nickbruun I added the test.

[nickbruun]

Let's get some docstrings on this for good meassure.

[ticosax]

ok done. I will squash all commits once you agree.

[nickbruun]

Come to think of it, this would defeat the purpose of django_atomic_celery and give unexpectedly different behavior when using and when not using CELERY_ALWAYS_EAGER. Instead, tasks should be executed immediately rather than scheduled when the outermost transaction block is reached if CELERY_ALWAYS_EAGER is set, if testing (which is usually the case whereCELERY_ALWAYS_EAGERis employed) is to be deterministic.

[ticosax]

I'm not sure I understand your point.
you wrote:

Instead, tasks should be executed immediately rather than scheduled when the outermost transaction block is reached if CELERY_ALWAYS_EAGER is set

I agree with you and it was my intent; the test is proving it.
Am I missing something ?