Improve typings for multipart.py (#4931)

* Improve multipart.py typings for mock iter

and slightly reformat

* Add .bugfix to CHANGES

CHANGES/4931.bugfix

@@ -0,0 +1 @@
+Fix typings for multipart __aiter__.

CONTRIBUTORS.txt

@@ -40,6 +40,8 @@ Andrii Soldatenko
 Antoine Pietri
 Anton Kasyanov
 Anton Zhdan-Pushkin
+Arseny Timoniq
+Artem Yushkovskiy
 Arthur Darcet
 Ben Bader
 Ben Timby

aiohttp/multipart.py

@@ -162,7 +162,7 @@ def unescape(text: str, *,
 
 
 def content_disposition_filename(params: Mapping[str, str],
-                                 name: str='filename') -> Optional[str]:
+                                 name: str = 'filename') -> Optional[str]:
     name_suf = '%s*' % name
     if not params:
         return None
@@ -259,8 +259,8 @@ def __init__(self, boundary: bytes,
         self._content_eof = 0
         self._cache = {}  # type: Dict[str, Any]
 
-    def __aiter__(self) -> 'BodyPartReader':
-        return self
+    def __aiter__(self) -> Iterator['BodyPartReader']:
+        return self  # type: ignore
 
     async def __anext__(self) -> bytes:
         part = await self.next()
@@ -274,7 +274,7 @@ async def next(self) -> Optional[bytes]:
             return None
         return item
 
-    async def read(self, *, decode: bool=False) -> bytes:
+    async def read(self, *, decode: bool = False) -> bytes:
         """Reads body part data.
 
         decode: Decodes data following by encoding
@@ -290,7 +290,7 @@ async def read(self, *, decode: bool=False) -> bytes:
             return self.decode(data)
         return data
 
-    async def read_chunk(self, size: int=chunk_size) -> bytes:
+    async def read_chunk(self, size: int = chunk_size) -> bytes:
         """Reads body part content chunk of the specified size.
 
         size: chunk size
@@ -534,12 +534,14 @@ def __init__(self, headers: Mapping[str, str],
         self._at_bof = True
         self._unread = []  # type: List[bytes]
 
-    def __aiter__(self) -> 'MultipartReader':
-        return self
+    def __aiter__(
+        self,
+    ) -> Iterator['BodyPartReader']:
+        return self  # type: ignore
 
     async def __anext__(
         self,
-    ) -> Union['MultipartReader', BodyPartReader]:
+    ) -> Optional[Union['MultipartReader', BodyPartReader]]:
         part = await self.next()
         if part is None:
             raise StopAsyncIteration  # NOQA