Prevent tasks from being called directly.

This is likely to be a large foot-gun for new users. Instead, require an explicit `call` / `acall` method.

Author: RealOrangeOne

django_tasks/task.py

@@ -6,7 +6,6 @@
     TYPE_CHECKING,
     Any,
     Callable,
-    Coroutine,
     Dict,
     Generic,
     Optional,
@@ -129,11 +128,6 @@ async def aget_result(self, result_id: str) -> "TaskResult[T]":
 
         return result
 
-    def __call__(
-        self, *args: P.args, **kwargs: P.kwargs
-    ) -> Union[T, Coroutine[T, None, None]]:
-        return self.func(*args, **kwargs)
-
     def call(self, *args: P.args, **kwargs: P.kwargs) -> T:
         if iscoroutinefunction(self.func):
             return async_to_sync(self.func)(*args, **kwargs)  # type:ignore[no-any-return]

tests/tests/test_tasks.py

@@ -134,14 +134,12 @@ async def test_invalid_priority(self) -> None:
             await test_tasks.noop_task.using(priority=0).aenqueue()
 
     def test_call_task(self) -> None:
-        self.assertEqual(test_tasks.calculate_meaning_of_life(), 42)
         self.assertEqual(test_tasks.calculate_meaning_of_life.call(), 42)
 
     async def test_call_task_async(self) -> None:
         self.assertEqual(await test_tasks.calculate_meaning_of_life.acall(), 42)
 
     async def test_call_async_task(self) -> None:
-        self.assertIsNone(await test_tasks.noop_task_async())  # type:ignore[func-returns-value]
         self.assertIsNone(await test_tasks.noop_task_async.acall())
 
     def test_call_async_task_sync(self) -> None: