Fix outputRootPath when outputRootPath is null and on player

Runtime/Utilities/PathUtils.cs

@@ -17,6 +17,11 @@ internal static class PathUtils
         /// <returns>Absolute path.</returns>
         public static string GetAbsolutePath(string path, string basePath)
         {
+            if (string.IsNullOrEmpty(path))
+            {
+                return basePath;
+            }
+
 #if UNITY_2021_2_OR_NEWER
             if (Path.IsPathFullyQualified(path))
             {

Tests/Runtime/Settings/AutopilotSettingsTest.cs

@@ -8,6 +8,7 @@
 using DeNA.Anjin.TestDoubles;
 using NUnit.Framework;
 using UnityEngine;
+using UnityEngine.TestTools;
 
 namespace DeNA.Anjin.Settings
 {
@@ -67,6 +68,56 @@ public void ExitCode_CustomButNotValid_ReturnsExitCodeWhenLifespanExpiredCustom(
             Assert.That(settings.ExitCode, Is.EqualTo((ExitCode)ExitCodeWhenLifespanExpired.Custom));
         }
 
+        [Test]
+        public void OutputDataPath_AbsolutePath_ReturnsAbsolutePath()
+        {
+            var settings = CreateAutopilotSettings();
+            settings.outputRootPath = "/OutputRootPath";
+
+            Assert.That(settings.OutputRootPath, Is.EqualTo("/OutputRootPath"));
+        }
+
+        [Test]
+        [UnityPlatform(RuntimePlatform.OSXEditor, RuntimePlatform.WindowsEditor, RuntimePlatform.LinuxEditor)]
+        public void OutputDataPath_RelativePathInEditor_ReturnsAbsolutePathBasedOnProjectRoot()
+        {
+            var settings = CreateAutopilotSettings();
+            settings.outputRootPath = "OutputRootPath";
+
+            Assert.That(settings.OutputRootPath, Is.EqualTo(Path.GetFullPath("OutputRootPath")));
+        }
+
+        [Test]
+        [UnityPlatform(RuntimePlatform.OSXPlayer, RuntimePlatform.WindowsPlayer, RuntimePlatform.LinuxPlayer)]
+        public void OutputDataPath_RelativePathOnPlayer_ReturnsAbsolutePathBasedOnPersistentDataPath()
+        {
+            var settings = CreateAutopilotSettings();
+            settings.outputRootPath = "OutputRootPath";
+
+            Assert.That(settings.OutputRootPath,
+                Is.EqualTo(Path.Combine(Application.persistentDataPath, "OutputRootPath")));
+        }
+
+        [Test]
+        [UnityPlatform(RuntimePlatform.OSXEditor, RuntimePlatform.WindowsEditor, RuntimePlatform.LinuxEditor)]
+        public void OutputDataPath_NullInEditor_ReturnsProjectRootPath()
+        {
+            var settings = CreateAutopilotSettings();
+            settings.outputRootPath = null;
+
+            Assert.That(settings.OutputRootPath, Is.EqualTo(Path.GetFullPath(".")));
+        }
+
+        [Test]
+        [UnityPlatform(RuntimePlatform.OSXPlayer, RuntimePlatform.WindowsPlayer, RuntimePlatform.LinuxPlayer)]
+        public void OutputDataPath_NullOnPlayer_ReturnsPersistentDataPath()
+        {
+            var settings = CreateAutopilotSettings();
+            settings.outputRootPath = null;
+
+            Assert.That(settings.OutputRootPath, Is.EqualTo(Application.persistentDataPath));
+        }
+
         [Test]
         public void OverwriteByCommandLineArguments_HasNotCommandlineArguments_KeepScriptableObjectValues()
         {

Tests/Runtime/Utilities/PathUtilsTest.cs

@@ -48,6 +48,18 @@ public void GetAbsolutePath_Empty_ReturnsBasePath()
             Assert.That(actual, Is.EqualTo(BasePath));
         }
 
+        [Test]
+        [UnityPlatform(RuntimePlatform.OSXEditor, RuntimePlatform.OSXPlayer, RuntimePlatform.LinuxEditor,
+            RuntimePlatform.LinuxPlayer)]
+        public void GetAbsolutePath_Null_ReturnsBasePath()
+        {
+            const string Path = null;
+            const string BasePath = "/Base/Path";
+
+            var actual = PathUtils.GetAbsolutePath(Path, BasePath);
+            Assert.That(actual, Is.EqualTo(BasePath));
+        }
+
         [Test]
         [UnityPlatform(RuntimePlatform.WindowsEditor, RuntimePlatform.WindowsPlayer)]
         public void GetAbsolutePath_WindowsAbsolutePath_ReturnsPath()