fix(test): address PR review comments

- JEngineTestBase: save and restore LogAssert.ignoreFailingMessages state
- JEngineTestBase: add XML documentation for virtual methods
- PanelUITests: remove unused _panel field and cleanup code
- PanelUITests: use discard pattern for unused root assignments
- BootstrapEditorUITests: use discard pattern for unused root assignments
- BuildManager: use LINQ Any() for IsIgnoredError filtering

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: JasonXuDeveloper - 傑 <jason@xgamedev.net>

Author: JasonXuDeveloper

UnityProject/Packages/com.jasonxudeveloper.jengine.core/Editor/CustomEditor/BuildManager.cs

@@ -26,6 +26,7 @@
 using System;
 using System.Collections.Generic;
 using System.IO;
+using System.Linq;
 using HybridCLR.Editor;
 using HybridCLR.Editor.Commands;
 using HybridCLR.Editor.Settings;
@@ -551,14 +552,7 @@ private int CountRealErrors(int recentErrorCount)
 
         private static bool IsIgnoredError(string errorMessage)
         {
-            foreach (var ignoredError in IgnoredErrors)
-            {
-                if (errorMessage.Contains(ignoredError))
-                {
-                    return true;
-                }
-            }
-            return false;
+            return IgnoredErrors.Any(ignoredError => errorMessage.Contains(ignoredError));
         }
 
         private int GetUnityErrorCount()

UnityProject/Packages/com.jasonxudeveloper.jengine.ui/Tests/Editor/Internal/BootstrapEditorUITests.cs

@@ -389,7 +389,7 @@ public void EncryptionDropdown_AllOptions_UpdateConfigFields(EncryptionOption op
             _bootstrap.encryptionOption = option;
             _serializedObject.Update();
 
-            var root = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
+            _ = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
 
             var bundleConfig = EncryptionMapping.GetBundleConfig(option);
 
@@ -521,9 +521,9 @@ public void CreateInspector_MultipleCalls_DoesNotThrow()
         {
             Assert.DoesNotThrow(() =>
             {
-                var root1 = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
-                var root2 = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
-                var root3 = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
+                _ = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
+                _ = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
+                _ = BootstrapEditorUI.CreateInspector(_serializedObject, _bootstrap);
             });
         }