Comment fixes

Author: teo-nikolov

src/Bellatrix.Desktop/DesktopPluginsConfiguration.cs

@@ -150,6 +150,7 @@ public static void ConfigureLLM()
     {
         if (ConfigurationService.GetSection<LargeLanguageModelsSettings>() == null)
         {
+            Logger.LogError("Could not load LargeLanguageModelsSettings section from testFrameworkSettings.json");
             return;
         }
 
@@ -174,4 +175,4 @@ public static void ConfigureLLM()
             Logger.LogError(ex.ToString());
         }
     }
-}
+}

src/Bellatrix.Desktop/components/Window.cs

@@ -85,25 +85,4 @@ public virtual void Attach()
 
         Attached?.Invoke(this, new ComponentActionEventArgs(this));
     }
-
-    // public virtual void Detach()
-    // {
-    //     Attaching?.Invoke(this, new ComponentActionEventArgs(this));
-    //
-    //     var currentAppConfiguration = ServicesCollection.Current.Resolve<AppInitializationInfo>("_currentAppConfiguration");
-    //
-    //     if (currentAppConfiguration.WindowHandle != WindowHandle)
-    //     {
-    //         throw new InvalidOperationException($"This window ({WindowHandle}) is not currently attached. Currently attached window: {currentAppConfiguration.WindowHandle}");
-    //     }
-    //
-    //     currentAppConfiguration.WindowHandle = null;
-    //     var driver = WrappedWebDriverCreateService.Create(currentAppConfiguration, ServicesCollection.Current);
-    //
-    //     WrappedDriver.Quit();
-    //     ServicesCollection.Current.UnregisterSingleInstance<WindowsDriver>();
-    //     ServicesCollection.Current.RegisterInstance(driver);
-    //
-    //     Attached?.Invoke(this, new ComponentActionEventArgs(this));
-    // }
 }

src/Bellatrix.Desktop/infrastructure/App.cs

@@ -36,13 +36,8 @@ namespace Bellatrix.Desktop;
 
 public class App : IDisposable
 {
-    // TODO: Change to be ThreadLocal.
-    private static bool _shouldStartLocalService;
-    // private static Process _appiumProcess;
-
     public App()
     {
-        _shouldStartLocalService = ConfigurationService.GetSection<DesktopSettings>().ExecutionSettings.ShouldStartLocalService;
         ServicesCollection.Main.RegisterInstance<IViewSnapshotProvider>(AppService);
     }
     private static readonly bool ShouldStartLocalService = ConfigurationService.GetSection<DesktopSettings>().ExecutionSettings.ShouldStartLocalService;

src/Bellatrix.Mobile/AndroidPluginsConfiguration.cs

@@ -159,6 +159,7 @@ public static void ConfigureLLM()
     {
         if (ConfigurationService.GetSection<LargeLanguageModelsSettings>() == null)
         {
+            Logger.LogError("Could not load LargeLanguageModelsSettings section from testFrameworkSettings.json");
             return;
         }
 
@@ -183,4 +184,4 @@ public static void ConfigureLLM()
             Logger.LogError(ex.ToString());
         }
     }
-}
+}

src/Bellatrix.Mobile/IOSPluginsConfiguration.cs

@@ -155,6 +155,7 @@ public static void ConfigureLLM()
     {
         if (ConfigurationService.GetSection<LargeLanguageModelsSettings>() == null)
         {
+            Logger.LogError("Could not load LargeLanguageModelsSettings section from testFrameworkSettings.json");
             return;
         }
 
@@ -179,4 +180,4 @@ public static void ConfigureLLM()
             Logger.LogError(ex.ToString());
         }
     }
-}
+}

src/Bellatrix.Playwright/WebPluginsConfiguration.cs

@@ -237,6 +237,7 @@ public static void ConfigureLLM()
     {
         if (ConfigurationService.GetSection<LargeLanguageModelsSettings>() == null)
         {
+            Logger.LogError("Could not load LargeLanguageModelsSettings section from testFrameworkSettings.json");
             return;
         }
 

src/Bellatrix.Web/WebPluginsConfiguration.cs

@@ -234,6 +234,7 @@ public static void ConfigureLLM()
     {
         if (ConfigurationService.GetSection<LargeLanguageModelsSettings>() == null)
         {
+            Logger.LogError("Could not load LargeLanguageModelsSettings section from testFrameworkSettings.json");
             return;
         }