andpp/mopidy
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

mpd: fix flake8 errors

Browse Source
This commit is contained in:
Alex Malone 2015-10-28 22:28:40 -05:00 committed by Stein Magnus Jodal
parent d6afcf0abf
commit 7aa8aa2967
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
mopidy/mpd/protocol/stored_playlists.py
View File

@ -154,13 +154,15 @@ def playlistadd(context, name, track_uri):
if not playlist:
# Create new playlist with this single track
lookup_res = context.core.library.lookup(uris=[track_uri]).get()
tracks = [track for selections in lookup_res.values() for track in selections]
tracks = [track for selections in lookup_res.values()
for track in selections]
_create_playlist(context, name, tracks)
else:
# Add track to existing playlist
uri_scheme = urlparse.urlparse(track_uri).scheme
lookup_res = context.core.library.lookup(uris=[track_uri]).get()
to_add = [track for selections in lookup_res.values() for track in selections]
to_add = [track for selections in lookup_res.values()
for track in selections]
playlist = playlist.replace(tracks=list(playlist.tracks) + to_add)
if context.core.playlists.save(playlist).get() is None:
playlist_scheme = urlparse.urlparse(playlist.uri).scheme
@ -186,7 +188,8 @@ def _create_playlist(context, name, tracks):
# Created and saved
return
# Can't use backend appropriate for passed uri schemes, use default one
default_scheme = context.dispatcher.config['mpd']['default_playlist_scheme']
default_scheme = context.dispatcher.config[
'mpd']['default_playlist_scheme']
playlist = context.core.playlists.create(name, default_scheme).get()
if not playlist:
# If even MPD's default backend can't save playlist, everything is lost

3
tests/mpd/protocol/test_stored_playlists.py
View File

@ -292,7 +292,8 @@ class PlaylistsHandlerTest(protocol.BaseTestCase):
name='old_name', uri='dummy:a1', tracks=[Track(uri='b')])])
self.send_request('rename "old_name" "new_name"')
self.assertInResponse('OK')
self.assertIsNotNone(self.backend.playlists.lookup("dummy:new_name").get())
self.assertIsNotNone(
self.backend.playlists.lookup("dummy:new_name").get())
def test_rm(self):
self.backend.playlists.set_dummy_playlists([