fix matching of filemimetype

Author: NeoCoderMatrix86

AudioCuesheetEditor.Tests/Services/IO/FileInputManagerTests.cs

@@ -0,0 +1,105 @@
+//This file is part of AudioCuesheetEditor.
+
+//AudioCuesheetEditor is free software: you can redistribute it and/or modify
+//it under the terms of the GNU General Public License as published by
+//the Free Software Foundation, either version 3 of the License, or
+//(at your option) any later version.
+
+//AudioCuesheetEditor is distributed in the hope that it will be useful,
+//but WITHOUT ANY WARRANTY; without even the implied warranty of
+//MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+//GNU General Public License for more details.
+
+//You should have received a copy of the GNU General Public License
+//along with Foobar.  If not, see
+//<http: //www.gnu.org/licenses />.
+using AudioCuesheetEditor.Services.IO;
+using Microsoft.AspNetCore.Components.Forms;
+using Microsoft.Extensions.Logging;
+using Microsoft.JSInterop;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+using Moq;
+using System.Net.Http;
+
+namespace AudioCuesheetEditor.Tests.Services.IO
+{
+    [TestClass()]
+    public class FileInputManagerTests
+    {
+        [TestMethod()]
+        public void CheckFileMimeType_ReturnsTrue_WhenContentTypeDoesNotMatchButExtension()
+        {
+            // Arrange
+            var jsRuntimeMock = new Mock<IJSRuntime>();
+            var httpClientMock = new Mock<HttpClient>();
+            var loggerMock = new Mock<ILogger<FileInputManager>>();
+            var file = CreateBrowserFile("test.mp3", "audio/wav");
+            var manager = new FileInputManager(jsRuntimeMock.Object, httpClientMock.Object, loggerMock.Object);
+
+            // Act
+            var result = manager.CheckFileMimeType(file, "audio/mpeg", ".mp3");
+
+            // Assert
+            Assert.IsTrue(result);
+        }
+
+        [TestMethod()]
+        public void CheckFileMimeType_ReturnsTrue_WhenContentTypeDoesMatchButNotExtension()
+        {
+            // Arrange
+            var jsRuntimeMock = new Mock<IJSRuntime>();
+            var httpClientMock = new Mock<HttpClient>();
+            var loggerMock = new Mock<ILogger<FileInputManager>>();
+            var file = CreateBrowserFile("test.mpeg", "audio/mpeg");
+            var manager = new FileInputManager(jsRuntimeMock.Object, httpClientMock.Object, loggerMock.Object);
+
+            // Act
+            var result = manager.CheckFileMimeType(file, "audio/mpeg", ".mp3");
+
+            // Assert
+            Assert.IsTrue(result);
+        }
+
+        [TestMethod()]
+        public void CheckFileMimeType_ReturnsFalse_WhenExtensionDoesNotMatchAndContentTypeIsEmpty()
+        {
+            // Arrange
+            var jsRuntimeMock = new Mock<IJSRuntime>();
+            var httpClientMock = new Mock<HttpClient>();
+            var loggerMock = new Mock<ILogger<FileInputManager>>();
+            var file = CreateBrowserFile("test.flac", string.Empty);
+            var manager = new FileInputManager(jsRuntimeMock.Object, httpClientMock.Object, loggerMock.Object);
+
+            // Act
+            var result = manager.CheckFileMimeType(file, "audio/flac", ".mp3");
+
+            // Assert
+            Assert.IsFalse(result);
+        }
+
+        [TestMethod()]
+        public void CheckFileMimeType_ReturnsTrue_WhenContentTypeAndExtensionMatch()
+        {
+            // Arrange
+            var jsRuntimeMock = new Mock<IJSRuntime>();
+            var httpClientMock = new Mock<HttpClient>();
+            var loggerMock = new Mock<ILogger<FileInputManager>>();
+            var file = CreateBrowserFile("test.wav", "audio/wave");
+            var manager = new FileInputManager(jsRuntimeMock.Object, httpClientMock.Object, loggerMock.Object);
+
+            // Act
+            var result = manager.CheckFileMimeType(file, "audio/wave", ".wav");
+
+            // Assert
+            Assert.IsTrue(result);
+        }
+
+        private static IBrowserFile CreateBrowserFile(string name, string contentType)
+        {
+            var fileMock = new Mock<IBrowserFile>();
+            fileMock.Setup(f => f.Name).Returns(name);
+            fileMock.Setup(f => f.ContentType).Returns(contentType);
+            return fileMock.Object;
+        }
+    }
+}

AudioCuesheetEditor/Model/IO/FileMimeTypes.cs

@@ -18,8 +18,8 @@ namespace AudioCuesheetEditor.Model.IO
     public static class FileMimeTypes
     {
         public const string Text = "text/plain";
-        public const string Projectfile = "text/*";
-        public const string Cuesheet = "text/*";
+        public const string Projectfile = "application/x-ace";
+        public const string Cuesheet = "application/x-cue";
         public const string CDTextfile = "text/*";
     }
 }

AudioCuesheetEditor/Services/IO/FileInputManager.cs

@@ -56,7 +56,7 @@ public Boolean CheckFileMimeType(IBrowserFile file, String mimeType, String file
                 {
                     fileMimeTypeMatches = file.ContentType.Equals(mimeType, StringComparison.CurrentCultureIgnoreCase);
                 }
-                else
+                if (fileMimeTypeMatches == false)
                 {
                     //Try to find by file extension
                     var extension = Path.GetExtension(file.Name);