menutoolbarcontroller

Author: BRUNER Patrick

src/LogExpert.Configuration/ConfigManager.cs

@@ -281,6 +281,22 @@ public void AddToFileHistory (string fileName)
         Save(SettingsFlags.FileHistory);
     }
 
+    [SupportedOSPlatform("windows")]
+    public void RemoveFromFileHistory (string fileName)
+    {
+        bool findName (string s) => s.ToUpperInvariant().Equals(fileName.ToUpperInvariant(), StringComparison.Ordinal);
+
+        var index = Instance.Settings.FileHistoryList.FindIndex(findName);
+
+        if (index != -1)
+        {
+            Instance.Settings.FileHistoryList.RemoveAt(index);
+        }
+
+        Save(SettingsFlags.FileHistory);
+    }
+
+
     public void ClearLastOpenFilesList ()
     {
         lock (_loadSaveLock)

src/LogExpert.Core/Interface/IConfigManager.cs

@@ -151,6 +151,12 @@ public interface IConfigManager
     /// This method is supported only on Windows platforms.</remarks>
     /// <param name="fileName">The name of the file to add to the file history list. Comparison is case-insensitive.</param>
     void AddToFileHistory (string fileName);
+    
+    /// <summary>
+    /// Removes the specified file name from the file history list.
+    /// </summary>
+    /// <param name="fileName">The name of the file to remove from the file history list. Comparison is case-insensitive.</param>
+    void RemoveFromFileHistory (string fileName);
 
     /// <summary>
     /// Clears the list of recently opened files.

src/LogExpert.Tests/Services/LedIndicatorServiceTests.cs

@@ -1,6 +1,7 @@
 using System.Runtime.Versioning;
 
-using LogExpert.UI.Services;
+using LogExpert.UI.Interface.Services;
+using LogExpert.UI.Services.LedService;
 
 using NUnit.Framework;
 
@@ -9,11 +10,12 @@ namespace LogExpert.Tests.Services;
 [TestFixture]
 [Apartment(ApartmentState.STA)] // Required for UI components
 [SupportedOSPlatform("windows")]
-public class LedIndicatorServiceTests
+public class LedIndicatorServiceTests : IDisposable
 {
     private LedIndicatorService? _service;
     private ApplicationContext? _appContext;
     private WindowsFormsSynchronizationContext? _syncContext;
+    private bool _disposed;
 
     [SetUp]
     public void Setup ()
@@ -140,9 +142,9 @@ public void StartStop_DoesNotThrowException ()
         _service!.Initialize(Color.Blue);
 
         // Act
-        _service.Start();
+        _service.StartService();
         Thread.Sleep(500); // Let timer tick a few times
-        _service.Stop();
+        _service.StopService();
 
         // Assert - no exception
         Assert.That(true, Is.True, "Service started and stopped without exceptions");
@@ -192,7 +194,7 @@ public void Dispose_DisposesAllResources ()
     {
         // Arrange
         _service!.Initialize(Color.Blue);
-        _service.Start();
+        _service.StartService();
 
         // Act
         _service.Dispose();
@@ -217,7 +219,7 @@ public void Start_WithoutInitialize_ThrowsException ()
         // Arrange - don't initialize
 
         // Act & Assert
-        _ = Assert.Throws<InvalidOperationException>(() => _service!.Start());
+        _ = Assert.Throws<InvalidOperationException>(() => _service!.StartService());
     }
 
     [Test]
@@ -316,4 +318,25 @@ public void GetIcon_WithSyncedState_ReturnsSyncedIcon ()
         // The icons should be different (synced has blue indicator on left side)
         Assert.That(iconSynced, Is.Not.EqualTo(iconNotSynced));
     }
+
+    public void Dispose ()
+    {
+        Dispose(true);
+        GC.SuppressFinalize(this);
+    }
+
+    protected virtual void Dispose (bool disposing)
+    {
+        if (_disposed)
+        {
+            return;
+        }
+
+        if (disposing)
+        {
+            _service?.Dispose();
+        }
+
+        _disposed = true;
+    }
 }