refactor(gui,api,logging): UI layout tweaks, cleanup, and improved diagnostics

- CbBxCamType now spans the full width of CbBxCameraModel
- BtnTest_Click moved to the bottom of AiQ_GUI.cs for easier testing/navigation
- Unused using directives removed across files
- Login credentials factored out and reused in MobileAPI.cs
- VLC.cs Capture function reviewed and superseded by TakeSnapshot
- InsertCamTab implementation updated to avoid use of `var`
- Misleading '// Non-ONVIF cameras' comment corrected
- Added Level.Debug logging, accessible only to developers, for detailed diagnostics

AiQ Tests/AiQLiteTests.cs

@@ -0,0 +1,13 @@
+using System;
+using System.Collections.Generic;
+using System.Text;
+
+namespace AiQ_GUI.AiQ_Tests
+{
+    internal class AiQLiteTests
+    {
+
+        // Tests to be written here later
+
+    }
+}

AiQ Tests/AiQTests.cs

@@ -0,0 +1,158 @@
+using AiQ_GUI;
+using AiQ_GUI.AiQ_Tests;
+public class AiQTests
+{
+    public static async Task AiQPreTest()
+    { 
+        if (!await CameraModules.SetZoomLockOn(MainForm.Instance.CamOnTest.IP))
+            Helper.RestartApp();
+
+        string LEDreply = await FlexiAPI.APIHTTPLED(MainForm.Instance.CamOnTest.IP, LEDPOWER.MID); // Set LED's to medium (0x30)
+        
+        if (!LEDreply.Contains("Power levels set successfully"))
+            MainForm.Instance.AddToActionsList($"LED level could not be set: {LEDreply}", Level.ERROR);
+
+        await CameraModules.FactoryResetModules(MainForm.Instance.CamOnTest.IP); // Reset both modules and double check
+
+        MainForm.Instance.sshData = SSH.CollectSSHData(MainForm.Instance.CamOnTest.IP); // SSH into camera to get Vaxtor packages, filesystem size and if tailscale is installed.
+        await SSH.CheckFSSize(MainForm.Instance.CamOnTest.IP, MainForm.Instance.LblFilesystemSize, MainForm.Instance.sshData); // Check Filesystem size is between 100GB & 150GB
+
+        Helper.DCPowerCheck(MainForm.Instance.LblDC); // If the camera is DC powered check it is within limits
+
+        // Requests, deserialises and checks the diagnostics API is correct
+        await TestingFunctions.CheckDiagsAPIPt1();
+
+        // Check module has gone to default config
+        CameraModules.CheckCamModule(TestingFunctions.DiagsAPI.IRmodule, MainForm.Instance.LblIRModule, MainForm.Instance.CamOnTest); // IR
+        CameraModules.CheckCamModule(TestingFunctions.DiagsAPI.OVmodule, MainForm.Instance.LblOVModule, MainForm.Instance.CamOnTest); // OV
+
+        // Check voltage and current are OK.
+        LED.CheckLEDs(TestingFunctions.DiagsAPI.LedVoltage, MainForm.Instance.LblLEDV, "V", CameraAccessInfo.LED_V);  // Voltage
+        LED.CheckLEDs(TestingFunctions.DiagsAPI.LedCurrent, MainForm.Instance.LblLEDI, "mA", CameraAccessInfo.LED_I); // Current
+
+        MainForm.Instance.Refresh(); // Make sure all labels are updated before checking them
+
+        // If there are any actions identified then fail the test. 
+        // If any labels are red then fail. Only labels in panel so can foreach on labels not controls
+        if (MainForm.Instance.RhTxBxActions.Text.Length < 2 && MainForm.Instance.PnlLbls.Controls.OfType<Label>().Any(c => c.ForeColor == Color.Red) == false)
+        {
+            // If camera already has a model or serial then ask if it is new
+            if (RegexCache.SerialRegex().IsMatch(TestingFunctions.DiagsAPI.serialNumber) || RegexCache.ModelRegex().IsMatch(TestingFunctions.DiagsAPI.modelNumber))
+            {
+                if (await MainForm.Instance.DisplayQuestion($"Would you like to allocate a serial number to this camera?"))
+                    await MainForm.Instance.AllocateSerial();
+                else if (GoogleAPI.UpdateSpreadSheetRePreTest(CameraAccessInfo.SpreadsheetID, MainForm.Instance.Vers) != "OK") // If rerun might be different values so update SS
+                    MainForm.Instance.AddToActionsList("Failed to write to spreadsheet, please check manually", Level.WARNING);
+                // else if (Excel.UpdateSpreadSheetPreTest(CameraAccessInfo.SpreadsheetID, Vers, CamOnTest.GetCamDesc(), CamOnTest.Model) != "OK")
+                //     AddToActionsList("Failed to write to spreadsheet, please check manually");
+            }
+            else // No serial or model so allocate one
+                await MainForm.Instance.AllocateSerial();
+
+            if (MainForm.Instance.RhTxBxActions.Text.Length < 2 && MainForm.Instance.PnlLbls.Controls.OfType<Label>().Any(c => c.ForeColor == Color.Red) == false)
+                await MainForm.Instance.PreTestPassed();
+        }
+        else
+        {
+            await MainForm.Instance.PreTestFailed("Diagnostic Failure");
+        }
+
+    }
+
+    public static async Task AiQFinalTest()
+    {
+        if (MainForm.Instance.LblTestTubePing.Text == "❌") // No test tube so test like an IQ
+        {
+            string LEDreply = await FlexiAPI.APIHTTPLED(MainForm.Instance.CamOnTest.IP, LEDPOWER.SAFE); // Set LED's to safe (0x0E) to help with eye safety and trim check.
+
+            if (!LEDreply.Contains("Power levels set successfully"))
+                MainForm.Instance.AddToActionsList($"LED level could not be set: {LEDreply}", Level.ERROR);
+        }
+        //else if (!await TestTube.CheckInTestTube(MainForm.Instance.CamOnTest.IP)) // Sets LED's to medium power after checking it is in the test tube
+        //    await MainForm.Instance.TestFailed(MainForm.Instance.BtnStartTest, "Camera not in test tube");
+        Task VisCheck = Helper.VisualCheck(MainForm.Instance.BtnStartTest);
+        if (!await CameraModules.ZoomModules("1F40", MainForm.Instance.CamOnTest.IP)) // Zoom to 8000 (1F40h) at the same time. 
+            await MainForm.Instance.TestFailed(MainForm.Instance.BtnStartTest, "Could not zoom modules to 8000");
+
+        if (!await CameraModules.SetZoomLockOn(MainForm.Instance.CamOnTest.IP))
+            Helper.RestartApp();
+
+        await Task.Delay(1000); // Without sleep it kept failing the factory reset as camera modules were not ready yet
+
+        await CameraModules.FactoryResetModules(MainForm.Instance.CamOnTest.IP); // Reset both modules and double check
+
+        string VISCAReply = await FlexiAPI.APIHTTPVISCA(MainForm.Instance.CamOnTest.IP, "8101043903FF", true); // Manual mode to be able to manipulate the SIG settings. 
+        if (VISCAReply != "9041FF9051FF")
+            MainForm.Instance.AddToActionsList("Couldn't set to manual mode", Level.ERROR);
+
+        await CameraModules.SetSIG(MainForm.Instance.CbBxShutter, MainForm.Instance.CbBxIris, MainForm.Instance.CbBxGain, MainForm.Instance.CamOnTest.IP); // Set SIG according to the drop downs in settings for a good picture ready for image check
+        await ImageProcessing.ImageCheck(MainForm.Instance.PicBxOV, MainForm.Instance.PicBxIRF2, MainForm.Instance.PicBxIRF16, MainForm.Instance.LblIRImageF2, MainForm.Instance.LblIRImageF16, MainForm.Instance.CamOnTest); // Populates the picture boxes and checks iris changes
+
+        await VisCheck; // Before changing UI elements wait for user to finish Visual check
+
+        MainForm.Instance.TabImagesandSettings.SelectedIndex = 2; // Swaps to the images tab
+        MainForm.Instance.Refresh(); // Show user things are happening by displaying images taken
+                                     // TODO - Force expire sighting.
+        Task Wait = Task.Delay(5000); // Wait for 5 seconds to allow the camera to zoom in, set settings and capture some plates before auto trim
+
+        if (CameraAccessInfo.HardwareExtras.Contains("GPS")) // Check GPS if the hardware has it
+            await FlexiAPI.GPSFix(MainForm.Instance.CamOnTest.IP);
+
+        // While waiting do the SSH tasks.
+        MainForm.Instance.sshData = SSH.CollectSSHData(MainForm.Instance.CamOnTest.IP); // SSH into camera to get Vaxtor packages, filesystem size and if tailscale is installed.
+        await SSH.CheckFSSize(MainForm.Instance.CamOnTest.IP, MainForm.Instance.LblFilesystemSize, MainForm.Instance.sshData); // Check Filesystem size is between 100GB & 150GB
+
+        Helper.DCPowerCheck(MainForm.Instance.LblDC); // If the camera is DC powered check it is within limits
+
+        if (CameraAccessInfo.HardwareExtras.Contains("4G")) // If it is a router camera then test the router.
+        {
+            MainForm.Instance.LblRouter.Visible = true;
+
+            if (Router.CheckRouter(Router.GetRouterInfo()))
+                MainForm.Instance.LblRouter.Text += "OK";
+            else
+                MainForm.Instance.LblRouter.Text += "Error with router";
+        }
+
+        await Wait; // Finished to 5s wait
+        await FlexiAPI.SetTrim(MainForm.Instance.CamOnTest.IP, MainForm.Instance.LblTestTubePing.Text); // Auto trims the cameras, some plates should have been captured in the meantime
+
+        if (!await CameraModules.ZoomModules("0000", MainForm.Instance.CamOnTest.IP)) // Zoom to full wide
+            await MainForm.Instance.TestFailed(MainForm.Instance.BtnStartTest, "Could not zoom modules to full wide");
+
+        await Task.Delay(1000); // Wait to be sure cameras are zoomed out. 
+
+        await CameraModules.FactoryResetModules(MainForm.Instance.CamOnTest.IP); // Reset both modules and double check
+
+        if (MainForm.Instance.LblTestTubePing.Text == "❌") // Set LED's to MID in prep for diagnostics API
+        {
+            string LEDreply = await FlexiAPI.APIHTTPLED(MainForm.Instance.CamOnTest.IP, LEDPOWER.MID); // Set LED's to medium (0x30)
+
+            if (!LEDreply.Contains("Power levels set successfully"))
+                MainForm.Instance.AddToActionsList($"LED level could not be set: {LEDreply}", Level.ERROR);
+        }
+        await FlexiAPI.SetVaxtorMinMaxPlate(MainForm.Instance.CamOnTest.IP);
+
+        DateTime PCTime = DateTime.Now; // Grab PC time as close to the API as possible to pass onto PDF later
+
+        await TestingFunctions.CheckDiagsAPIPt1(); // Requests, deserialises and checks the diagnostics API is correct
+        await TestingFunctions.CheckDiagsAPIPt2(); // For only final test parts
+
+        // Check module has gone to default config
+        CameraModules.CheckCamModule(TestingFunctions.DiagsAPI.IRmodule, MainForm.Instance.LblIRModule, MainForm.Instance.CamOnTest); // IR
+        CameraModules.CheckCamModule(TestingFunctions.DiagsAPI.OVmodule, MainForm.Instance.LblOVModule, MainForm.Instance.CamOnTest); // OV
+
+        // Check voltage and current are OK.
+        LED.CheckLEDs(TestingFunctions.DiagsAPI.LedVoltage, MainForm.Instance.LblLEDV, "V", CameraAccessInfo.LED_V);  // Voltage
+        LED.CheckLEDs(TestingFunctions.DiagsAPI.LedCurrent, MainForm.Instance.LblLEDI, "mA", CameraAccessInfo.LED_I); // Current
+
+        MainForm.Instance.Refresh(); // Make sure all labels are updated before checking them
+
+        // If there are any actions identified then fail the test. 
+        // If any labels are red then fail. Only labels in panel so can foreach on labels not controls
+        if (MainForm.Instance.RhTxBxActions.Text.Length > 2 || MainForm.Instance.PnlLbls.Controls.OfType<Label>().Any(c => c.ForeColor == Color.Red) == true)
+            await MainForm.Instance.TestFailed(MainForm.Instance.BtnStartTest, "Diagnostic Failure");// If approved then pass otherwise GUI would have restarted before getting to TestPassed.
+
+        await MainForm.Instance.TestPassed(PCTime);
+    } 
+}

