Fixed compilation errors in tests during compilation player's build

Author: IvanMurzak

Assets/_PackageRoot/Tests/Base/Extensions.Unity.ImageLoader.Tests.asmdef

@@ -8,12 +8,14 @@
     "includePlatforms": [],
     "excludePlatforms": [],
     "allowUnsafeCode": false,
-    "overrideReferences": false,
+    "overrideReferences": true,
     "precompiledReferences": [
-        "NSubstitute.dll"
+        "nunit.framework.dll"
+    ],
+    "autoReferenced": false,
+    "defineConstraints": [
+        "UNITY_INCLUDE_TESTS"
     ],
-    "autoReferenced": true,
-    "defineConstraints": [],
     "versionDefines": [],
     "noEngineReferences": false
 }

Assets/_PackageRoot/Tests/Base/Utils/TestUtils.LoadFail.cs

@@ -28,11 +28,10 @@ public static IEnumerator LoadFail(string url, FutureLoadingFrom? expectedLoadin
 
             var task1 = future.AsTask();
 
-// TODO: To find a way to use it in a player build
-#if UNITY_EDITOR
+
             if (expectedLoadingFrom.HasValue && expectedLoadingFrom.Value == FutureLoadingFrom.Source) // exception should be thrown only if ONLY loading from Source
                 UnityEngine.TestTools.LogAssert.Expect(LogType.Error, $"[ImageLoader] Future[id={future.Id}] Timeout ({timeout}): {url}"); // compilation error in player build
-#endif
+
             yield return TestUtils.WaitWhile(() => future.IsInProgress, TimeSpan.FromSeconds(10));
             var task2 = future.AsTask();
 

Assets/_PackageRoot/Tests/Base/Utils/TestUtils.cs

@@ -31,21 +31,17 @@ public static IEnumerator ClearEverything(string message)
         {
             if (message != null)
                 Debug.Log(message.PadRight(50, '-'));
-// TODO: To find a way to use it in a player build
-#if UNITY_EDITOR
+
             UnityEngine.TestTools.LogAssert.ignoreFailingMessages = true; // compilation error in player build
-#endif
+
             UniTaskScheduler.UnobservedExceptionWriteLogType = LogType.Exception;
             ImageLoader.ClearSpriteRef();
             ImageLoader.ClearTextureRef();
             yield return ImageLoader.ClearCacheAll().TimeoutCoroutine(TimeSpan.FromSeconds(10));
 
             WaitForGCFast();
 
-// TODO: To find a way to use it in a player build
-#if UNITY_EDITOR
             UnityEngine.TestTools.LogAssert.ignoreFailingMessages = false; // compilation error in player build
-#endif
         }
         public static void WaitForGCFast()
         {

Assets/_PackageRoot/Tests/Editor/Extensions.Unity.ImageLoader.Tests.Editor.asmdef

@@ -12,12 +12,14 @@
     ],
     "excludePlatforms": [],
     "allowUnsafeCode": false,
-    "overrideReferences": false,
+    "overrideReferences": true,
     "precompiledReferences": [
-        "NSubstitute.dll"
+        "nunit.framework.dll"
+    ],
+    "autoReferenced": false,
+    "defineConstraints": [
+        "UNITY_INCLUDE_TESTS"
     ],
-    "autoReferenced": true,
-    "defineConstraints": [],
     "versionDefines": [],
     "noEngineReferences": false
 }

Assets/_PackageRoot/Tests/Runtime/Extensions.Unity.ImageLoader.Tests.Runtime.asmdef

@@ -9,12 +9,14 @@
     "includePlatforms": [],
     "excludePlatforms": [],
     "allowUnsafeCode": false,
-    "overrideReferences": false,
+    "overrideReferences": true,
     "precompiledReferences": [
-        "NSubstitute.dll"
+        "nunit.framework.dll"
+    ],
+    "autoReferenced": false,
+    "defineConstraints": [
+        "UNITY_INCLUDE_TESTS"
     ],
-    "autoReferenced": true,
-    "defineConstraints": [],
     "versionDefines": [],
     "noEngineReferences": false
 }