@

Phase 2: Launcher-UI + eigenstaendige Modul-Fenster

- Form1 -> LauncherForm (Dateien via git mv, Designer/resx angepasst)
- Modul-Tab: Karten aus ModuleRegistry mit "Fenster oeffnen"-Button je Modul
- UI/WindowManager: Fenster-Tracking (Key->Form), Re-Open fokussiert, CloseAll
- UI/ModuleFormBase: Basisklasse fuer eigenstaendige Modul-Fenster
- CongressTradingForm: DB-Kennzahlen + manueller Scrape-Trigger (Phase-4-Ausbau folgt)
- WindowManager in DI; Launcher schliesst Modul-Fenster beim Beenden
- Tests: WindowManager (6) -> 21/21 gruen; Launcher-Start verifiziert

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@

IBKRTrader.Tests/Modules/CongressTradingModuleTests.cs

@@ -1,8 +1,13 @@
 using FluentAssertions;
+using IBKRTrader.Core.Database;
+using IBKRTrader.Core.Logging;
+using IBKRTrader.Core.Settings;
+using IBKRTrader.Core.Workers;
 using IBKRTrader.Modules.CongressTrading;
 using IBKRTrader.Modules.CongressTrading.Database;
 using IBKRTrader.Modules.CongressTrading.Scraper;
 using IBKRTrader.Modules.CongressTrading.Workers;
+using IBKRTrader.UI;
 using Microsoft.Extensions.DependencyInjection;
 
 namespace IBKRTrader.Tests.Modules;
@@ -39,11 +44,22 @@ public class CongressTradingModuleTests
     }
 
     [Fact]
-    public void CreateWindow_ReturnsForm()
+    public void CreateWindow_ReturnsModuleForm()
     {
-        using var form = new CongressTradingModule().CreateWindow(new ServiceCollection().BuildServiceProvider());
+        // Minimaler DI-Container mit allen von CreateWindow benötigten Abhängigkeiten.
+        var services = new ServiceCollection();
+        services.AddSingleton<SettingsService>();
+        services.AddSingleton<LoggingService>();
+        services.AddSingleton<DatabaseService>();
+        services.AddSingleton<IEnumerable<IWorker>>(_ => Array.Empty<IWorker>());
+        services.AddSingleton<WorkerEngine>();
+        var module = new CongressTradingModule();
+        module.RegisterServices(services);
 
-        form.Should().NotBeNull();
-        form.Text.Should().Contain("CT");
+        using var provider = services.BuildServiceProvider();
+        using var form = module.CreateWindow(provider);
+
+        form.Should().BeAssignableTo<ModuleFormBase>();
+        ((ModuleFormBase)form).ModuleKey.Should().Be("CT");
     }
 }

IBKRTrader.Tests/UI/WindowManagerTests.cs

@@ -0,0 +1,100 @@
+using FluentAssertions;
+using IBKRTrader.UI;
+
+namespace IBKRTrader.Tests.UI;
+
+[Trait("cat", "unit")]
+public class WindowManagerTests
+{
+    /// <summary>
+    /// Testbare Variante: unterdrückt echte Show-/Focus-Aufrufe (kein Message-Loop),
+    /// zählt aber die Focus-Aufrufe.
+    /// </summary>
+    private sealed class TestWindowManager : WindowManager
+    {
+        public int FocusCount;
+        protected override void Present(Form form) { /* kein Show im Test */ }
+        protected override void Focus(Form form) => FocusCount++;
+    }
+
+    [Fact]
+    public void OpenOrFocus_NewKey_InvokesFactory_AndTracks()
+    {
+        var wm = new TestWindowManager();
+        var created = 0;
+
+        using var form = wm.OpenOrFocus("CT", () => { created++; return new Form(); });
+
+        created.Should().Be(1);
+        wm.IsOpen("CT").Should().BeTrue();
+        wm.OpenCount.Should().Be(1);
+    }
+
+    [Fact]
+    public void OpenOrFocus_SameKey_FocusesExisting_DoesNotRecreate()
+    {
+        var wm = new TestWindowManager();
+        var created = 0;
+
+        using var first  = wm.OpenOrFocus("CT", () => { created++; return new Form(); });
+        var       second = wm.OpenOrFocus("CT", () => { created++; return new Form(); });
+
+        created.Should().Be(1);
+        wm.FocusCount.Should().Be(1);
+        second.Should().BeSameAs(first);
+        wm.OpenCount.Should().Be(1);
+    }
+
+    [Fact]
+    public void OpenOrFocus_DifferentKeys_TracksSeparately()
+    {
+        var wm = new TestWindowManager();
+
+        using var a = wm.OpenOrFocus("A", () => new Form());
+        using var b = wm.OpenOrFocus("B", () => new Form());
+
+        wm.OpenCount.Should().Be(2);
+        wm.IsOpen("A").Should().BeTrue();
+        wm.IsOpen("B").Should().BeTrue();
+    }
+
+    [Fact]
+    public void OpenOrFocus_AfterDisposed_RecreatesWindow()
+    {
+        var wm = new TestWindowManager();
+        var created = 0;
+
+        var first = wm.OpenOrFocus("CT", () => { created++; return new Form(); });
+        first.Dispose();
+
+        wm.IsOpen("CT").Should().BeFalse();
+
+        using var second = wm.OpenOrFocus("CT", () => { created++; return new Form(); });
+
+        created.Should().Be(2);
+        second.Should().NotBeSameAs(first);
+    }
+
+    [Fact]
+    public void CloseAll_RemovesAllWindows()
+    {
+        var wm = new TestWindowManager();
+        wm.OpenOrFocus("A", () => new Form());
+        wm.OpenOrFocus("B", () => new Form());
+
+        wm.CloseAll();
+
+        wm.OpenCount.Should().Be(0);
+        wm.IsOpen("A").Should().BeFalse();
+    }
+
+    [Fact]
+    public void OpenOrFocus_BlankKey_Throws()
+    {
+        var wm = new TestWindowManager();
+
+        var act = () => wm.OpenOrFocus("   ", () => new Form());
+
+        act.Should().Throw<ArgumentException>();
+    }
+}