PICARD-1839: Support tagging DSDIFF (DFF) files with ID3

picard/formats/__init__.py

@@ -39,6 +39,7 @@ from picard.formats.apev2 import (
 from picard.formats.asf import ASFFile
 from picard.formats.id3 import (
     AiffFile,
+    DSDIFFFile,
     DSFFile,
     MP3File,
     TrueAudioFile,
@@ -71,6 +72,8 @@ register_format(AACFile)
 register_format(AC3File)
 register_format(AiffFile)
 register_format(ASFFile)
+if DSDIFFFile:
+    register_format(DSDIFFFile)
 register_format(DSFFile)
 register_format(FLACFile)
 register_format(MIDIFile)

picard/formats/id3.py

@@ -736,3 +736,17 @@ class AiffFile(NonCompatID3File):
     EXTENSIONS = [".aiff", ".aif", ".aifc"]
     NAME = "Audio Interchange File Format (AIFF)"
     _File = mutagen.aiff.AIFF
+
+
+try:
+    import mutagen.dsdiff
+
+    class DSDIFFFile(NonCompatID3File):
+
+        """DSF file."""
+        EXTENSIONS = [".dff"]
+        NAME = "DSDIFF"
+        _File = mutagen.dsdiff.DSDIFF
+
+except ImportError:
+    DSDIFFFile = None

test/formats/test_id3.py

@@ -510,6 +510,31 @@ class DSFTest(CommonId3Tests.Id3TestCase):
     unexpected_info = ['~video']
 
 
+if id3.DSDIFFFile:
+    class DSDIFFTest(CommonId3Tests.Id3TestCase):
+        testfile = 'test-dsd.dff'
+        supports_ratings = True
+        expected_info = {
+            'length': 10,
+            '~channels': '2',
+            '~sample_rate': '5644800',
+            '~bitrate': '11289.6',
+            '~bits_per_sample': '1',
+        }
+        unexpected_info = ['~video']
+
+    class DSDIFFDSTTest(CommonId3Tests.Id3TestCase):
+        testfile = 'test-dst.dff'
+        supports_ratings = True
+        expected_info = {
+            'length': 0,
+            '~channels': '2',
+            '~sample_rate': '5644800',
+            '~bits_per_sample': '1',
+        }
+        unexpected_info = ['~video']
+
+
 class AIFFTest(CommonId3Tests.Id3TestCase):
     testfile = 'test.aiff'
     supports_ratings = False