Remove dead Last.fm artist band members
This commit is contained in:
parent
5ae5f995cd
commit
c0a25fbabe
|
@ -1738,17 +1738,6 @@ class Artist(_BaseObject, _Taggable):
|
||||||
return self.network._get_url(
|
return self.network._get_url(
|
||||||
domain_name, "artist") % {'artist': artist}
|
domain_name, "artist") % {'artist': artist}
|
||||||
|
|
||||||
def get_band_members(self):
|
|
||||||
"""Returns a list of band members or None if unknown."""
|
|
||||||
|
|
||||||
names = None
|
|
||||||
doc = self._request(self.ws_prefix + ".getInfo", True)
|
|
||||||
|
|
||||||
for node in doc.getElementsByTagName("bandmembers"):
|
|
||||||
names = _extract_all(node, "name")
|
|
||||||
|
|
||||||
return names
|
|
||||||
|
|
||||||
|
|
||||||
class Country(_BaseObject):
|
class Country(_BaseObject):
|
||||||
"""A country at Last.fm."""
|
"""A country at Last.fm."""
|
||||||
|
|
|
@ -237,27 +237,6 @@ class TestPyLastArtist(PyLastTestCase):
|
||||||
# Act / Assert
|
# Act / Assert
|
||||||
self.assertTrue(artist1 != artist2)
|
self.assertTrue(artist1 != artist2)
|
||||||
|
|
||||||
def test_band_members(self):
|
|
||||||
# Arrange
|
|
||||||
artist = pylast.Artist("The Beatles", self.network)
|
|
||||||
|
|
||||||
# Act
|
|
||||||
band_members = artist.get_band_members()
|
|
||||||
|
|
||||||
# Assert
|
|
||||||
self.skip_if_lastfm_api_broken(band_members)
|
|
||||||
self.assertGreaterEqual(len(band_members), 4)
|
|
||||||
|
|
||||||
def test_no_band_members(self):
|
|
||||||
# Arrange
|
|
||||||
artist = pylast.Artist("John Lennon", self.network)
|
|
||||||
|
|
||||||
# Act
|
|
||||||
band_members = artist.get_band_members()
|
|
||||||
|
|
||||||
# Assert
|
|
||||||
self.assertIsNone(band_members)
|
|
||||||
|
|
||||||
def test_artist_get_correction(self):
|
def test_artist_get_correction(self):
|
||||||
# Arrange
|
# Arrange
|
||||||
artist = pylast.Artist("guns and roses", self.network)
|
artist = pylast.Artist("guns and roses", self.network)
|
||||||
|
|
Loading…
Reference in a new issue