AiQ Tests/Diagnostics.cs

@@ -0,0 +1,208 @@
+namespace AiQ_GUI.AiQ_Tests
+{
+
+    public class TestingFunctions
+    {
+        public static Diags DiagsAPI = new();
+
+        // Colours
+        public static readonly Color BtnColour = Color.FromArgb(70, 65, 80);
+        public static readonly Color TxBxColour = Color.FromArgb(53, 51, 64);
+
+        // ***** Testing functions *****
+
+
+        public static async Task CheckDiagsAPIPt1() // Parts done on pre and final test
+        {
+            DiagsAPI = await FlexiAPI.GetDiagnostics(MainForm.Instance.CamOnTest.IP); // Diagnostic API request
+
+            // Check Flexi Version
+            string flexiVerText = "Flexi Version = " + DiagsAPI.FlexiVersion;
+            bool flexiVerGreen = DiagsAPI.FlexiVersion == UniversalData.ExpFlexiVer;
+            if (!flexiVerGreen)
+                flexiVerText += " Expected = " + UniversalData.ExpFlexiVer;
+            MainForm.Instance.AddLabelToPanel(flexiVerText, !flexiVerGreen);
+
+            // Check Flexi Revision
+            string flexiRevText = "Flexi Revision = " + DiagsAPI.FlexiRevision;
+            bool flexiRevGreen = DiagsAPI.FlexiRevision == UniversalData.ExpFlexiRev;
+            if (!flexiRevGreen)
+                flexiRevText += " Expected = " + UniversalData.ExpFlexiRev;
+            MainForm.Instance.AddLabelToPanel(flexiRevText, !flexiRevGreen);
+
+            // Display MAC
+            string macText = "MAC = " + DiagsAPI.MAC;
+            bool macIsRed = false;
+            if (RegexCache.MACRegexNVIDIA().IsMatch(DiagsAPI.MAC)) // Checks it is in the right format and is a NVIDIA registered MAC address
+            {
+                // Valid NVIDIA MAC
+            }
+            else if (RegexCache.MACRegex().IsMatch(DiagsAPI.MAC)) // Is a valid MAC, but not NVIDIA
+            {
+                macIsRed = true;
+                MainForm.Instance.AddToActionsList($"{DiagsAPI.MAC} not recognised as NVIDIA MAC address", Level.ERROR);
+            }
+            else
+            {
+                macIsRed = true;
+                MainForm.Instance.AddToActionsList($"{DiagsAPI.MAC} not recognised as a MAC address", Level.ERROR);
+            }
+            MainForm.Instance.AddLabelToPanel(macText, macIsRed);
+
+            // Check timestamp
+            DateTime dateTime = new(1970, 1, 1, 0, 0, 0, 0, DateTimeKind.Utc);
+            dateTime = dateTime.AddSeconds(DiagsAPI.timeStamp).ToLocalTime();
+            long timediff = DateTimeOffset.UtcNow.ToUnixTimeSeconds() - DiagsAPI.timeStamp;
+
+            string timestampText = "Timestamp = " + dateTime;
+            bool timestampIsRed = false;
+            if (timediff > 10) // Over 10 seconds ago
+            {
+                timestampText += $" Time behind by {timediff}s";
+                timestampIsRed = true;
+            }
+            else if (timediff < 0) // Time is in the future.
+            {
+                timestampText += $" Time is in the future by {Math.Abs(timediff)}s";
+                timestampIsRed = true;
+            }
+            MainForm.Instance.AddLabelToPanel(timestampText, timestampIsRed);
+
+            // Check Temperature
+            string tempText = "Temperature = " + DiagsAPI.IntTemperature + "°C";
+            bool tempIsRed = DiagsAPI.IntTemperature <= 20 || DiagsAPI.IntTemperature >= 70;
+            MainForm.Instance.AddLabelToPanel(tempText, tempIsRed);
+
+            // Check Zoom Lock
+            string zoomLockText = "Zoom Lock = " + DiagsAPI.zoomLock;
+            bool zoomLockIsRed = false;
+            if (DiagsAPI.zoomLock == true)
+            {
+                if (DiagsAPI.IRmodule.zoom != DiagsAPI.OVmodule.zoom) // Checks if zoomlock is doing what is says it should
+                {
+                    zoomLockText += $" Zoomlock on but {DiagsAPI.IRmodule.zoom}≠{DiagsAPI.OVmodule.zoom}";
+                    zoomLockIsRed = true;
+                }
+            }
+            else
+                zoomLockIsRed = true;
+            MainForm.Instance.AddLabelToPanel(zoomLockText, zoomLockIsRed);
+        }
+
+        public static async Task CheckDiagsAPIPt2() // Parts only done on final test
+        {
+            if (MainForm.Instance.RhTxBxActions.Text.Contains("Error reading JSON")) // If failed to deserialise in part 1
+                return;
+
+            try // In case serial or model are blank
+            {
+                string serialText = "Serial Number = " + DiagsAPI.serialNumber;
+                bool serialIsRed = !RegexCache.SerialRegex().IsMatch(DiagsAPI.serialNumber);
+                MainForm.Instance.AddLabelToPanel(serialText, serialIsRed);
+
+                string modelText = "Model Number = " + DiagsAPI.modelNumber;
+                bool modelIsRed = !RegexCache.ModelRegex().IsMatch(DiagsAPI.modelNumber);
+                MainForm.Instance.AddLabelToPanel(modelText, modelIsRed);
+
+                string apiModel = DiagsAPI.modelNumber?.Trim();
+                string selectedModel = MainForm.Instance.CamOnTest.Model?.Trim();
+
+                if (string.IsNullOrWhiteSpace(selectedModel) && MainForm.Instance.CbBxCameraModel.SelectedItem != null)
+                    selectedModel = MainForm.Instance.CbBxCameraModel.SelectedItem.ToString().Split('-')[0].Trim();
+
+                if (RegexCache.SerialRegex().IsMatch(DiagsAPI.serialNumber) && RegexCache.ModelRegex().IsMatch(DiagsAPI.modelNumber))
+                {
+                    // If nothing selected yet, do not warn (matches old logic)
+                    if (!string.IsNullOrWhiteSpace(selectedModel)
+                        && !string.Equals(apiModel, selectedModel, StringComparison.OrdinalIgnoreCase))
+                    {
+                        MainForm.Instance.AddToActionsList(
+                            "Model number in camera doesn't match what has been selected",
+                            Level.WARNING);
+                    }
+                    else if (!string.IsNullOrWhiteSpace(CameraAccessInfo.SpreadsheetID)&& !GoogleAPI.CheckWIP(DiagsAPI.serialNumber, CameraAccessInfo.SpreadsheetID)) // Check WIP column in serial number register, if not ticked then RMA
+                    {
+                        MainForm.Instance.CamOnTest.RMANum = GoogleAPI.CheckRMANum(DiagsAPI.serialNumber, DiagsAPI.modelNumber); // Corrected by qualifying with the type name  
+
+                        if (MainForm.Instance.CamOnTest.RMANum == 0) // Couldn't find RMA num in spreadsheet
+                        {
+                            MainForm.Instance.CamOnTest.RMANum = Convert.ToInt32(await MainForm.Instance.DisplayInput("What is the RMA number?"));
+
+                            if (MainForm.Instance.CamOnTest.RMANum == -1) // Means they chose the 'I don't know' option
+                                await MainForm.Instance.TestFailed(MainForm.Instance.BtnStartTest, "Please get RMA number from operations team before continuing");
+                        }
+                        else if (!await MainForm.Instance.DisplayQuestion($"Is {MainForm.Instance.CamOnTest.RMANum} the RMA Number?")) // '!' because if its not the right RMA number let the user to it manually 
+                        {
+                            MainForm.Instance.CamOnTest.RMANum = Convert.ToInt32(await MainForm.Instance.DisplayInput("What is the RMA number?"));
+
+                            if (MainForm.Instance.CamOnTest.RMANum == -1) // Means they chose the 'I don't know' option
+                                await MainForm.Instance.TestFailed(MainForm.Instance.BtnStartTest, "Please get RMA number from operations team before continuing");
+                        }
+                    }
+                }
+                else
+                {
+                    MainForm.Instance.AddToActionsList(
+                        "Camera has not been given a valid serial and model number, suggest you run through pre test again and check serial number register for any issues.",
+                        Level.ERROR);
+                }
+            }
+            catch (Exception ex)
+            {
+                MainForm.Instance.AddToActionsList($"Error in CheckDiagsAPIPt2 serial/model: {ex.Message}", Level.ERROR);
+            }
+
+            // Check licenses
+            List<string> licensesOnCam = new List<string>(); // Temporary list for licenses on camera
+
+            if (DiagsAPI.licenses.saf1) licensesOnCam.Add("SAF1");
+            if (DiagsAPI.licenses.saf2) licensesOnCam.Add("SAF2");
+            if (DiagsAPI.licenses.saf3) licensesOnCam.Add("SAF3");
+            if (DiagsAPI.licenses.saf4) licensesOnCam.Add("SAF4");
+            if (DiagsAPI.licenses.audit) licensesOnCam.Add("Audit");
+            if (DiagsAPI.licenses.stream) licensesOnCam.Add("Stream");
+            if (MainForm.Instance.sshData != null && MainForm.Instance.sshData.tailscale) licensesOnCam.Add("Tailscale");
+
+            MainForm.Instance.AddLabelToPanel(
+                "Licenses = " + (licensesOnCam.Count == 0 ? "None" : string.Join(", ", licensesOnCam)),
+                false);
+
+            double CPUround = Math.Round(DiagsAPI.CPUusage); // Check CPU usage isn't near max
+            MainForm.Instance.AddLabelToPanel(
+                "CPU Usage = " + CPUround + "%",
+                CPUround <= 50 || CPUround >= 98);
+
+            // Check Vaxtor license
+            bool vaxtorIsRed = false;
+            string vaxtorText = "Vaxtor Key ID = " + DiagsAPI.licenses.raptorKeyID;
+
+            if (DiagsAPI.licenses.raptorKeyID != "Not Licensed") vaxtorIsRed = false;
+            else if (CameraAccessInfo.VaxtorLic == false) vaxtorIsRed = false;
+            else if (await MainForm.Instance.DisplayQuestion("Was this camera licensed manually?"))
+            {
+                string ProdcutKeyID = await MainForm.Instance.DisplayInput("What is the Key ID?", false);
+
+                if (RegexCache.VaxtorRegex().IsMatch(ProdcutKeyID))
+                {
+                    Access.Stats("Please Get A Valid Vaxtor Product Key Before Continuing", MainForm.Instance.CamOnTest.Model);
+                    await MainForm.Instance.TestFailed(MainForm.Instance.BtnStartTest, "Please get a valid Vaxtor Product Key before continuing");
+                }
+
+                DiagsAPI.licenses.raptorKeyID = MainForm.Instance.TxBxProductKey.Text;
+                vaxtorText = "Vaxtor Key ID = " + DiagsAPI.licenses.raptorKeyID;
+                vaxtorIsRed = false;
+            }
+            else vaxtorIsRed = true;
+
+            MainForm.Instance.AddLabelToPanel(vaxtorText, vaxtorIsRed);
+
+            // Check trim is within 10% both horizontally and vertically
+            const int HMax = 96; // 5% of 1920 each way = ±96
+            const int VMax = 54; // 5% of 1080 each way = ±54
+
+            MainForm.Instance.AddLabelToPanel(
+                "Trim = H: " + DiagsAPI.trim[0] + " V: " + DiagsAPI.trim[1],
+                Math.Abs(DiagsAPI.trim[0]) > HMax || Math.Abs(DiagsAPI.trim[1]) > VMax);
+        }
+    }
+}

