Refactor test utilities to improve compatibility with player builds

Author: IvanMurzak

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

@@ -3,7 +3,6 @@
 using Cysharp.Threading.Tasks;
 using NUnit.Framework;
 using UnityEngine;
-using UnityEngine.TestTools;
 
 namespace Extensions.Unity.ImageLoader.Tests.Utils
 {
@@ -29,10 +28,10 @@ public static IEnumerator LoadFail(string url, FutureLoadingFrom? expectedLoadin
 
             var task1 = future.AsTask();
 
-// TODO: To find a way to use it a player build
+// 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
-                LogAssert.Expect(LogType.Error, $"[ImageLoader] Future[id={future.Id}] Timeout ({timeout}): {url}"); // compilation error in player build
+                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

@@ -4,7 +4,6 @@
 using System.Linq;
 using Cysharp.Threading.Tasks;
 using UnityEngine;
-using UnityEngine.TestTools;
 
 namespace Extensions.Unity.ImageLoader.Tests.Utils
 {
@@ -32,9 +31,9 @@ public static IEnumerator ClearEverything(string message)
         {
             if (message != null)
                 Debug.Log(message.PadRight(50, '-'));
-// TODO: To find a way to use it a player build
+// TODO: To find a way to use it in a player build
 #if UNITY_EDITOR
-            LogAssert.ignoreFailingMessages = true; // compilation error in player build
+            UnityEngine.TestTools.LogAssert.ignoreFailingMessages = true; // compilation error in player build
 #endif
             UniTaskScheduler.UnobservedExceptionWriteLogType = LogType.Exception;
             ImageLoader.ClearSpriteRef();
@@ -43,9 +42,9 @@ public static IEnumerator ClearEverything(string message)
 
             WaitForGCFast();
 
-// TODO: To find a way to use it a player build
+// TODO: To find a way to use it in a player build
 #if UNITY_EDITOR
-            LogAssert.ignoreFailingMessages = false; // compilation error in player build
+            UnityEngine.TestTools.LogAssert.ignoreFailingMessages = false; // compilation error in player build
 #endif
         }
         public static void WaitForGCFast()

Assets/_PackageRoot/Tests/Runtime/TestConcurrency.cs

@@ -3,10 +3,10 @@
 using System.Collections;
 using NUnit.Framework;
 using Cysharp.Threading.Tasks;
-using UnityEngine.TestTools;
 using UnityEngine;
 using System.Threading.Tasks;
 using Extensions.Unity.ImageLoader.Tests.Utils;
+using UnityEngine.TestTools;
 
 namespace Extensions.Unity.ImageLoader.Tests
 {