mirror of
https://github.com/beetbox/beets.git
synced 2025-12-06 08:39:17 +01:00
Mark albumtype/s expected test failure as fixed
This commit is contained in:
parent
cd52a05d3a
commit
27218a9490
1 changed files with 12 additions and 9 deletions
|
|
@ -701,27 +701,30 @@ class UpdateTest(_common.TestCase):
|
|||
item = self.lib.items().get()
|
||||
self.assertEqual(item.title, 'full')
|
||||
|
||||
@unittest.expectedFailure
|
||||
def test_multivalued_albumtype_roundtrip(self):
|
||||
# https://github.com/beetbox/beets/issues/4528
|
||||
|
||||
# albumtypes is empty for our test fixtures, so populate it first
|
||||
album = self.album
|
||||
# setting albumtypes does not set albumtype currently...
|
||||
# FIXME: When actually fixing the issue 4528, consider whether this
|
||||
# should be set to "album" or ["album"]
|
||||
album.albumtype = "album"
|
||||
album.albumtypes = "album"
|
||||
correct_albumtypes = ["album", "live"]
|
||||
|
||||
# Setting albumtypes does not set albumtype, currently.
|
||||
# Using x[0] mirrors https://github.com/beetbox/mediafile/blob/057432ad53b3b84385e5582f69f44dc00d0a725d/mediafile.py#L1928
|
||||
correct_albumtype = correct_albumtypes[0]
|
||||
|
||||
album.albumtype = correct_albumtype
|
||||
album.albumtypes = correct_albumtypes
|
||||
album.try_sync(write=True, move=False)
|
||||
|
||||
album.load()
|
||||
albumtype_before = album.albumtype
|
||||
self.assertEqual(albumtype_before, "album")
|
||||
self.assertEqual(album.albumtype, correct_albumtype)
|
||||
self.assertEqual(album.albumtypes, correct_albumtypes)
|
||||
|
||||
self._update()
|
||||
|
||||
album.load()
|
||||
self.assertEqual(albumtype_before, album.albumtype)
|
||||
self.assertEqual(album.albumtype, correct_albumtype)
|
||||
self.assertEqual(album.albumtypes, correct_albumtypes)
|
||||
|
||||
|
||||
class PrintTest(_common.TestCase):
|
||||
|
|
|
|||
Loading…
Reference in a new issue