AiQ_GUI_NET_Test.csproj

@@ -55,15 +55,22 @@
     <PackageReference Include="ClosedXML" Version="0.105.0" />
     <PackageReference Include="Emgu.CV" Version="4.12.0.5764" />
     <PackageReference Include="Emgu.CV.runtime.windows" Version="4.12.0.5764" />
+    <PackageReference Include="FlaUI.Core" Version="5.0.0" />
+    <PackageReference Include="FlaUI.UIA3" Version="5.0.0" />
     <PackageReference Include="Google.Apis.Auth" Version="1.73.0" />
     <PackageReference Include="Google.Apis.Sheets.v4" Version="1.72.0.3966" />
+    <PackageReference Include="LibVLCSharp" Version="3.9.5" />
+    <PackageReference Include="LibVLCSharp.WinForms" Version="3.9.5" />
     <PackageReference Include="Newtonsoft.Json" Version="13.0.4" />
+    <PackageReference Include="Newtonsoft.Json.Bson" Version="1.0.3" />
+    <PackageReference Include="OnvifDiscovery" Version="2.1.1" />
     <PackageReference Include="PDFsharp-MigraDoc-gdi" Version="6.2.3" />
     <PackageReference Include="Selenium.Support" Version="4.38.0" />
     <PackageReference Include="Selenium.WebDriver" Version="4.38.0" />
     <PackageReference Include="Selenium.WebDriver.ChromeDriver" Version="142.0.7444.17500" />
     <PackageReference Include="SSH.NET" Version="2025.1.0" />
     <PackageReference Include="System.Data.OleDb" Version="10.0.0" />
+    <PackageReference Include="VideoLAN.LibVLC.Windows" Version="3.0.21" />
   </ItemGroup>
 
   <ItemGroup>

Camera/CameraModules.cs

@@ -6,9 +6,9 @@ namespace AiQ_GUI
         // Chack camera modules are in default state according to what the diagnostics API. 
         public static void CheckCamModule(Module CamMod, Label Lbl, Camera CamOnTest)
         {
-            if (CamMod == null || Lbl == null)
+            if (CamMod == null)
             {
-                MainForm.Instance.AddToActionsList("Camera module or label was null in CheckCamModule.");
+                MainForm.Instance.AddToActionsList("Camera module was null in CheckCamModule.", Level.ERROR);
                 return;
             }
 
@@ -25,7 +25,7 @@ namespace AiQ_GUI
             }
             catch
             {
-                MainForm.Instance.AddToActionsList($"{CamMod.firmwareVer} or {UniversalData.WonwooFirmware} could not be converted to a double");
+                MainForm.Instance.AddToActionsList($"{CamMod.firmwareVer} or {UniversalData.WonwooFirmware} could not be converted to a double", Level.ERROR);
             }
 
             if (CamOnTest.RMANum > 0 && !LessTanOrEqualTo)
@@ -37,25 +37,44 @@ namespace AiQ_GUI
             if (CamMod.expMode != 0) // Auto 0=0x00
                 errMssg += $"Exp mode not set: {CamMod.expMode} ";
 
+            // Determine display result
+            string displayResult = string.IsNullOrWhiteSpace(errMssg) ? "OK" : errMssg;
+            bool isError = !string.IsNullOrWhiteSpace(errMssg);
+
+            // Create label dynamically if not provided
+            if (Lbl == null)
+            {
+                string moduleName;
+                try
+                {
+                    if (object.ReferenceEquals(CamMod, AiQ_GUI.AiQ_Tests.TestingFunctions.DiagsAPI.IRmodule))
+                        moduleName = "IR Module";
+                    else if (object.ReferenceEquals(CamMod, AiQ_GUI.AiQ_Tests.TestingFunctions.DiagsAPI.OVmodule))
+                        moduleName = "OV Module";
+                    else
+                        moduleName = CamMod.firmwareVer == null ? "OV Module" : (CamMod.zoom < 5000 ? "IR Module" : "OV Module");
+                }
+                catch
+                {
+                    moduleName = CamMod.firmwareVer == null ? "OV Module" : (CamMod.zoom < 5000 ? "IR Module" : "OV Module");
+                }
+
+                MainForm.Instance.AddLabelToPanel($"{moduleName} = {displayResult}", isError);
+                return;
+            }
+
+            // Update existing label
             try
             {
                 Lbl.Invoke(() =>
                 {
-                    if (string.IsNullOrWhiteSpace(errMssg))
+                    Lbl.Text = Lbl.Text.TrimEnd('=', ' ') + " = " + displayResult;
-                    {
+                    Lbl.ForeColor = isError ? Color.Red : Color.LimeGreen;
-                        Lbl.Text += "OK";
-                        Lbl.ForeColor = Color.LightGreen;
-                    }
-                    else
-                    {
-                        Lbl.Text += errMssg;
-                        Lbl.ForeColor = Color.Red;
-                    }
                 });
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Exception in CheckCamModule: " + ex.Message);
+                MainForm.Instance.AddToActionsList("Exception in CheckCamModule: " + ex.Message, Level.ERROR);
             }
         }
 
@@ -65,7 +84,7 @@ namespace AiQ_GUI
         {
             if (Shutter.SelectedIndex == -1 || Iris.SelectedIndex == -1 || Gain.SelectedIndex == -1)
             {
-                MainForm.Instance.AddToActionsList("Shutter, Iris and Gain need selecting in images tab.");
+                MainForm.Instance.AddToActionsList("Shutter, Iris and Gain need selecting in images tab.", Level.WARNING);
                 return;
             }
 
@@ -75,7 +94,7 @@ namespace AiQ_GUI
 
             if (ShutterVISCA.Contains("ERROR") || IrisVISCA.Contains("ERROR") || GainVISCA.Contains("ERROR"))
             {
-                MainForm.Instance.AddToActionsList("Problem with selected SIG values");
+                MainForm.Instance.AddToActionsList("Problem with selected SIG values", Level.ERROR);
                 return;
             }
 
@@ -85,7 +104,7 @@ namespace AiQ_GUI
             string OneshotReply = await FlexiAPI.APIHTTPVISCA(IPAddress, "8101041801FF", true); // Oneshot auto focus
 
             if (!ShutterReply.Contains("41") || !IrisReply.Contains("41") || !GainReply.Contains("41") || !OneshotReply.Contains("41"))
-                MainForm.Instance.AddToActionsList("Could not set Shutter, Iris, Gain correctly" + Environment.NewLine + "Shutter: " + ShutterReply + Environment.NewLine + "Iris: " + IrisReply + Environment.NewLine + "Gain: " + GainReply + Environment.NewLine + "Oneshot: " + OneshotReply);
+                MainForm.Instance.AddToActionsList("Could not set Shutter, Iris, Gain correctly" + Environment.NewLine + "Shutter: " + ShutterReply + Environment.NewLine + "Iris: " + IrisReply + Environment.NewLine + "Gain: " + GainReply + Environment.NewLine + "Oneshot: " + OneshotReply, Level.ERROR);
         }
 
         // Sets back to the latest factory defaults CSV that is in Flexi. 
@@ -97,7 +116,7 @@ namespace AiQ_GUI
             await Task.WhenAll(IRReply, OVReply);
 
             if (IRReply.Result != "Factory reset OK." || OVReply.Result != "Factory reset OK.")
-                MainForm.Instance.AddToActionsList($"Could not reset camera modules to factory default.{Environment.NewLine}{IRReply}{Environment.NewLine}{OVReply}");
+                MainForm.Instance.AddToActionsList($"Could not reset camera modules to factory default.{Environment.NewLine}{IRReply}{Environment.NewLine}{OVReply}", Level.ERROR);
         }
 
         public static string BuildVISCACommand(string command, int hexValue)

Camera/FlexiAPI.cs

@@ -112,10 +112,11 @@ namespace AiQ_GUI
             catch (HttpRequestException ex)
             {
                 return $"HTTP error uploading to {url}: {ex.Message}";
+                 
             }
             catch (Exception ex)
             {
-                return $"Unexpected error uploading to {url}: {ex.Message} {(ex.InnerException?.Message ?? "")}";
+                return $"Unexpected error uploading to {url}: {ex.Message} {(ex.InnerException?.Message ?? string.Empty)}";
             }
         }
 
@@ -145,7 +146,7 @@ namespace AiQ_GUI
 
             if (JSON == null || JSON.Contains("Error") || JSON.Contains("Timeout"))
             {
-                MainForm.Instance.AddToActionsList("Error talking to Flexi, are you sure this is an AiQ?" + Environment.NewLine + JSON);
+                MainForm.Instance.AddToActionsList($"Error talking to Flexi, are you sure this is an AiQ?" + Environment.NewLine + JSON, Level.WARNING);
                 return null;
             }
 
@@ -173,7 +174,7 @@ namespace AiQ_GUI
                 // Treat "operation was canceled" as a successful apply
                 if (response.Contains("The operation was canceled", StringComparison.OrdinalIgnoreCase))
                 {
-                    Logging.LogMessage("SetVaxtorMinMaxPlate: Camera applied config but closed connection early (safe to ignore).");
+                    Logging.LogMessage($"SetVaxtorMinMaxPlate: Camera applied config but closed connection early (safe to ignore).");
                     return true;
                 }
 
@@ -187,7 +188,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Could not set Vaxtor Plate height and min confidence: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Could not set Vaxtor Plate height and min confidence: {ex.Message}", Level.ERROR);
                 return false;
             }
         }
@@ -216,7 +217,7 @@ namespace AiQ_GUI
             }
             catch
             {
-                MainForm.Instance.AddToActionsList("Error reading trim JSON - " + trimData);
+                MainForm.Instance.AddToActionsList($"Error reading trim JSON - " + trimData, Level.ERROR);
                 return;
             }
 
@@ -225,7 +226,7 @@ namespace AiQ_GUI
             {
                 if (RetryCount >= 3)
                 {
-                    await MainForm.Instance.DisplayOK("Please align trim in webpage then click OK."); // Awaited till OK has been clicked
+                    await MainForm.Instance.DisplayOK($"Please align trim in webpage then click OK."); // Awaited till OK has been clicked
                     return;
                 }
 
@@ -253,11 +254,11 @@ namespace AiQ_GUI
                 }
                 else // Ask user to centre the plate in the field of view
                 {
-                    await MainForm.Instance.DisplayOK("Please centralise plate in view THEN press OK"); // Awaited till OK has been clicked
+                    await MainForm.Instance.DisplayOK($"Please centralise plate in view THEN press OK "); // Awaited till OK has been clicked
 
                     if (RetryCount >= 3)
                     {
-                        await MainForm.Instance.DisplayOK("Please align trim in webpage then click OK."); // Awaited till OK has been clicked
+                        await MainForm.Instance.DisplayOK($"Please align trim in webpage then click OK. "); // Awaited till OK has been clicked
                         return;
                     }
                     await Task.Delay(5000); // Give 5 second delay for it to see a plate
@@ -274,7 +275,7 @@ namespace AiQ_GUI
             string TrimResp = await HTTP_Update("SightingCreator", IPAddress, Trim_JSON);
 
             if (!TrimResp.Contains($"\"propInterCameraOffsetX\": {{\"value\": \"{Convert.ToString(TrimX)}\", \"datatype\": \"int\"}}, \"propInterCameraOffsetY\": {{\"value\": \"{Convert.ToString(TrimY)}\", \"datatype\": \"int\"}},"))
-                MainForm.Instance.AddToActionsList("Could not set camera trim");
+                MainForm.Instance.AddToActionsList($"Could not set camera trim", Level.ERROR);
         }
 
         // Processes the network config from the camera and returns a string indicating the status
@@ -292,7 +293,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Error in getting network config from camera: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Error in getting network config from camera: {ex.Message}{Level.ERROR}");
                 return null; // Return empty string if there is an error
             }
         }
