🚨 Cover subprocess.py on mypy (#1053)

* 🚨 Cover subprocess.py on mypy

* Update scripts/check

Co-authored-by: euri10 <euri10@users.noreply.github.com>

scripts/check

@@ -10,6 +10,6 @@ set -x
 
 ${PREFIX}black --check --diff --target-version=py36 $SOURCE_FILES
 ${PREFIX}flake8 $SOURCE_FILES
-${PREFIX}mypy
+${PREFIX}mypy --show-error-codes
 ${PREFIX}isort --check --diff --project=uvicorn $SOURCE_FILES
 ${PREFIX}python -m tools.cli_usage --check

setup.cfg

@@ -25,7 +25,8 @@ files =
     uvicorn/middleware/asgi2.py,
     uvicorn/_handlers,
     uvicorn/__init__.py,
-    uvicorn/__main__.py
+    uvicorn/__main__.py,
+    uvicorn/subprocess.py
 
 
 [mypy-tests.*]

uvicorn/subprocess.py

@@ -5,12 +5,21 @@
 import multiprocessing
 import os
 import sys
+from multiprocessing.context import SpawnProcess
+from socket import socket
+from typing import Callable, List, Optional
+
+from uvicorn.config import Config
 
 multiprocessing.allow_connection_pickling()
 spawn = multiprocessing.get_context("spawn")
 
 
-def get_subprocess(config, target, sockets):
+def get_subprocess(
+    config: Config,
+    target: Callable[..., None],
+    sockets: List[socket],
+) -> SpawnProcess:
     """
     Called in the parent process, to instantiate a new child process instance.
     The child is not yet started at this point.
@@ -23,6 +32,7 @@ def get_subprocess(config, target, sockets):
     """
     # We pass across the stdin fileno, and reopen it in the child process.
     # This is required for some debugging environments.
+    stdin_fileno: Optional[int]
     try:
         stdin_fileno = sys.stdin.fileno()
     except OSError:
@@ -38,7 +48,12 @@ def get_subprocess(config, target, sockets):
     return spawn.Process(target=subprocess_started, kwargs=kwargs)
 
 
-def subprocess_started(config, target, sockets, stdin_fileno):
+def subprocess_started(
+    config: Config,
+    target: Callable[..., None],
+    sockets: List[socket],
+    stdin_fileno: Optional[int],
+) -> None:
     """
     Called when the child process starts.