From 332d917b5334ed1adc7eb8b2d1a1038521fdf071 Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Wed, 31 Mar 2010 23:44:17 +0200 Subject: [PATCH] Update frontend to match CPC.get() changes --- mopidy/mpd/frontend.py | 8 ++++---- tests/mpd/frontend_test.py | 6 +++--- 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/mopidy/mpd/frontend.py b/mopidy/mpd/frontend.py index 97ceb097..c2ac84ea 100644 --- a/mopidy/mpd/frontend.py +++ b/mopidy/mpd/frontend.py @@ -302,7 +302,7 @@ class MpdFrontend(object): try: track = self.backend.current_playlist.get(id=songid) return self.backend.current_playlist.remove(track) - except KeyError as e: + except LookupError as e: raise MpdAckError(e[0]) @handle_pattern(r'^clear$') @@ -390,7 +390,7 @@ class MpdFrontend(object): try: track = self.backend.current_playlist.get(uri=needle) return track.mpd_format() - except KeyError: + except LookupError: return None raise MpdNotImplemented # TODO @@ -409,7 +409,7 @@ class MpdFrontend(object): songid = int(songid) track = self.backend.current_playlist.get(id=songid) return track.mpd_format() - except KeyError as e: + except LookupError as e: raise MpdAckError(e[0]) else: return self.backend.current_playlist.playlist.mpd_format() @@ -834,7 +834,7 @@ class MpdFrontend(object): else: track = self.backend.current_playlist.get(id=songid) return self.backend.playback.play(track) - except KeyError as e: + except LookupError as e: raise MpdAckError(e[0]) @handle_pattern(r'^play "(?P\d+)"$') diff --git a/tests/mpd/frontend_test.py b/tests/mpd/frontend_test.py index dbc0e511..4fb829ba 100644 --- a/tests/mpd/frontend_test.py +++ b/tests/mpd/frontend_test.py @@ -476,7 +476,7 @@ class PlaybackControlHandlerTest(unittest.TestCase): def test_playid_which_does_not_exist(self): self.b.current_playlist.load(Playlist(tracks=[Track(id=0)])) result = self.h.handle_request(u'playid "1"') - self.assert_(u'ACK Track matching "id=1" not found' in result) + self.assert_(u'ACK "id=1" match no tracks' in result) def test_previous(self): result = self.h.handle_request(u'previous') @@ -598,7 +598,7 @@ class CurrentPlaylistHandlerTest(unittest.TestCase): self.assertEquals(self.b.current_playlist.playlist.length, 2) result = self.h.handle_request(u'deleteid "0"') self.assertEquals(self.b.current_playlist.playlist.length, 2) - self.assert_(u'ACK Track matching "id=0" not found' in result) + self.assert_(u'ACK "id=0" match no tracks' in result) def test_move_songpos(self): self.b.current_playlist.load(Playlist(tracks=[ @@ -700,7 +700,7 @@ class CurrentPlaylistHandlerTest(unittest.TestCase): self.b.current_playlist.load(Playlist( tracks=[Track(name='a', id=33), Track(name='b', id=38)])) result = self.h.handle_request(u'playlistid "25"') - self.assert_(u'ACK Track matching "id=25" not found' in result) + self.assert_(u'ACK "id=25" match no tracks' in result) def test_playlistinfo_without_songpos_or_range(self): result = self.h.handle_request(u'playlistinfo')