@@ -348,11 +349,11 @@ namespace AiQ_GUI
 
             if (FoundCams.Contains("192.168.1.211"))
             {
-                MainForm.Instance.AddToActionsList("Could not set camera to DHCP please check camera.");
+                MainForm.Instance.AddToActionsList($"Could not set camera to DHCP please check camera.", Level.ERROR);
                 return false;
             }
 
-            MainForm.Instance.AddToActionsList("Camera successfully set to DHCP.");
+            MainForm.Instance.AddToActionsList($"Camera successfully set to DHCP.{Level.Success}");
             return true;
         }
 
@@ -410,14 +411,14 @@ namespace AiQ_GUI
                 string result = await APIHTTPVISCA(ipAddress, command, isIR);
 
                 if (result.Contains(expectedResponse))
-                    MainForm.Instance.AddToActionsList($"{name}: Success ({(isIR ? "IR" : "Colour")})", Level.LOG);
+                    MainForm.Instance.AddToActionsList($"{name}: Success ({(isIR ? "IR" : "Colour")})", Level.Success);
                 else
                     MainForm.Instance.AddToActionsList($"{name}: Unexpected response ({result})", Level.ERROR);
 
                 await Task.Delay(150);
             }
 
-            MainForm.Instance.AddToActionsList($"Upload complete ({(isIR ? "IR" : "Colour")}).", Level.LOG);
+            MainForm.Instance.AddToActionsList($"Upload complete ({(isIR ? "IR" : "Colour")}).", Level.Success);
         }
 
         public static async void UploadBlob(List<Camera> soakCameraList)
@@ -431,7 +432,7 @@ namespace AiQ_GUI
 
                 if (fileToUpload == null)
                 {
-                    MainForm.Instance.AddToActionsList("No .blob file found in the directory.", Level.ERROR);
+                    MainForm.Instance.AddToActionsList("No .blob file found in the directory.", Level.WARNING);
                     return;
                 }
             }
@@ -472,7 +473,7 @@ namespace AiQ_GUI
                     result = await SendBlobFileUpload(apiUrl, fileToUpload, fileName);
                 }
 
-                MainForm.Instance.AddToActionsList($"Upload result for {cam.IP}: {result}", Level.LOG);
+                MainForm.Instance.AddToActionsList($"Upload result for {cam.IP}: {result}");
                 await Task.Delay(500);
             }
         }

Camera/ImageProcessing.cs

@@ -26,7 +26,7 @@ namespace AiQ_GUI
                 HttpResponseMessage response = await httpClient.GetAsync(requestUrl);
                 if (!response.IsSuccessStatusCode)
                 {
-                    MainForm.Instance.AddToActionsList($"No success from {requestUrl} replied {response.StatusCode}");
+                    MainForm.Instance.AddToActionsList($"No success from {requestUrl} replied {response.StatusCode}", Level.ERROR);
                     return null;
                 }
 
@@ -34,7 +34,7 @@ namespace AiQ_GUI
 
                 if (imageBytes.Length == 0) // Check if the imageBytes is empty
                 {
-                    MainForm.Instance.AddToActionsList($"No image data received from {requestUrl}");
+                    MainForm.Instance.AddToActionsList($"No image data received from {requestUrl}", Level.ERROR);
                     return null;
                 }
 
@@ -43,7 +43,7 @@ namespace AiQ_GUI
                 CvInvoke.Imdecode(imageBytes, ImreadModes.AnyColor, mat);
                 if (mat.IsEmpty)
                 {
-                    MainForm.Instance.AddToActionsList("Failed to decode image with Emgu CV.");
+                    MainForm.Instance.AddToActionsList($"Failed to decode image with Emgu CV.", Level.ERROR);
                     return null;
                 }
 
@@ -70,12 +70,12 @@ namespace AiQ_GUI
             }
             catch (HttpRequestException ex)
             {
-                MainForm.Instance.AddToActionsList($"HTTP error: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"HTTP error: {ex.Message}", Level.ERROR);
                 return null;
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Error processing image: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Error processing image: {ex.Message}", Level.ERROR);
                 return null;
             }
         }
@@ -123,7 +123,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Error awaiting Colour snapshot: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Error awaiting Colour snapshot: {ex.Message}", Level.ERROR);
                 return;
             }
 
@@ -146,7 +146,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Error calculating luminance: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Error calculating luminance: {ex.Message}",Level.ERROR);
                 return;
             }
         }

Camera/LED.cs

@@ -9,7 +9,24 @@
             {
                 VorI.Sort(); // Sort the list from lowest to highest to prepare for finding the median
                 double medianVorI = (VorI[2] + VorI[3]) / 2.0; // Will always be even (6) number of channels therefore average the two middle elements
-                lblVorI.Text += $"Median: {medianVorI}{VormA} "; // Display median value
+                string medianText = $"Median: {medianVorI}{VormA}";
+
+                //Expected value of 0 means informational only (force green)
+                if (ExpVorI == 0)
+                {
+                    if (lblVorI == null)
+                    {
+                        MainForm.Instance.AddLabelToPanel($"LED {VormA} = {medianText}", false);
+                        return;
+                    }
+
+                    lblVorI.Invoke(() =>
+                    {
+                        lblVorI.Text = lblVorI.Text.TrimEnd('=', ' ') + " = " + medianText;
+                        lblVorI.ForeColor = Color.LimeGreen;
+                    });
+                    return;
+                }
 
                 // Define the 20% threshold ranges
                 double LowerThreshold = ExpVorI * 0.8;
@@ -18,8 +35,17 @@
                 // Check median is within 20% of the expected value
                 if (medianVorI < LowerThreshold || medianVorI > UpperThreshold)
                 {
-                    lblVorI.Text += $" Median away from excepted {ExpVorI}{VormA}";
+                    medianText += $" (away from expected {ExpVorI}{VormA})";
-                    lblVorI.ForeColor = Color.Red;
+                    if (lblVorI == null)
+                    {
+                        MainForm.Instance.AddLabelToPanel($"LED {VormA} = {medianText}", true);
+                        return;
+                    }
+                    lblVorI.Invoke(() =>
+                    {
+                        lblVorI.Text = lblVorI.Text.TrimEnd('=', ' ') + " = " + medianText;
+                        lblVorI.ForeColor = Color.Red;
+                    });
                     return;
                 }
 
@@ -35,17 +61,35 @@
                 // If there are no single channels outside the threshold then green, else red
                 if (outOfRangeVoltageChannels.Count == 0)
                 {
-                    lblVorI.ForeColor = Color.LightGreen;
+                    if (lblVorI == null)
+                    {
+                        MainForm.Instance.AddLabelToPanel($"LED {VormA} = {medianText}", false);
+                        return;
+                    }
+                    lblVorI.Invoke(() =>
+                    {
+                        lblVorI.Text = lblVorI.Text.TrimEnd('=', ' ') + " = " + medianText;
+                        lblVorI.ForeColor = Color.LimeGreen;
+                    });
                 }
                 else if (outOfRangeVoltageChannels.Count != 0)
                 {
-                    lblVorI.Text += "error on " + string.Join(", ", outOfRangeVoltageChannels); // Join all problem channels together to present on form
+                    string errorText = medianText + " (error on " + string.Join(", ", outOfRangeVoltageChannels) + ")";
-                    lblVorI.ForeColor = Color.Red;
+                    if (lblVorI == null)
+                    {
+                        MainForm.Instance.AddLabelToPanel($"LED {VormA} = {errorText}", true);
+                        return;
+                    }
+                    lblVorI.Invoke(() =>
+                    {
+                        lblVorI.Text = lblVorI.Text.TrimEnd('=', ' ') + " = " + errorText;
+                        lblVorI.ForeColor = Color.Red;
+                    });
                 }
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Error checking LEDs: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Error checking LEDs: {ex.Message}", Level.ERROR);
             }
         }
     }

Camera/Licences.cs

@@ -25,13 +25,13 @@ namespace AiQ_GUI
             else if (Type == "Audit")
                 salt = Auditsalt;
             else
-                return "Unrecognised challenge type: " + Type;
+                return $"Unrecognised challenge type:" + Type;
 
             if (string.IsNullOrEmpty(challenge) || challenge.Length != 6) // Check challenge format
-                return "Invalid challenge format. Challenge must be 6 characters.";
+                return $"Invalid challenge format. Challenge must be 6 characters.";
 
             if (string.IsNullOrEmpty(salt) || salt.Length != 32) // Check salt format
-                return "Invalid salt format. Salt must be 32 characters.";
+                return $"Invalid salt format. Salt must be 32 characters.";
 
             // Hash computation using SHA256 algorithm
             byte[] inputBytes = Encoding.UTF8.GetBytes(challenge + " " + salt); // SHA hash format challenge and salt with space between
@@ -60,7 +60,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                return "Error: Could not generate password " + ex.Message;
+                return $"Error: Could not generate password" + ex.Message;
             }
         }
 
@@ -85,7 +85,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Exception in FetchDevPassword: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Exception in FetchDevPassword:" + ex.Message, Level.ERROR);
                 return null;
             }
         }

Camera/Router.cs

@@ -32,7 +32,7 @@ namespace AiQ_GUI
             }
             catch
             {
-                MainForm.Instance.AddToActionsList("Is the router on the network? Has the MAV Config file been applied?");
+                MainForm.Instance.AddToActionsList($"Is the router on the network? Has the MAV Config file been applied?", Level.WARNING);
             }
 
             return null;
@@ -48,31 +48,31 @@ namespace AiQ_GUI
 
             if (Strength < 25.0)
             {
-                MainForm.Instance.AddToActionsList($"Router signal strength is {Strength} which is below 25%. Please check the router connection.");
+                MainForm.Instance.AddToActionsList($"Router signal strength is {Strength} which is below 25%. Please check the router connection.", Level.WARNING);
                 PassTest = false;
             }
 
             if (!Router.SimStatus.Contains("SIM Ready"))
             {
-                MainForm.Instance.AddToActionsList($"SIM card is not ready. {Router.SimStatus} Please check the SIM card status.");
+                MainForm.Instance.AddToActionsList($"SIM card is not ready. {Router.SimStatus} Please check the SIM card status.", Level.WARNING);
                 PassTest = false;
             }
 
             if (!Router.Port3Status.Contains("port:3 link:up speed:100baseT full-duplex"))
             {
-                MainForm.Instance.AddToActionsList($"Port 3 is not connected properly. {Router.Port3Status} Please check the connection.");
+                MainForm.Instance.AddToActionsList($"Port 3 is not connected properly. {Router.Port3Status} Please check the connection.", Level.WARNING);
                 PassTest = false;
             }
 
             if (!Router.Port4Status.Contains("port:4 link:up speed:100baseT full-duplex"))
             {
-                MainForm.Instance.AddToActionsList($"Port 4 is not connected properly. {Router.Port4Status} Please check the connection.");
+                MainForm.Instance.AddToActionsList($"Port 4 is not connected properly. {Router.Port4Status} Please check the connection. ", Level.WARNING);
                 PassTest = false;
             }
 
             if (!Router.GoodPing)
             {
-                MainForm.Instance.AddToActionsList("Router could not ping 8.8.8.8. Please check the online connection.");
+                MainForm.Instance.AddToActionsList($"Router could not ping 8.8.8.8. Please check the online connection.", Level.WARNING);
                 PassTest = false;
             }
 

Camera/SSH.cs

@@ -9,7 +9,7 @@ namespace AiQ_GUI
         public const string SSHPassword = "mavPA$$";
         public const string SSHPasswordNEW = "#!mavsoftMESA19"; // New password for SSH after last one got leaked
 
-        private static SshClient SshConnect(string IPAddress)
+        public static SshClient SshConnect(string IPAddress)
         {
             SshClient client = new(IPAddress, SSHUsername, SSHPasswordNEW);
             try
@@ -25,7 +25,7 @@ namespace AiQ_GUI
             }
             catch (Exception Ex)
             {
-                MainForm.Instance.AddToActionsList($"SSH connection failed: {Ex.Message}. Check password or network.");
+                MainForm.Instance.AddToActionsList($"SSH connection failed: {Ex.Message}. Check password or network. ", Level.WARNING);
             }
 
             return null;
