flake8: Fix new warnings after flake8 upgrade

(cherry picked from commit a693993)

Conflicts:
	mopidy/audio/actor.py
	mopidy/audio/playlists.py

mopidy/audio/actor.py

@@ -109,8 +109,8 @@ def _setup_playbin(self):
         playbin = gst.element_factory_make('playbin2')
         playbin.set_property('flags', PLAYBIN_FLAGS)
 
-        playbin.set_property('buffer-size', 2*1024*1024)
-        playbin.set_property('buffer-duration', 2*gst.SECOND)
+        playbin.set_property('buffer-size', 2 * 1024 * 1024)
+        playbin.set_property('buffer-duration', 2 * gst.SECOND)
 
         self._connect(playbin, 'about-to-finish', self._on_about_to_finish)
         self._connect(playbin, 'notify::source', self._on_new_source)

mopidy/audio/playlists.py

@@ -76,8 +76,8 @@ def parse_pls(data):
     for section in cp.sections():
         if section.lower() != 'playlist':
             continue
-        for i in xrange(cp.getint(section, 'numberofentries')):
-            yield cp.get(section, 'file%d' % (i+1))
+        for i in range(cp.getint(section, 'numberofentries')):
+            yield cp.get(section, 'file%d' % (i + 1))
 
 
 def parse_xspf(data):

mopidy/local/__init__.py

@@ -27,7 +27,7 @@ def get_config_schema(self):
         schema['playlists_dir'] = config.Path()
         schema['tag_cache_file'] = config.Deprecated()
         schema['scan_timeout'] = config.Integer(
-            minimum=1000, maximum=1000*60*60)
+            minimum=1000, maximum=1000 * 60 * 60)
         schema['scan_flush_threshold'] = config.Integer(minimum=0)
         schema['excluded_file_extensions'] = config.List(optional=True)
         return schema

mopidy/local/json.py

@@ -75,7 +75,7 @@ def __init__(self, uris):
             parent_uri = None
             child = None
             for i in reversed(range(len(parts))):
-                directory = '/'.join(parts[:i+1])
+                directory = '/'.join(parts[:i + 1])
                 uri = translator.path_to_local_directory_uri(directory)
 
                 # First dir we process is our parent

mopidy/local/translator.py

@@ -49,7 +49,7 @@ def m3u_extinf_to_track(line):
         return Track()
     (runtime, title) = m.groups()
     if int(runtime) > 0:
-        return Track(name=title, length=1000*int(runtime))
+        return Track(name=title, length=1000 * int(runtime))
     else:
         return Track(name=title)
 

setup.cfg

@@ -1,6 +1,10 @@
 [flake8]
 application-import-names = mopidy,tests
 exclude = .git,.tox,build,js
+# Ignored flake8 warnings:
+# - E402 module level import not at top of file
+# - E731 do not assign a lambda expression, use a def
+ignore = E402,E731
 
 [wheel]
 universal = 1

tests/__init__.py

@@ -20,7 +20,7 @@ def __eq__(self, rhs):
         try:
             return isinstance(rhs, self.klass)
         except TypeError:
-            return type(rhs) == type(self.klass)
+            return type(rhs) == type(self.klass)  # flake8: noqa
 
     def __ne__(self, rhs):
         return not self.__eq__(rhs)

tests/utils/test_jsonrpc.py

@@ -614,7 +614,7 @@ def test_inspector_can_describe_a_bunch_of_large_classes(self):
             'core.library': core.LibraryController,
             'core.playback': core.PlaybackController,
             'core.playlists': core.PlaylistsController,
-            'core.tracklist':  core.TracklistController,
+            'core.tracklist': core.TracklistController,
         })
 
         methods = inspector.describe()