Fix flake8 warnings in fs and tests modules

Author: althonos

fs/ftpfs.py

@@ -267,7 +267,7 @@ def writable(self):
         return self.mode.writing
 
     def write(self, data):
-        # type: (Union[bytes, bytearray, memoryview, array.array[Any], mmap.mmap]) -> int
+        # type: (Union[bytes, memoryview, array.array[Any], mmap.mmap]) -> int
         if not self.mode.writing:
             raise IOError("File not open for writing")
 
@@ -289,7 +289,7 @@ def write(self, data):
         return data_pos
 
     def writelines(self, lines):
-        # type: (Iterable[Union[bytes, bytearray, memoryview, array.array[Any], mmap.mmap]]) -> None
+        # type: (Iterable[Union[bytes, memoryview, array.array[Any], mmap.mmap]]) -> None  # noqa: E501
         if not self.mode.writing:
             raise IOError("File not open for writing")
         data = bytearray()

fs/info.py

@@ -72,8 +72,8 @@ def _make_datetime(self, t):
         # type: (None) -> None
         pass
 
-    @overload  # noqa: F811
-    def _make_datetime(self, t):
+    @overload
+    def _make_datetime(self, t):  # noqa: F811
         # type: (int) -> datetime
         pass
 
@@ -90,7 +90,7 @@ def get(self, namespace, key):
         pass
 
     @overload  # noqa: F811
-    def get(self, namespace, key, default):
+    def get(self, namespace, key, default):  # noqa: F811
         # type: (Text, Text, T) -> Union[Any, T]
         pass
 

fs/iotools.py

@@ -90,7 +90,7 @@ def truncate(self, size=None):
         return self._f.truncate(size)
 
     def write(self, data):
-        # type: (Union[bytes, bytearray, memoryview, array.array[Any], mmap.mmap]) -> int
+        # type: (Union[bytes, memoryview, array.array[Any], mmap.mmap]) -> int
         if isinstance(data, array.array):
             count = self._f.write(data.tobytes())
         else:
@@ -144,7 +144,7 @@ def readlines(self, hint=None):
         return self._f.readlines(-1 if hint is None else hint)
 
     def writelines(self, lines):
-        # type: (Iterable[Union[bytes, bytearray, memoryview, array.array[Any], mmap.mmap]]) -> None
+        # type: (Iterable[Union[bytes, memoryview, array.array[Any], mmap.mmap]]) -> None  # noqa: E501
         _lines = (
             line.tobytes() if isinstance(line, array.array) else line for line in lines
         )

fs/memoryfs.py

@@ -193,17 +193,15 @@ def writable(self):
         return self._mode.writing
 
     def write(self, data):
-        # type: (Union[bytes, bytearray, memoryview, array.array[Any], mmap.mmap]) -> int
+        # type: (Union[bytes, memoryview, array.array[Any], mmap.mmap]) -> int
         if not self._mode.writing:
             raise IOError("File not open for writing")
         with self._seek_lock():
             self.on_modify()
             return self._bytes_io.write(data)
 
     def writelines(self, sequence):
-        # type: (Iterable[Union[bytes, bytearray, memoryview, array.array[Any], mmap.mmap]]) -> None
-        # FIXME(@althonos): For some reason the stub for IOBase.writelines
-        #      is List[Any] ?! It should probably be Iterable[ByteString]
+        # type: (Iterable[Union[bytes, memoryview, array.array[Any], mmap.mmap]]) -> None  # noqa: E501
         with self._seek_lock():
             self.on_modify()
             self._bytes_io.writelines(sequence)
@@ -248,18 +246,18 @@ def size(self):
                 _bytes_file.seek(0, os.SEEK_END)
                 return _bytes_file.tell()
 
-    @overload  # noqa: F811
-    def get_entry(self, name, default):
+    @overload
+    def get_entry(self, name, default):  # noqa: F811
         # type: (Text, _DirEntry) -> _DirEntry
         pass
 
-    @overload  # noqa: F811
-    def get_entry(self, name):
+    @overload
+    def get_entry(self, name):  # noqa: F811
         # type: (Text) -> Optional[_DirEntry]
         pass
 
-    @overload  # noqa: F811
-    def get_entry(self, name, default):
+    @overload
+    def get_entry(self, name, default):  # noqa: F811
         # type: (Text, None) -> Optional[_DirEntry]
         pass
 

tests/test_appfs.py

@@ -1,6 +1,5 @@
 from __future__ import unicode_literals
 
-import os
 import shutil
 import tempfile
 import unittest

tests/test_ftpfs.py

@@ -28,7 +28,7 @@
 
 try:
     from pytest.mark import slow
-except:
+except ImportError:
 
     def slow(cls):
         return cls