@@ -40,6 +40,13 @@ namespace AiQ_GUI
             {
                 SshClient client = SshConnect(IPAddress);
 
+                // CRITICAL: Check if connection actually succeeded
+                if (client == null)
+                {
+                    MainForm.Instance.AddToActionsList($"SSH connection failed for {IPAddress}", Level.ERROR);
+                    return Data; // Return empty data
+                }
+
                 try
                 {
                     Data.packages = GetVaxtorPackages(client);
@@ -75,12 +82,9 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"SSH connection failed: {ex.Message}. Check password or network.");
+                MainForm.Instance.AddToActionsList($"SSH connection failed: {ex.Message}. Check password or network. ", Level.WARNING);
             }
 
-            string LogMssg = string.Join(" | ", typeof(SSHData).GetProperties().Select(p => $"{p.Name}: {p.GetValue(Data)}"));
-            Logging.LogMessage(LogMssg); // Log all of Data
-
             return Data;
         }
 
@@ -170,7 +174,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"SSH connection failed: {ex.Message}. Check password or network.");
+                MainForm.Instance.AddToActionsList($"SSH connection failed: {ex.Message}. Check password or network. ", Level.WARNING);
             }
 
             return (string.Empty, string.Empty);
@@ -234,6 +238,14 @@ namespace AiQ_GUI
             const double Deviation = 20.0; // ±20GB
 
             double currentSize = NormaliseFSSize(sshData.FilesystemSize);
+            
+            // Create label dynamically if not provided
+            if (LblFSSize == null)
+            {
+                MainForm.Instance.AddLabelToPanel($"Filesystem Size = {currentSize}GB", currentSize < (GoodSize - Deviation) || currentSize > (GoodSize + Deviation));
+                return sshData;
+            }
+
             LblFSSize.Text = $"Filesystem Size = {currentSize}GB";
 
             if (Math.Abs(GoodSize - currentSize) < Deviation)
@@ -280,7 +292,8 @@ namespace AiQ_GUI
         {
             try
             {
-                if (string.IsNullOrWhiteSpace(rootSize)) return 0;
+                if (string.IsNullOrWhiteSpace(rootSize))
+                    return 0;
 
                 // Extract value & unit
                 System.Text.RegularExpressions.Match match = RegexCache.FileSizeRegex().Match(rootSize.Trim());
@@ -323,7 +336,7 @@ namespace AiQ_GUI
 
                 if (checkDevice.Result.Trim() != "OK") // Device not found
                 {
-                    MainForm.Instance.AddToActionsList($"Block device {device} not found.");
+                    MainForm.Instance.AddToActionsList($"Block device {device} not found. ", Level.WARNING);
                     return false;
                 }
 
@@ -365,6 +378,24 @@ namespace AiQ_GUI
                 return false;
             }
         }
+        public static bool MobiletxtCheck(string IPAddress)
+        {
+            try
+            {
+                using (SshClient client = SshConnect(IPAddress))
+                {
+                    // test -f returns exit status 0 if file exists
+                    SshCommand checkDevice = client.RunCommand("test -f /home/mav/Mobile-Setup-configuration-marker.txt"
+                    );
+                    return checkDevice.ExitStatus == 0;
+                }
+            }
+            catch (Exception ex)
+            {
+                MainForm.Instance.AddToActionsList($"Can't check txt file: Error {ex}", Level.ERROR);
+                return false;
+            }
+        }
 
         public static void Sync(string IPAddress)
         {
@@ -378,7 +409,7 @@ namespace AiQ_GUI
 
                 if (checkDevice.Result.Trim().Length > 1) // Device not found
                 {
-                    MainForm.Instance.AddToActionsList($"Cannot sync files to disk. Replied: {checkDevice.Result}. DO NOT TURN OFF, GET SUPERVISOR");
+                    MainForm.Instance.AddToActionsList($"Cannot sync files to disk. Replied: {checkDevice.Result}. DO NOT TURN OFF, GET SUPERVISOR", Level.ERROR);
                     return;
                 }
 
@@ -387,7 +418,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Cannot sync becuase: {ex.Message}. DO NOT TURN OFF, GET SUPERVISOR");
+                MainForm.Instance.AddToActionsList($"Cannot sync becuase: {ex.Message}. DO NOT TURN OFF, GET SUPERVISOR", Level.ERROR);
             }
         }
     }

GoogleAPI.cs

@@ -107,7 +107,7 @@ namespace AiQ_GUI
                 }
                 else
                 {
-                    return "Last serial number not found";
+                    return $"Last serial number not found";
                 }
             }
             catch (Exception ex)
@@ -148,7 +148,7 @@ namespace AiQ_GUI
                 }
                 else
                 {
-                    return "Serial number not found";
+                    return $"Serial number not found";
                 }
             }
             catch (Exception ex)
@@ -196,7 +196,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                return "Failed to update spreadsheet data, please check manually" + ex.Message;
+                return $"Failed to update spreadsheet data, please check manually" + ex.Message;
             }
         }
 
@@ -225,7 +225,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                return "Failed to update spreadsheet data, please check manually" + ex.Message;
+                return $"Failed to update spreadsheet data, please check manually" + ex.Message;
             }
         }
 

Helper.cs

@@ -83,26 +83,26 @@ namespace AiQ_GUI
                     case 0:
                         if (!await MainForm.Instance.DisplayQuestion("Is the sleeve aligned correctly?"))
                         {
-                            await MainForm.Instance.TestFailed(Btn, "Visual Test Fail - Sleeve not aligned");
+                            await MainForm.Instance.TestFailed(Btn, $"Visual Test Fail - Sleeve not aligned");
 
                         }
                         break;
                     case 1:
                         if (!await MainForm.Instance.DisplayQuestion("Are all the screws fitted in the front?"))
                         {
-                            await MainForm.Instance.TestFailed(Btn, "Visual Test Fail - Not all front screws fitted");
+                            await MainForm.Instance.TestFailed(Btn, $"Visual Test Fail - Not all front screws fitted");
                         }
                         break;
                     case 2:
                         if (!await MainForm.Instance.DisplayQuestion("Are all the screws fitted in the rear?"))
                         {
-                            await MainForm.Instance.TestFailed(Btn, "Visual Test Fail - Not all rear screws fitted");
+                            await MainForm.Instance.TestFailed(Btn, $"Visual Test Fail - Not all rear screws fitted");
                         }
                         break;
                     case 3:
                         if (await MainForm.Instance.DisplayQuestion("Shake unit, does it rattle?"))
                         {
-                            await MainForm.Instance.TestFailed(Btn, "Visual Test Fail - Unit rattles");
+                            await MainForm.Instance.TestFailed(Btn, $"Visual Test Fail - Unit rattles");
                         }
                         break;
                     default:
@@ -123,7 +123,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Error fetching versions for {IPAddress}: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Error fetching versions for {IPAddress}: {ex.Message}", Level.ERROR);
                 return null;
             }
 
@@ -187,6 +187,7 @@ namespace AiQ_GUI
         public bool IsChecked { get; set; } // Is it checked for soak test?
         public CheckBox CheckBox { get; set; } = new CheckBox();
         public string TestReportLoc { get; set; } = string.Empty; // Location of test report file
+        public string Manfr { get; internal set; }
     }
 
     // Static class for global flags

LDS.cs

@@ -30,7 +30,7 @@ namespace AiQ_GUI
             }
             catch // If file can't deserialise
             {
-                MainForm.Instance.AddToActionsList("Error loading Local Data Store");
+                MainForm.Instance.AddToActionsList($"Error loading Local Data Store", Level.WARNING);
                 return null; // Return null to indicate failure
             }
         }
@@ -45,7 +45,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList($"Error saving Local Data Store: {ex.Message}");
+                MainForm.Instance.AddToActionsList($"Error saving Local Data Store: {ex.Message}", Level.WARNING);
             }
         }
     }

Logging.cs

