Add a test for an unknown header + fix

rfd59 · Sep 11, 2022 · 6d83fee · 6d83fee
1 parent ad26ed4
commit 6d83fee
Show file tree

Hide file tree

Showing 4 changed files with 23 additions and 1 deletion.
diff --git a/OFX-Tool.Library/Core/Load.cs b/OFX-Tool.Library/Core/Load.cs
@@ -61,7 +61,7 @@ private HeaderDocument LoadHeader()
                         else if (setting[0] == Entity.GetHeader(nameof(HeaderDocument.NewFileUid)))
                             header.NewFileUid = setting[1];
                         else
-                            throw new InvalidOperationException($"Unexpected header! [{setting[0]}]");
+                            throw new InvalidOperationException($"Unexpected header! [{line}]");
                     }
                 }
             }

diff --git a/OFX-Tool.Test/Core/LoadTests.cs b/OFX-Tool.Test/Core/LoadTests.cs
@@ -50,5 +50,13 @@ public void LoadTest()
             Assert.AreEqual(ofx.Response, load.Ofx.Response);
             Assert.AreEqual(ofx, load.Ofx);
         }
+
+        [TestMethod]
+        [ExpectedException(typeof(OFXToolException))]
+        public void LoadTest_UnknownHeader()
+        {
+            var ofxFile = "TestFiles/LoadTest_UnknownHeader.ofx";
+            new Load(ofxFile);
+        }
     }
 }
diff --git a/OFX-Tool.Test/OFX-Tool.Test.csproj b/OFX-Tool.Test/OFX-Tool.Test.csproj
@@ -30,6 +30,9 @@
     <None Update="Ofx.xsd">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>
+    <None Update="TestFiles\LoadTest_UnknownHeader.ofx">
+      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
+    </None>
     <None Update="TestFiles\LoadTest1.ofx">
       <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>
     </None>

diff --git a/OFX-Tool.Test/TestFiles/LoadTest_UnknownHeader.ofx b/OFX-Tool.Test/TestFiles/LoadTest_UnknownHeader.ofx
@@ -0,0 +1,11 @@
+
+OFXHEADER:000
+DATA:AZERTY
+VERSION:111
+SECURITY:SECU
+No correct line ....
+ENCODING:ENCO
+CHARSET:1234
+COMPRESSION:COMP
+OLDFILEUID:OLD
+NEWFILEUID:NEW