Cleanup some typing (#3014)

* Cleanup some typing

* Cleanup some typing

* Use actual dumps in test

Author: ahopkins

sanic/app.py

@@ -1672,14 +1672,15 @@ def _loop_add_task(
         name: Optional[str] = None,
         register: bool = True,
     ) -> Task:
+        tsk: Task = task
         if not isinstance(task, Future):
             prepped = cls._prep_task(task, app, loop)
-            task = loop.create_task(prepped, name=name)
+            tsk = loop.create_task(prepped, name=name)
 
         if name and register:
-            app._task_registry[name] = task
+            app._task_registry[name] = tsk
 
-        return task
+        return tsk
 
     @staticmethod
     async def dispatch_delayed_tasks(
@@ -1708,7 +1709,7 @@ async def dispatch_delayed_tasks(
     async def run_delayed_task(
         app: Sanic,
         loop: AbstractEventLoop,
-        task: Task[Any],
+        task: Union[Future[Any], Task[Any], Awaitable[Any]],
     ) -> None:
         """Executes a delayed task within the context of a given app and loop.
 

sanic/mixins/static.py

@@ -248,7 +248,7 @@ async def _static_request_handler(
         self,
         request: Request,
         *,
-        file_or_directory: PathLike,
+        file_or_directory: str,
         use_modified_since: bool,
         use_content_range: bool,
         stream_large_files: Union[bool, int],
@@ -258,7 +258,7 @@ async def _static_request_handler(
     ):
         not_found = FileNotFound(
             "File not found",
-            path=file_or_directory,
+            path=Path(file_or_directory),
             relative_url=__file_uri__,
         )
 

tests/test_requests.py

@@ -19,7 +19,7 @@
 from sanic.constants import DEFAULT_HTTP_CONTENT_TYPE
 from sanic.exceptions import ServerError
 from sanic.request import RequestParameters
-from sanic.response import html, json, text
+from sanic.response import BaseHTTPResponse, html, json, text
 
 
 def encode_basic_auth_credentials(username, password):
@@ -2253,8 +2253,9 @@ async def delete(request, foo):
             {"name": request.route.name, "body": str(request.body), "foo": foo}
         )
 
+    dumps = BaseHTTPResponse._dumps
     payload = {"test": "OK"}
-    data = str(json_dumps(payload).encode())
+    data = str(dumps(payload).encode())
 
     _, response = app.test_client.put("/", json=payload)
     assert response.status == 200