Fix tests

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "discogs-tag"
-version = "1.3.0"
+version = "1.3.1"
 description = "An audio tagger based on Discogs metadata."
 authors = ["infojunkie <[email protected]>"]
 repository = "https://github.com/infojunkie/discogs-tag.py"

src/discogs_tag/cli.py

@@ -344,9 +344,9 @@ def list_files(dir):
   ]))
 
 def parse_options(options):
+  for skip in SKIP_KEYS:
+    options['skip_' + skip.lower()] = False
   if 'skip' in options and options['skip'] is not None:
-    for skip in SKIP_KEYS:
-      options['skip_' + skip.lower()] = False
     if isinstance(options['skip'], str):
       options['skip'] = [options['skip']]
     for skip in options['skip']:

tests/test_discogs_tag.py

@@ -164,6 +164,81 @@ def test_apply_metadata_track():
   assert audio['title'] == 'Title: Sub track 1 / Sub track 2'
   assert audio['tracknumber'] == '2'
 
+def test_skip_only_options():
+  audio = apply_metadata_track({
+    'year': 2002,
+    'artists': [{
+      'anv': 'Artist 1'
+    }, {
+      'name': 'Artist 2'
+    }, {
+      'anv': '',
+      'name': 'Artist 3 (56)'
+    }],
+  }, {
+    'title': 'Title',
+    'position': '',
+    'sub_tracks': [{
+      'type_': 'track',
+      "title": "Sub track 1",
+    }, {
+      'type_': 'track',
+      'title': 'Sub track 2',
+    }, {
+      'type_': 'data',
+      'title': "Not a track",
+    }],
+    'extraartists': [{
+      'role': 'Guitar',
+      'name': 'Guitarist'
+    }, {
+      'role': 'Written-By',
+      'name': 'Composer 1'
+    }, {
+      'role': 'Composed By',
+      'name': 'Composer 2'
+    }]
+  }, { 'title': 'Some other title' }, 2, parse_options({ 'skip': ['title'] }))
+  assert audio['title'] == 'Some other title'
+  assert audio['tracknumber'] == '2'
+
+  audio = apply_metadata_track({
+    'year': 2002,
+    'artists': [{
+      'anv': 'Artist 1'
+    }, {
+      'name': 'Artist 2'
+    }, {
+      'anv': '',
+      'name': 'Artist 3 (56)'
+    }],
+  }, {
+    'title': 'Title',
+    'position': '',
+    'sub_tracks': [{
+      'type_': 'track',
+      "title": "Sub track 1",
+    }, {
+      'type_': 'track',
+      'title': 'Sub track 2',
+    }, {
+      'type_': 'data',
+      'title': "Not a track",
+    }],
+    'extraartists': [{
+      'role': 'Guitar',
+      'name': 'Guitarist'
+    }, {
+      'role': 'Written-By',
+      'name': 'Composer 1'
+    }, {
+      'role': 'Composed By',
+      'name': 'Composer 2'
+    }]
+  }, { 'title': 'Some other title' }, 2, parse_options({ 'only': ['title'] }))
+  assert audio['title'] == 'Title: Sub track 1 / Sub track 2'
+  assert 'tracknumber' not in audio
+
 def test_apply_metadata():
   with open('tests/18051880.json') as release:
     data = json.load(release)