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

Update remove test and method to use track instead of position

Browse Source
This commit is contained in:
Thomas Adamcik 2010-02-13 23:12:53 +01:00
parent 2c064059d4
commit 151321d200
2 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
mopidy/backends/__init__.py
View File

@ -81,9 +81,8 @@ class BaseCurrentPlaylistController(object):
self.playlist = Playlist(tracks=new_tracks) self.playlist = Playlist(tracks=new_tracks)
def remove(self, position): def remove(self,track):
tracks = self.playlist.tracks tracks = filter(lambda t: t != track, self.playlist.tracks)
del tracks[position]
self.playlist = Playlist(tracks=tracks) self.playlist = Playlist(tracks=tracks)

2
tests/backends/__init__.py
View File

@ -147,7 +147,7 @@ class BaseCurrentPlaylistControllerTest(object):
def test_remove(self): def test_remove(self):
track1 = self.controller.playlist.tracks[1] track1 = self.controller.playlist.tracks[1]
track2 = self.controller.playlist.tracks[2] track2 = self.controller.playlist.tracks[2]
self.controller.remove(1) self.controller.remove(track1)
self.assert_(track1 not in self.controller.playlist.tracks) self.assert_(track1 not in self.controller.playlist.tracks)
self.assertEqual(track2, self.controller.playlist.tracks[1]) self.assertEqual(track2, self.controller.playlist.tracks[1])