Fixing GEOB bug (#96)

Author: benrr101

src/id3v2/frames/attachmentFrame.ts

@@ -429,7 +429,7 @@ export default class AttachmentFrame extends Frame implements IPicture {
             // Filename               <text string according to encoding> $00 (00)
             // Content description    <text string according to encoding> $00 (00)
             // Encapsulated object    <binary data>
-            const mimeTypeEndIndex = data.find(ByteVector.getTextDelimiter(StringType.Latin1));
+            const mimeTypeEndIndex = data.offsetFind(ByteVector.getTextDelimiter(StringType.Latin1), 1);
             if (mimeTypeEndIndex === -1) {
                 return;
             }

test-unit/id3v2/attachmentsFrameTests.ts

@@ -240,7 +240,7 @@ const getCustomTestFrame = (
     }
 
     @test
-    public fromRawData_geob() {
+    public fromRawData_geob_nonLatinEncoding() {
         // Arrange
         const testData = ByteVector.fromString("fuxbuxqux", StringType.Latin1);
         const header = new Id3v2FrameHeader(FrameIdentifiers.GEOB);
@@ -273,6 +273,40 @@ const getCustomTestFrame = (
         );
     }
 
+    @test
+    public fromRawData_geob_latinEncoding() {
+        // Arrange
+        const testData = ByteVector.fromString("fuxbuxqux", StringType.Latin1);
+        const header = new Id3v2FrameHeader(FrameIdentifiers.GEOB);
+        header.frameSize = 60;
+        const data = ByteVector.concatenate(
+            header.render(4),
+            StringType.Latin1,
+            ByteVector.fromString("image/gif", StringType.Latin1),
+            ByteVector.getTextDelimiter(StringType.Latin1),
+            ByteVector.fromString("image.gif", StringType.Latin1),
+            ByteVector.getTextDelimiter(StringType.Latin1),
+            ByteVector.fromString("foobarbaz", StringType.Latin1),
+            ByteVector.getTextDelimiter(StringType.Latin1),
+            testData
+        );
+
+        // Act
+        const frame = AttachmentFrame.fromRawData(data, 4);
+
+        // Assert
+        Id3v2_AttachmentFrame_ConstructorTests.verifyFrame(
+            frame,
+            FrameIdentifiers.GEOB,
+            testData,
+            "foobarbaz",
+            "image.gif",
+            "image/gif",
+            StringType.Latin1,
+            PictureType.NotAPicture
+        );
+    }
+
     @test
     public fromOffsetRawData_apicV4() {
         // Arrange