From f74899ab9af4427fc87e6535fffeb0c85329ad65 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?J=C3=A9r=C3=A9mie=20Detrey?= Date: Mon, 30 Apr 2018 18:26:06 +0200 Subject: [PATCH] Fix errors due to missing track['id'] in unit tests. --- test/test_importer.py | 1 + test/test_mb.py | 15 +++++++++++---- 2 files changed, 12 insertions(+), 4 deletions(-) diff --git a/test/test_importer.py b/test/test_importer.py index e30f5609c..6721f0dc2 100644 --- a/test/test_importer.py +++ b/test/test_importer.py @@ -1819,6 +1819,7 @@ def mocked_get_release_by_id(id_, includes=[], release_status=[], 'id': id_, 'medium-list': [{ 'track-list': [{ + 'id': 'baz', 'recording': { 'title': 'foo', 'id': 'bar', diff --git a/test/test_mb.py b/test/test_mb.py index 55df22944..b61ebe59a 100644 --- a/test/test_mb.py +++ b/test/test_mb.py @@ -63,12 +63,15 @@ class MBAlbumInfoTest(_common.TestCase): 'country': 'COUNTRY', 'status': 'STATUS', } + i = 0 track_list = [] if tracks: - for i, recording in enumerate(tracks): + for recording in tracks: + i += 1 track = { + 'id': 'RELEASE TRACK ID %d' % i, 'recording': recording, - 'position': i + 1, + 'position': i, 'number': 'A1', } if track_length: @@ -90,10 +93,12 @@ class MBAlbumInfoTest(_common.TestCase): track_list.append(track) data_track_list = [] if data_tracks: - for i, recording in enumerate(data_tracks): + for recording in data_tracks: + i += 1 data_track = { + 'id': 'RELEASE TRACK ID %d' % i, 'recording': recording, - 'position': len(track_list) + i + 1, + 'position': i, 'number': 'A1', } data_track_list.append(data_track) @@ -194,6 +199,7 @@ class MBAlbumInfoTest(_common.TestCase): self._make_track('TITLE TWO', 'ID TWO', 200.0 * 1000.0)] release = self._make_release(tracks=[tracks[0]]) second_track_list = [{ + 'id': 'RELEASE TRACK ID 2', 'recording': tracks[1], 'position': '1', 'number': 'A1', @@ -551,6 +557,7 @@ class MBLibraryTest(unittest.TestCase): 'id': mbid, 'medium-list': [{ 'track-list': [{ + 'id': 'baz', 'recording': { 'title': 'foo', 'id': 'bar',