@@ -1,4 +1,6 @@
-namespace AiQ_GUI
+using System.Diagnostics;
+
+namespace AiQ_GUI
 {
     internal class Logging
     {

Microsoft/Access.cs

@@ -4,14 +4,32 @@ namespace AiQ_GUI
 {
     class Access
     {
-        public const string connString = @"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=G:\Shared drives\MAV Production GUI's\AiQ\GUI's\AiQ_Final_Test.accdb;Persist Security Info=False;OLE DB Services=-1;";
+ 
+        public const string connString =
+            @"Provider=Microsoft.ACE.OLEDB.12.0;Data Source=G:\Shared drives\MAV Production GUI's\AiQ\GUI's\AiQ_Final_Test.accdb;Persist Security Info=False;OLE DB Services=-1;";
 
-        // Reads camera model numbers and descriptions from the database, sorts them alphabetically by model number (except "AB12CD", which appears last), and formats each entry as "ModelNumber - Description".
+        // Allows Different Cam Types with different schemas to be handled safely
-        public static string[] ReadCamTypes()
+       
+        private static bool HasColumn(OleDbDataReader reader, string columnName)
         {
-            List<Tuple<string, string>> modelTuples = new(30); // Preallocate list with estimated capacity to reduce internal resizing
+            for (int i = 0; i < reader.FieldCount; i++)
+                if (reader.GetName(i).Equals(columnName, StringComparison.OrdinalIgnoreCase))
+                    return true;
+
+            return false;
+        }
+        public static string[] ReadCamTypes(string camType)
+        {
+            // No camera type selected
+            if (string.IsNullOrWhiteSpace(camType))
+                return null;
+
+            // Preallocate list to reduce resizing
+            List<Tuple<string, string>> modelTuples = new(30);
+
             using OleDbConnection conn = new(connString);
 
+            // Attempt to open the database
             try
             {
                 conn.Open();
@@ -22,31 +40,36 @@ namespace AiQ_GUI
                 return null;
             }
 
-            const string query = "SELECT ModelNumber, Description FROM AiQ WHERE MarkNumber > 1";
+            // Mobile table does not have MarkNumber
+            // AiQ and others might do ????? - TODO ask
+            string query = camType == "Mobile"
+                ? "SELECT ModelNumber, Description FROM [Mobile]"
+                : $"SELECT ModelNumber, Description FROM [{camType}] WHERE MarkNumber > 1";
 
             using OleDbCommand cmd = new(query, conn);
             using OleDbDataReader reader = cmd.ExecuteReader();
+
+            // Read all models from the selected table
             while (reader.Read())
             {
-                // Extract each model number and description, using empty string if null
                 string modelNumber = reader["ModelNumber"] as string ?? string.Empty;
                 string description = reader["Description"] as string ?? string.Empty;
+
                 modelTuples.Add(Tuple.Create(modelNumber.Trim(), description.Trim()));
             }
 
-            // Sort: push "AB12CD" to the bottom, then sort remaining items alphabetically
+            // Sort models, pushing AB12CD to the bottom
-            IOrderedEnumerable<Tuple<string, string>> sorted = modelTuples.OrderBy(t => t.Item1.Equals("AB12CD", StringComparison.OrdinalIgnoreCase) ? 1 : 0)
+            var sorted = modelTuples
-                                    .ThenBy(t => t.Item1, StringComparer.OrdinalIgnoreCase);
+                .OrderBy(t => t.Item1.Equals("AB12CD", StringComparison.OrdinalIgnoreCase) ? 1 : 0)
+                .ThenBy(t => t.Item1, StringComparer.OrdinalIgnoreCase);
 
-            conn.Close();
+            // Format for combo box display
-            // Format the sorted tuples as "ModelNumber - Description" strings and return as array
             return sorted.Select(t => $"{t.Item1} - {t.Item2}").ToArray();
         }
-
-        // Read the universal data table from the database and populate the UniversalData class with the values.
         public static void ReadUniData()
         {
             using OleDbConnection conn = new(connString);
+
             try
             {
                 conn.Open();
@@ -57,24 +80,28 @@ namespace AiQ_GUI
                 return;
             }
 
-            const string query = "SELECT FlexiVersion, FlexiRevision, WonwooFirmware, AiQGUIVersion, PowerConsumption, LicencingServerURL FROM UniversalData"; // Grab the universal data
+            const string query =
+                "SELECT FlexiVersion, FlexiRevision, WonwooFirmware, AiQGUIVersion, PowerConsumption, LicencingServerURL, SRZFirmware FROM UniversalData";
 
             using OleDbCommand cmd = new(query, conn);
             using OleDbDataReader reader = cmd.ExecuteReader();
+
+            // UniversalData is expected to contain a single row
             reader.Read();
+
             UniversalData.ExpFlexiVer = Convert.ToString(reader["FlexiVersion"]);
             UniversalData.ExpFlexiRev = Convert.ToString(reader["FlexiRevision"]);
             UniversalData.WonwooFirmware = Convert.ToString(reader["WonwooFirmware"]);
             UniversalData.LatestVersion = Convert.ToString(reader["AiQGUIVersion"]);
             UniversalData.PowerConsumption = Convert.ToInt16(reader["PowerConsumption"]);
             UniversalData.LicencingServerURL = Convert.ToString(reader["LicencingServerURL"]);
-            conn.Close();
+            UniversalData.SRZFirmware = Convert.ToString(reader["SRZFirmware"]);
         }
-
+        // Populates CameraAccessInfo dynamically based on available columns
-        // Knowing the model number on test, this function reads the database and populates the Camera class with the values.
+        public static void ReadModelRow(string camType, string ModelOnTest)
-        public static void ReadModelRow(string ModelOnTest)
         {
             using OleDbConnection conn = new(connString);
+
             try
             {
                 conn.Open();
@@ -85,66 +112,85 @@ namespace AiQ_GUI
                 return;
             }
 
-            string query = $"SELECT * FROM AiQ WHERE ModelNumber = '{ModelOnTest}';"; // Grab all the info for specified model
+            // Parameterised query to prevent injection
+            string query = $"SELECT * FROM [{camType}] WHERE ModelNumber = ?";
 
             using OleDbCommand cmd = new(query, conn);
+            cmd.Parameters.AddWithValue("?", ModelOnTest);
+
             using OleDbDataReader reader = cmd.ExecuteReader();
-            reader.Read();
 
-            // Populate the CameraAccessInfo class with the values from the database
+            // No matching model found
-            CameraAccessInfo.Processor = Convert.ToString(reader["Processor"]);
+            if (!reader.Read())
-            CameraAccessInfo.VaxtorLic = Convert.ToBoolean(reader["Vaxtor"]);
+                return;
-            CameraAccessInfo.HardwareExtras = Convert.ToString(reader["HardwareExtras"]);
+
-            CameraAccessInfo.PowerType = Convert.ToString(reader["PowerType"]);
+            // Populate CameraAccessInfo only if columns exist
-            CameraAccessInfo.LED_V = Convert.ToDouble(reader["LEDVoltage"]);
+            CameraAccessInfo.Processor =
-            CameraAccessInfo.LED_I = Convert.ToInt32(reader["LEDCurrent"]);
+                HasColumn(reader, "Processor") ? Convert.ToString(reader["Processor"]) : string.Empty;
-            CameraAccessInfo.SpreadsheetID = Convert.ToString(reader["SSID"]);
+
-            conn.Close();
+            CameraAccessInfo.VaxtorLic =
+                HasColumn(reader, "Vaxtor") && Convert.ToString(reader["Vaxtor"]) == "Yes";
+
+            CameraAccessInfo.HardwareExtras =
+                HasColumn(reader, "HardwareExtras") ? Convert.ToString(reader["HardwareExtras"]) : string.Empty;
+
+            CameraAccessInfo.PowerType =
+                HasColumn(reader, "PowerType") ? Convert.ToString(reader["PowerType"]) : string.Empty;
+
+            CameraAccessInfo.LED_V =
+                HasColumn(reader, "LEDVoltage") ? Convert.ToDouble(reader["LEDVoltage"]) : 0;
+
+            CameraAccessInfo.LED_I =
+                HasColumn(reader, "LEDCurrent") ? Convert.ToInt32(reader["LEDCurrent"]) : 0;
+
+            CameraAccessInfo.SpreadsheetID =
+                HasColumn(reader, "SSID") ? Convert.ToString(reader["SSID"]) : string.Empty;
         }
-
         public static void Stats(string TypeOfTest, string modelNumber)
         {
             Stats([TypeOfTest], modelNumber);
         }
-
         public static void Stats(string[] TypeOfTest, string modelNumber)
         {
-            using OleDbConnection conn = new(connString); // Opens connection to Access database
+            using OleDbConnection conn = new(connString);
+
             try
             {
-                conn.Open(); // Opens DB
+                conn.Open();
 
                 foreach (string type in TypeOfTest)
                 {
-                    string query = $"UPDATE AiQ SET [{type}] = [{type}] + 1 WHERE [ModelNumber] = ?"; // Add one for every test ran of this type for this model number
+                    string query =
-                    using OleDbCommand cmd = new(query, conn); // Create command
+                        $"UPDATE AiQ SET [{type}] = [{type}] + 1 WHERE [ModelNumber] = ?";
-                    cmd.Parameters.AddWithValue("?", modelNumber); // Add model number to prevent injection
+
+                    using OleDbCommand cmd = new(query, conn);
+                    cmd.Parameters.AddWithValue("?", modelNumber);
 
                     int rowsAffected = cmd.ExecuteNonQuery();
-                    // Execute the command and get the number of rows affected
+
-                    if (rowsAffected == 0) // If one or more rows were updated
+                    if (rowsAffected == 0)
-                        MainForm.Instance.AddToActionsList($"No rows affected for {modelNumber}");
+                        MainForm.Instance.AddToActionsList($"No rows affected for {modelNumber}", Level.ERROR);
                 }
-                conn.Close();
             }
             catch
             {
-                MainForm.Instance.AddToActionsList("Could not access Access in Google Drive. Is it running?");
+                MainForm.Instance.AddToActionsList(
-                return;
+                    "Could not access Access in Google Drive. Is it running?", Level.WARNING);
             }
         }
-
         public static void StatsDiags(string redDiagLabels, string RhTxBxActionsText, string ModelNumber)
         {
             using OleDbConnection conn = new(connString);
             conn.Open();
 
-            // Null checks
+            // Replace null or empty values
             string redVal = string.IsNullOrWhiteSpace(redDiagLabels) ? "-" : redDiagLabels;
             string actVal = string.IsNullOrWhiteSpace(RhTxBxActionsText) ? "-" : RhTxBxActionsText;
             string model = string.IsNullOrWhiteSpace(ModelNumber) ? "-" : ModelNumber;
 
-            const string sql = @"INSERT INTO DiagsStats ([Date], [Model], [Red Diags Labels], [RhTxBxActions Contents]) VALUES (?, ?, ?, ?)";
+            const string sql =
+                @"INSERT INTO DiagsStats ([Date], [Model], [Red Diags Labels], [RhTxBxActions Contents])
+                  VALUES (?, ?, ?, ?)";
 
             using OleDbCommand cmd = new(sql, conn);
 
@@ -153,19 +199,18 @@ namespace AiQ_GUI
                 OleDbType = OleDbType.Date,
                 Value = DateTime.Now
             });
+
             cmd.Parameters.AddWithValue("?", model);
             cmd.Parameters.AddWithValue("?", redVal);
             cmd.Parameters.AddWithValue("?", actVal);
 
             int rows = cmd.ExecuteNonQuery();
-            conn.Close();
 
             if (rows == 0)
-                MainForm.Instance.AddToActionsList("No rows inserted into DiagsStats (unexpected).");
+                MainForm.Instance.AddToActionsList(
+                    "No rows inserted into DiagsStats (unexpected).", Level.ERROR);
         }
     }
-
-    // Expected universal data for the GUI, read from the database
     public class UniversalData
     {
         public static string ExpFlexiVer { get; set; } = string.Empty;
@@ -174,9 +219,9 @@ namespace AiQ_GUI
         public static string LatestVersion { get; set; } = string.Empty;
         public static int PowerConsumption { get; set; } = 0;
         public static string LicencingServerURL { get; set; } = string.Empty;
+        public static string SRZFirmware { get; set; } = string.Empty;
     }
 
-    // One object to contain all the camera info from the model info access database
     public class CameraAccessInfo
     {
         public static string Processor { get; set; } = string.Empty;

Microsoft/Excel.cs

@@ -17,7 +17,7 @@ namespace AiQ_GUI
             }
             else
             {
-                MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                MainForm.Instance.AddToActionsList($"Could not find spreadsheet :( ", Level.ERROR);
             }
         }
 
@@ -31,7 +31,7 @@ namespace AiQ_GUI
             }
             else
             {
-                MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                MainForm.Instance.AddToActionsList($"Could not find spreadsheet :( ", Level.ERROR);
             }
             return null;
         }
@@ -40,7 +40,7 @@ namespace AiQ_GUI
         {
             if (!File.Exists(FilePath))
             {
-                MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                MainForm.Instance.AddToActionsList($"Could not find spreadsheet :( ", Level.ERROR);
                 return -1;
             }
 
@@ -62,7 +62,7 @@ namespace AiQ_GUI
             {
                 if (!File.Exists(FilePath))
                 {
-                    MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                    MainForm.Instance.AddToActionsList($"Could not find spreadsheet :(  ", Level.ERROR);
                     return "Spreadsheet not found";
                 }
 
@@ -114,7 +114,7 @@ namespace AiQ_GUI
             {
                 if (!File.Exists(filePath))
                 {
-                    MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                    MainForm.Instance.AddToActionsList($"Could not find spreadsheet :) ", Level.ERROR);
                     return "Spreadsheet not found";
                 }
 
@@ -150,7 +150,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Error updating spreadsheet: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Error updating spreadsheet:{Level.ERROR} " + ex.Message);
                 return $"ERROR: {ex.Message}";
             }
         }
@@ -161,7 +161,7 @@ namespace AiQ_GUI
             {
                 if (!File.Exists(FilePath))
                 {
-                    MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                    MainForm.Instance.AddToActionsList($"Could not find spreadsheet :(  ", Level.ERROR);
                     return "Spreadsheet not found";
                 }
 
@@ -202,8 +202,8 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Error updating spreadsheet: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Error updating spreadsheet: {Level.WARNING}" + ex.Message);
-                return "Failed to update spreadsheet data, please check manually: " + ex.Message;
+                return $"Failed to update spreadsheet data, please check manually: {Level.WARNING}" + ex.Message;
             }
         }
 
@@ -213,8 +213,9 @@ namespace AiQ_GUI
             {
                 if (!File.Exists(FilePath))
                 {
-                    MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                    MainForm.Instance.AddToActionsList($"Could not find spreadsheet :( ", Level.ERROR);
-                    return "Spreadsheet not found";
+                    return $"Spreadsheet not found{ Level.ERROR}"
+                    ;
                 }
 
                 using XLWorkbook workbook = new(FilePath);
@@ -239,8 +240,8 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Error updating Vaxtor spreadsheet: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Error updating Vaxtor spreadsheet: {Level.WARNING}" + ex.Message);
-                return "Failed to update spreadsheet data, please check manually: " + ex.Message;
+                return $"Failed to update spreadsheet data, please check manually: {Level.WARNING}" + ex.Message;
             }
         }
 
@@ -250,7 +251,7 @@ namespace AiQ_GUI
             {
                 if (!File.Exists(filePath))
                 {
-                    MainForm.Instance.AddToActionsList("Could not find RMA Control spreadsheet :(");
+                    MainForm.Instance.AddToActionsList($"Could not find RMA Control spreadsheet :(", Level.ERROR);
                     return 0;
                 }
 
@@ -280,7 +281,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Error reading RMA Control spreadsheet: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Error reading RMA Control spreadsheet: {Level.ERROR}" + ex.Message);
             }
 
             return 0; // Default if not found
