feat: Logger tests (MAPCO-9054)

Assets/EditorTests/Logger/LoggerServiceConfigTests.cs

@@ -0,0 +1,19 @@
+using com.mapcolonies.core.Services.LoggerService;
+using NUnit.Framework;
+
+namespace EditorTests.Logger
+{
+    public class LoggerServiceConfigTests
+    {
+        [Test]
+        public void Init_WhenJsonExists_LoadsBasicProperties()
+        {
+            LoggerServiceConfig config = new LoggerServiceConfig();
+            config.Init();
+            Assert.IsTrue(config.ServiceEnabled);
+            Assert.IsTrue(config.EnableConsole);
+            Assert.AreEqual("Logger/log4net.xml", config.Log4NetConfigXml);
+            Assert.AreEqual("DEBUG", config.MinLogLevel);
+        }
+    }
+}

Assets/EditorTests/Logger/LoggerServiceIntegrationTests.cs

@@ -0,0 +1,45 @@
+using System.IO;
+using com.mapcolonies.core.Services.LoggerService;
+using NUnit.Framework;
+using UnityEngine;
+
+namespace EditorTests.Logger
+{
+    public class LoggerServiceIntegrationTests
+    {
+        [Test]
+        public void DebugLog_WritesToInfoLogFile()
+        {
+            LoggerServiceConfig config = new LoggerServiceConfig();
+            config.Init();
+
+            Assert.IsTrue(config.ServiceEnabled, "ServiceEnabled must be true for this integration test.");
+            Assert.IsFalse(string.IsNullOrEmpty(config.Log4NetConfigXml), "Log4NetConfigXml must point to a valid XML file.");
+
+            string logsDir = config.GetSystemLogsDirectory();
+            Directory.CreateDirectory(logsDir);
+
+            foreach (string file in Directory.GetFiles(logsDir, "info.log*"))
+            {
+                File.Delete(file);
+            }
+
+            ILogHandler originalHandler = Debug.unityLogger.logHandler;
+
+            using (new LoggerService(config))
+            {
+                Assert.IsInstanceOf<Log4NetHandler>(Debug.unityLogger.logHandler, "LoggerService must replace Unity's log handler with Log4NetHandler when initialized.");
+
+                Debug.Log("IntegrationTest: DebugLog_WritesToInfoLogFile");
+            }
+
+            Debug.unityLogger.logHandler = originalHandler;
+
+            string[] infoFiles = Directory.GetFiles(logsDir, "info.log*");
+            Assert.IsNotEmpty(infoFiles, $"No info.log* files were found after logging. Log directory: {logsDir}");
+
+            FileInfo fi = new FileInfo(infoFiles[0]);
+            Assert.Greater(fi.Length, 0, $"The log file '{fi.FullName}' is empty, but it should contain at least one log entry.");
+        }
+    }
+}

Assets/EditorTests/Logger/LoggerServiceTests.cs

@@ -0,0 +1,45 @@
+using com.mapcolonies.core.Services.LoggerService;
+using NUnit.Framework;
+using UnityEngine;
+
+namespace EditorTests.Logger
+{
+    public class LoggerServiceTests
+    {
+        private ILogHandler _originalHandler;
+
+        [SetUp]
+        public void SetUp()
+        {
+            _originalHandler = Debug.unityLogger.logHandler;
+        }
+
+        [TearDown]
+        public void TearDown()
+        {
+            Debug.unityLogger.logHandler = _originalHandler;
+        }
+
+        [Test]
+        public void Ctor_WhenConfigValid_ReplacesUnityLogHandlerWithLog4NetHandler()
+        {
+            LoggerServiceConfig config = new LoggerServiceConfig();
+            config.Init();
+
+            ILogHandler before = Debug.unityLogger.logHandler;
+            LoggerService loggerService = null;
+
+            try
+            {
+                loggerService = new LoggerService(config);
+                Assert.IsInstanceOf<Log4NetHandler>(Debug.unityLogger.logHandler);
+                Assert.IsTrue(Debug.unityLogger.logEnabled);
+            }
+            finally
+            {
+                loggerService?.Dispose();
+                Assert.AreSame(before, Debug.unityLogger.logHandler);
+            }
+        }
+    }
+}

Assets/PlayModeTests/Logger/LoggerInitializerPlayModeTests.cs

@@ -0,0 +1,49 @@
+using System.Collections;
+using System.Reflection;
+using com.mapcolonies.core.Services.LoggerService;
+using NUnit.Framework;
+using UnityEngine;
+using UnityEngine.TestTools;
+
+namespace PlayModeTests.Logger
+{
+    public class LoggerInitializerPlayModeTests
+    {
+        private ILogHandler _originalHandler;
+
+        [SetUp]
+        public void SetUp()
+        {
+            _originalHandler = Debug.unityLogger.logHandler;
+            Log4NetHandler.ApplicationDataPath = null;
+            Log4NetHandler.UnityVersion = null;
+        }
+
+        [TearDown]
+        public void TearDown()
+        {
+            TryCallDispose();
+            Debug.unityLogger.logHandler = _originalHandler;
+        }
+
+        [UnityTest]
+        public IEnumerator Init_SetsHandlerAndStaticFields_AndDisposeRestoresHandler()
+        {
+            LoggerInitializer.Init();
+            yield return null;
+
+            Assert.IsInstanceOf<Log4NetHandler>(Debug.unityLogger.logHandler);
+            Assert.AreEqual(Application.dataPath, Log4NetHandler.ApplicationDataPath);
+            Assert.NotNull(Log4NetHandler.UnityVersion);
+
+            TryCallDispose();
+            Assert.AreSame(_originalHandler, Debug.unityLogger.logHandler);
+        }
+
+        private void TryCallDispose()
+        {
+            MethodInfo m = typeof(LoggerInitializer).GetMethod("Dispose", BindingFlags.NonPublic | BindingFlags.Static);
+            m?.Invoke(null, null);
+        }
+    }
+}