@@ -292,7 +293,7 @@ namespace AiQ_GUI
             {
                 if (!File.Exists(filePath))
                 {
-                    MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                    MainForm.Instance.AddToActionsList($"Could not find spreadsheet :( ", Level.ERROR);
                     return 0;
                 }
 
@@ -316,7 +317,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Error checking serial number row: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Error checking serial number row: {Level.ERROR}" + ex.Message);
             }
 
             return 0;
@@ -328,7 +329,7 @@ namespace AiQ_GUI
             {
                 if (!File.Exists(filePath))
                 {
-                    MainForm.Instance.AddToActionsList("Could not find spreadsheet :(");
+                    MainForm.Instance.AddToActionsList($"Could not find spreadsheet :(", Level.ERROR);
                     return -1;
                 }
 
@@ -343,7 +344,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Error checking next free row: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Error checking next free row:" + ex.Message, Level.ERROR);
                 return -1;
             }
         }

Microsoft/StatsExcel.cs

@@ -84,8 +84,28 @@ namespace AiQ_GUI
                 XLWorkbook workbook;
                 if (File.Exists(exportPath))
                 {
-                    workbook = new XLWorkbook(exportPath);
+                    try
-                    MainForm.Instance.AddToActionsList("Opened existing workbook.");
+                    {
+                        workbook = new XLWorkbook(exportPath);
+                        MainForm.Instance.AddToActionsList("Opened existing workbook.");
+                    }
+                    catch (Exception loadEx)
+                    {
+                        // Workbook is corrupted, backup the old file and create a new one
+                        string backupPath = exportPath + ".corrupted_" + DateTime.Now.ToString("yyyyMMdd_HHmmss");
+                        try
+                        {
+                            File.Move(exportPath, backupPath, overwrite: true);
+                            MainForm.Instance.AddToActionsList($"WARNING: Existing workbook was corrupted. Backed up to: {backupPath}");
+                        }
+                        catch (Exception backupEx)
+                        {
+                            MainForm.Instance.AddToActionsList($"WARNING: Could not backup corrupted workbook. Details: {backupEx.Message}");
+                        }
+
+                        workbook = new XLWorkbook();
+                        MainForm.Instance.AddToActionsList("Created new workbook (previous file was corrupted).");
+                    }
                 }
                 else
                 {

MobileTests/MobileAPI.cs

@@ -0,0 +1,168 @@
+using AiQ_GUI;
+using System;
+using System.Net.Http;
+using System.Net.Http.Json;
+using System.Security.Cryptography;
+using System.Text;
+using System.Text.Json;
+using System.Threading.Tasks;
+
+namespace AiQ_GUI.Mobile_Tests
+{
+    public static class MobileAPI
+    {
+        // Login credentials
+        private const string DefaultUsername = "ADMIN";
+        private const string DefaultPassword = "1234";
+
+        public static async Task CheckFirmwareAsync()
+        {
+            using HttpClient client = new HttpClient
+            {
+                BaseAddress = new Uri($"http://{MainForm.Instance.CamOnTest.IP}")
+            };
+
+            try
+            {
+                // Login and get JWT token
+                string token = await LoginAsync(client, DefaultUsername, DefaultPassword);
+
+                // Attach JWT to all requests
+                client.DefaultRequestHeaders.Authorization = new System.Net.Http.Headers.AuthenticationHeaderValue("Bearer", token);
+
+                // Request firmware information
+                HttpResponseMessage response = await client.GetAsync("/app/v1/system/firmware");
+
+                string body = await response.Content.ReadAsStringAsync();
+
+                response.EnsureSuccessStatusCode();
+
+                JsonElement json = JsonSerializer.Deserialize<JsonElement>(body);
+
+                string version = json.GetProperty("version").GetString()?.Trim();
+
+                // Compare against expected SRZFirmware from UniversalData
+                if (string.IsNullOrEmpty(UniversalData.SRZFirmware))
+                {
+                    MainForm.Instance.AddToActionsList($"Firmware check failed: Expected SRZFirmware not loaded in UniversalData", Level.ERROR);
+                    MainForm.Instance.AddLabelToPanel($"Firmware = {version}", true);
+                }
+                else if (version == UniversalData.SRZFirmware)
+                {
+                    MainForm.Instance.AddLabelToPanel($"Firmware = {version}", false);
+                }
+                else
+                {
+                    MainForm.Instance.AddLabelToPanel($"Firmware = {version}", true);
+                }
+            }
+            catch (Exception ex)
+            {
+                MainForm.Instance.AddToActionsList($"Firmware check failed: {ex.Message}", Level.ERROR);
+                MainForm.Instance.AddLabelToPanel("Firmware = Unknown", true);
+            }
+        }
+
+        public static async Task ChangeDayNightModeAsync(uint mode)
+        {
+            // mode: 0 = Auto, 1 = Day, 2 = Night
+            string modeName = mode switch
+            {
+                0 => "Auto",
+                1 => "Day",
+                2 => "Night",
+                _ => "Unknown"
+            };
+
+            // operatingMode required by firmware (!)
+            // day -> 1, night -> 0
+            int operatingMode = mode switch
+            {
+                1 => 1, // Day
+                2 => 0, // Night
+                _ => 1  // Auto: keep day as safe default
+            };
+
+            using HttpClient client = new HttpClient
+            {
+                BaseAddress = new Uri($"http://{MainForm.Instance.CamOnTest.IP}")
+            };
+
+            try
+            {
+                // Login
+                string token = await LoginAsync(client, DefaultUsername, DefaultPassword);
+                client.DefaultRequestHeaders.Authorization = new System.Net.Http.Headers.AuthenticationHeaderValue("Bearer", token);
+
+                var payload = new
+                {
+                    dayNightMode = new
+                    {
+                        mode = mode,
+                        dayToNightSens = 4,
+                        nightToDaySens = 24,
+                        transitionDelay = 20,
+                        operatingMode = operatingMode
+                    }
+                };
+
+                HttpResponseMessage response = await client.PostAsJsonAsync("/app/v1/camera/image", payload);
+
+                if (response.IsSuccessStatusCode)
+                {
+                    MainForm.Instance.AddToActionsList($"Camera set to {modeName} mode successfully",Level.Success);
+                }
+                else
+                {
+                    string error = await response.Content.ReadAsStringAsync();
+                    MainForm.Instance.AddToActionsList($"Failed to set {modeName} mode: {response.StatusCode} - {error}",Level.ERROR);
+                }
+            }
+            catch (Exception ex)
+            {
+                MainForm.Instance.AddToActionsList($"Error changing to {modeName} mode: {ex.Message}",Level.ERROR);
+            }
+        }
+
+
+        public static async Task SetDayModeAsync()
+        {
+            await ChangeDayNightModeAsync(1);
+        }
+
+        public static async Task SetNightModeAsync()
+        {
+            await ChangeDayNightModeAsync(2);
+        }
+
+        public static async Task SetAutoModeAsync()
+        {
+            await ChangeDayNightModeAsync(0);
+        }
+
+        private static async Task<string> LoginAsync(HttpClient client, string user, string password)
+        {
+            var payload = new
+            {
+                userId = user,
+                userPassword = Sha256(password)
+            };
+
+            HttpResponseMessage response = await client.PostAsJsonAsync("/app/v1/login", payload);
+
+            response.EnsureSuccessStatusCode();
+
+            JsonElement json = await response.Content.ReadFromJsonAsync<JsonElement>();
+
+            return json.GetProperty("token").GetString();
+        }
+
+        private static string Sha256(string input)
+        {
+            using SHA256 sha = SHA256.Create();
+            byte[] bytes = sha.ComputeHash(Encoding.UTF8.GetBytes(input));
+            return BitConverter.ToString(bytes).Replace("-", "").ToLower();
+        }
+    }
+}
+

MobileTests/MobileTests.cs

@@ -0,0 +1,83 @@
+using LibVLCSharp.WinForms;
+using System;
+using System.Threading.Tasks;
+
+namespace AiQ_GUI.Mobile_Tests
+{
+    public static class MobileTests
+    {
+        public static async Task RunPreTestAsync()
+        {
+
+            await MobileAPI.CheckFirmwareAsync();// Check firmware version
+
+            VLC.Play(MainForm.Instance.VidView);// Test Live Video streaming
+            await Task.Delay(5000);
+
+            VLC.TakeSnapshot(MainForm.Instance.VidView);
+
+            if (!VLC.IsPLaying(MainForm.Instance.VidView))// Check Live Video streaming
+            {
+                MainForm.Instance.AddToActionsList("Live Video streaming test failed: Video is not playing.", Level.ERROR);
+                MainForm.Instance.AddLabelToPanel("Live Video = Not Playing", true);
+            }
+            else
+            {
+                MainForm.Instance.AddLabelToPanel("Live Video = Playing", false);
+            }
+            
+            await TestDayNightMode();// Test day/night mode cycling
+
+            double dayLuminance = ImageProcessing.GetMeanLuminance(MainForm.Instance.DayImgPcbx.Image); // Calculates the Luminance
+            double nightLuminance = ImageProcessing.GetMeanLuminance(MainForm.Instance.NightImgPcbx.Image);
+
+            MainForm.Instance.DayImage.Text += $" - Luminance: {dayLuminance}%";
+            MainForm.Instance.NightImage.Text += $" - Luminance: {nightLuminance}%";
+
+            if (dayLuminance > nightLuminance)
+            {
+                MainForm.Instance.AddLabelToPanel("Day/Night Mode = Passed", false);
+            }
+            else
+            {
+                MainForm.Instance.AddToActionsList($"Day/Night mode test failed: Day luminance ({dayLuminance}%) is not greater than night luminance ({nightLuminance}%).", Level.ERROR);
+                MainForm.Instance.AddLabelToPanel("Day/Night Mode = Failed", true);
+
+            }
+        }
+        public static async Task RunFinalTestAsync()
+        {
+            // Placeholder for any final tests if needed in the future
+            //RunPreTestAsync().Wait();
+            //SSH.MobiletxtCheck(MainForm.Instance.CamOnTest.IP);// Verify mobile.txt presence
+            if (SSH.MobiletxtCheck(MainForm.Instance.CamOnTest.IP) == true)
+            {
+                MainForm.Instance.AddLabelToPanel("MobileSetup.sh = True", false);
+            }
+            else
+            {
+                MainForm.Instance.AddToActionsList("MobileSetup.sh test failed: mobile.txt not found on device.", Level.ERROR);
+                MainForm.Instance.AddLabelToPanel("MobileSetup.sh  = False", true);
+            }
+        }
+
+        private static async Task TestDayNightMode()
+        {
+            await MobileAPI.SetDayModeAsync();// Set to Day mode
+
+            await Task.Delay(5000);
+
+            VLC.TakeSnapshot(MainForm.Instance.VidView, "Mobile_day_snapshot.png");// Take Day Image Snapshot
+
+            MainForm.Instance.DayImgPcbx.Image = System.Drawing.Image.FromFile(Path.Combine(LDS.MAVPath, "Mobile_day_snapshot.png"));// Display Day mode snapshot
+
+            await MobileAPI.SetNightModeAsync();// Set to Night mode
+
+            await Task.Delay(5000);
+
+            VLC.TakeSnapshot(MainForm.Instance.VidView, "Mobile_night_snapshot.png");// Take Night Image Snapshot
+            MainForm.Instance.NightImgPcbx.Image = System.Drawing.Image.FromFile(Path.Combine(LDS.MAVPath, "Mobile_night_snapshot.png"));// Display Night mode snapshot
+        }
+    }
+}
+

MobileTests/VLC.cs

@@ -0,0 +1,43 @@
+using AiQ_GUI;
+using Emgu.CV.Dai;
+using LibVLCSharp.Shared;
+using LibVLCSharp.WinForms;
+using System.IO;
+using System.Security.Policy;
+
+internal class VLC
+{
+    public static LibVLC libVLC;
+
+    static VLC()
+    {
+        // Ensure LibVLC is initialized before first use
+        Core.Initialize();
+        libVLC = new LibVLC();
+    }
+
+    public static string SnapshotPath = Path.Combine(LDS.MAVPath, "Mobile_ov_snapshot.png");
+
+    public static void Play(VideoView VLCVV)
+    {
+        var media = new Media(libVLC, $"rtsp://ADMIN:1234@{MainForm.Instance.CamOnTest.IP}:554/live/main", FromType.FromLocation); 
+
+        VLCVV.MediaPlayer.Play(media);
+    }
+
+    public static bool IsPLaying(VideoView VLCVV)
+    {
+        return VLCVV.MediaPlayer.IsPlaying;
+    }
+
+    public static void TakeSnapshot(VideoView VLCVV, string filename = "Mobile_ov_snapshot.png")
+    {
+        string path = Path.Combine(LDS.MAVPath, filename);
+        VLCVV.MediaPlayer.TakeSnapshot(0, path, 1280, 720);
+    }
+
+    public static void Stop(VideoView VLCVV)
+    {
+        VLCVV.MediaPlayer.Stop();
+    }
+}

Network.cs

@@ -1,4 +1,6 @@
-using System.Net;
+using OnvifDiscovery;
+using OnvifDiscovery.Models;
+using System.Net;
 using System.Net.NetworkInformation;
 using System.Net.Sockets;
 using System.Text;
@@ -88,9 +90,15 @@ namespace AiQ_GUI
             const int sendPort = 6666;
             const int receivePort = 6667;
             const int discoveryTimeoutMs = 1000;
-            IList<string> FoundCams = [];
 
-            byte[] discoveryPacket = [0x50, 0x4f, 0x4c, 0x4c, 0xaf, 0xb0, 0xb3, 0xb3, 0xb6, 0x01, 0xa8, 0xc0, 0x0b, 0x1a, 0x00, 0x00];
+            List<string> FoundCams = [];
+            HashSet<string> discoveredIPs = new(StringComparer.OrdinalIgnoreCase);
+
+            byte[] discoveryPacket =
+            [
+                0x50, 0x4f, 0x4c, 0x4c, 0xaf, 0xb0, 0xb3, 0xb3,
+                0xb6, 0x01, 0xa8, 0xc0, 0x0b, 0x1a, 0x00, 0x00
+            ];
 
             async Task SendAndListen(IPAddress localIp)
             {
@@ -101,41 +109,42 @@ namespace AiQ_GUI
                     sender.Send(discoveryPacket, discoveryPacket.Length);
                 }
 
-                using UdpClient receiver = new(receivePort); // Listen for replies on fixed port
+                using UdpClient receiver = new(receivePort);
                 receiver.Client.ReceiveTimeout = discoveryTimeoutMs;
 
                 DateTime timeout = DateTime.Now.AddMilliseconds(discoveryTimeoutMs);
-                try
+
+                while (DateTime.Now < timeout)
                 {
-                    while (DateTime.Now < timeout)
+                    if (receiver.Available > 0)
                     {
-                        if (receiver.Available > 0)
+                        UdpReceiveResult result = await receiver.ReceiveAsync();
+                        byte[] recvBuffer = result.Buffer;
+
+                        if (recvBuffer.Length >= 52)
                         {
-                            UdpReceiveResult result = await receiver.ReceiveAsync();
+                            byte[] ipBytes = recvBuffer
-                            byte[] recvBuffer = result.Buffer;
+                                .Skip(recvBuffer.Length - 52)
+                                .Take(4)
+                                .Reverse()
+                                .ToArray();
 
-                            if (recvBuffer.Length >= 52) // Safety check
+                            string ip = string.Join(".", ipBytes);
-                            {
-                                byte[] ipBytes = recvBuffer.Skip(recvBuffer.Length - 52).Take(4).Reverse().ToArray();
-                                string ipToAdd = string.Join(".", ipBytes);
 
-                                if (!FoundCams.Contains(ipToAdd))
+                            if (discoveredIPs.Add(ip))
-                                    FoundCams.Add(ipToAdd);
+                                FoundCams.Add(ip);
-                            }
                         }
-
-                        await Task.Delay(50); // brief wait to allow data in
                     }
-                }
+
-                catch (SocketException ex) when (ex.SocketErrorCode == SocketError.TimedOut)
+                    await Task.Delay(50);
-                {
-                    // No data received in time — normal case
                 }
             }
 
-            // Get first IPv4 interface (non-loopback)
             foreach (IPAddress ip in Dns.GetHostEntry(Dns.GetHostName()).AddressList)
             {
+                if (ip.AddressFamily != AddressFamily.InterNetwork)
+                    continue;
+
                 try
                 {
                     await SendAndListen(ip);
@@ -143,9 +152,31 @@ namespace AiQ_GUI
                 catch { }
             }
 
+            //ONVIF discovery 
+            try
+            {
+                Discovery onvifDiscovery = new();
+
+                await foreach (DiscoveryDevice device in onvifDiscovery.DiscoverAsync(5))
+                {
+                    string ip = device.Address;
+
+                    // Already found via UDP → skip
+                    if (!discoveredIPs.Add(ip))
+                        continue;
+
+                    // ONVIF-only camera
+                    FoundCams.Add($"{ip} - Onvif");
+                }
+            }
+            catch
+            {
+            }
+
             return FoundCams;
         }
 
+
         // Ping to make sure devices are connected to the network, be aware it isn't consistant across subnets. 
         public async static Task<bool> PingIP(string ipAddress)
         {
@@ -157,9 +188,9 @@ namespace AiQ_GUI
                     PingReply reply = await myPing.SendPingAsync(ipAddress, 1000); // Timeout is 1s
                     return reply.Status == IPStatus.Success;
                 }
-                catch (PingException ex) { MainForm.Instance.AddToActionsList($"PingException: Unable to ping {ipAddress}. Reason: {ex.Message}"); }
+                catch (PingException ex) { MainForm.Instance.AddToActionsList($"PingException: Unable to ping {ipAddress}. Reason: {ex.Message}{Level.WARNING}"); }
-                catch (SocketException ex) { MainForm.Instance.AddToActionsList($"SocketException: Network error while pinging {ipAddress}. Reason: {ex.Message}"); }
+                catch (SocketException ex) { MainForm.Instance.AddToActionsList($"SocketException: Network error while pinging {ipAddress}. Reason: {ex.Message}{Level.ERROR}"); }
-                catch (Exception ex) { MainForm.Instance.AddToActionsList($"Unexpected error while pinging {ipAddress}: {ex.Message}"); }
+                catch (Exception ex) { MainForm.Instance.AddToActionsList($"Unexpected error while pinging {ipAddress}: {ex.Message}{Level.ERROR}"); }
             }
 
             return false;

Soak/Selenium.cs

@@ -19,7 +19,7 @@ namespace AiQ_GUI
             }
             catch (WebDriverTimeoutException)
             {
-                MainForm.Instance.AddToActionsList("Could not load web page " + url + "1. Check camera has no password set. " + Environment.NewLine + "2. If unable to fix speak to supervisor.");
+                MainForm.Instance.AddToActionsList($"Could not load web page " + url + $"1. Check camera has no password set. " + Environment.NewLine + $"2. If unable to fix speak to supervisor.", Level.ERROR);
             }
         }
 
@@ -71,11 +71,11 @@ namespace AiQ_GUI
 
                 loginBtn.Click();
 
-                MainForm.Instance.AddToActionsList("Switched user and logged in.");
+                MainForm.Instance.AddToActionsList($"Switched user and logged in.", Level.Success);
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("SwitchUser failed: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"SwitchUser failed:" + ex.Message,Level.ERROR);
             }
         }
 
@@ -99,7 +99,7 @@ namespace AiQ_GUI
                     ClickElementByID(elementID, driver, false);
                 }
 
-                MainForm.Instance.AddToActionsList("Could not click " + elementID);
+                MainForm.Instance.AddToActionsList($"Could not click " + elementID, Level.WARNING);
             }
         }
 
@@ -134,7 +134,7 @@ namespace AiQ_GUI
             }
             catch (Exception ex)
             {
-                MainForm.Instance.AddToActionsList("Failed to create ChromeDriver: " + ex.Message);
+                MainForm.Instance.AddToActionsList($"Failed to create ChromeDriver: " + ex.Message, Level.ERROR);
                 throw;
             }
         }

Soak/SoakTest.cs

@@ -37,7 +37,7 @@ namespace AiQ_GUI
                     catch (Exception ex)
                     {
                         SoakError($"Initial connection failed: {ex.Message}", SoakLogFile, CamInfo.CheckBox);
-                        MainForm.Instance.AddToActionsList($"[{CamInfo.IP}] Initial connection failed: {ex.Message}");
+                        MainForm.Instance.AddToActionsList($"[{CamInfo.IP}] Initial connection failed:{ex.Message}", Level.ERROR);
 
                         // Wait 10 seconds before trying again
                         await Task.Delay(TimeSpan.FromSeconds(10), token);
@@ -184,13 +184,13 @@ namespace AiQ_GUI
                     }
                     catch (Exception ex)
                     {
-                        MainForm.Instance.AddToActionsList($"Failed to find final test report: {ex.Message}");
+                        MainForm.Instance.AddToActionsList($"Failed to find final test report: {Level.ERROR}{ex.Message}");
                     }
 
                     // Link PDFs if both exist
                     if (!string.IsNullOrEmpty(finalTestPath) && File.Exists(finalTestPath) && !string.IsNullOrEmpty(SoakTestPath) && File.Exists(SoakTestPath))
                     {
-                        string outputPath = finalTestDir + $"Final&SoakTestReport_{CamInfo.Model}_{CamInfo.Serial}_{DateTime.Now.ToString("dd-MM-yyyy_HH-mm-ss")}.pdf";
+                        string outputPath = finalTestDir + $"Final&SoakTestReport_{CamInfo.Model}_{CamInfo.Serial}_{DateTime.Now.ToString("dd-MM-yyyy_HH-mm-ss")}.pdf{Level.Success}";
                         try
                         {
                             if (PDF.LinkPDFs(finalTestPath, SoakTestPath, outputPath)) // Delete the separate soak and final test reports if Linking was successful
@@ -200,7 +200,7 @@ namespace AiQ_GUI
                                 File.Delete(SoakTestPath);
                             }
                             else
-                                MainForm.Instance.AddToActionsList($"Failed to link or delete PDFs");
+                                MainForm.Instance.AddToActionsList($"Failed to link or delete PDFs ", Level.ERROR);
                         }
                         catch (Exception ex)
                         {
@@ -226,7 +226,7 @@ namespace AiQ_GUI
             if (ImageBright == null)
             {
                 Logging.LogWarningMessage($"Bright image is null for {controlType} at setting {SettingMinMax[0]}", SoakLogFile);
-                MainForm.Instance.AddToActionsList($"Bright image is null for {controlType} at setting {SettingMinMax[0]}");
+                MainForm.Instance.AddToActionsList($"Bright image is null for {controlType} at setting {SettingMinMax[0]}{Level.WARNING}");
                 return;
             }
 
@@ -240,7 +240,7 @@ namespace AiQ_GUI
             if (ImageDark == null)
             {
                 Logging.LogWarningMessage($"Dark image is null for {controlType} at setting {SettingMinMax[1]}", SoakLogFile);
-                MainForm.Instance.AddToActionsList($"Dark image is null for {controlType} at setting {SettingMinMax[1]}");
+                MainForm.Instance.AddToActionsList($"Dark image is null for {controlType} at setting {SettingMinMax[1]}", Level.WARNING);
                 return;
             }
 
@@ -251,7 +251,7 @@ namespace AiQ_GUI
             if (Bright_Lum < Dark_Lum * 1.01)
             {
                 SoakError(
-                    $"Insufficient luminance contrast. Bright: {Bright_Lum:F2}, Dark: {Dark_Lum:F2} | Type: {controlType} | Bright Setting: {SettingMinMax[0]}, Dark Setting: {SettingMinMax[1]}",
+                    $"Insufficient luminance contrast. Bright: {Bright_Lum:F2}, Dark: {Dark_Lum:F2} | Type: {controlType} | Bright Setting: {SettingMinMax[0]}, Dark Setting: {SettingMinMax[1]}{Level.WARNING}",
                     SoakLogFile,
                     CamInfo.CheckBox
                 );