Fix: pipeline breaking

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/ActionRunnerTests.cs

@@ -5,6 +5,7 @@
 using NSubstitute;
 using NSubstitute.ReturnsExtensions;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
@@ -32,6 +33,11 @@ public void SetUp()
             this.proxy = Substitute.For<IAgentProxy>();
 
             this.actionRunner = new ActionRunner(this.agent, this.proxy);
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/AgentBehaviourTests.cs

@@ -8,6 +8,7 @@
 using FluentAssertions;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 using UnityEngine;
 
 namespace CrashKonijn.Goap.UnitTests
@@ -32,6 +33,11 @@ public void Setup()
             this.provider.Receiver = this.agent;
 
             this.agent.ActionProvider = this.provider;
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/AgentTypeJobRunnerTests.cs

@@ -7,6 +7,7 @@
 using NSubstitute;
 using NSubstitute.ReturnsExtensions;
 using NUnit.Framework;
+using Unity.Collections;
 using UnityEngine;
 using ICondition = CrashKonijn.Goap.Core.ICondition;
 
@@ -30,6 +31,11 @@ public void Setup()
             this.agentType = Substitute.For<IAgentType>();
 
             this.goapActionProvider.AgentType.Returns(this.agentType);
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/ClassResolverTests.cs

@@ -5,6 +5,7 @@
 using CrashKonijn.Goap.UnitTests.Support;
 using FluentAssertions;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
@@ -16,6 +17,11 @@ public class ClassResolverTests
         public void Init()
         {
             this.factory.Setup<ClassResolver>();
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/ConditionObserverTests.cs

@@ -1,11 +1,21 @@
 using CrashKonijn.Goap.Core;
 using CrashKonijn.Goap.Runtime;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
     public class ConditionObserverTests
     {
+        [SetUp]
+        public void Init()
+        {
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
+        }
+
         [Test]
         public void IsMet_Positive_IsPresent()
         {

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/GoapActionBaseTests.cs

@@ -4,6 +4,7 @@
 using CrashKonijn.Goap.Runtime;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 using UnityEngine;
 
 namespace CrashKonijn.Goap.UnitTests
@@ -59,6 +60,15 @@ public class Data : IActionData
                 public ITarget Target { get; set; }
             }
         }
+
+        [SetUp]
+        public void Init()
+        {
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
+        }
 
         [TestCase(false)]
         [TestCase(true)]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/GraphResolverTests.cs

@@ -68,6 +68,15 @@ public T ToMock()
             }
         }
 
+        [SetUp]
+        public void Init()
+        {
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
+        }
+
         [Test]
         public void Resolve_WithNoActions_ReturnsEmptyList()
         {

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/ManualControllerTests.cs

@@ -3,6 +3,7 @@
 using CrashKonijn.Goap.Runtime;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
@@ -23,6 +24,11 @@ public void SetUp()
 
             this.manualController = new ManualController();
             this.manualController.Initialize(this.goap);
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/MultiSensorBaseTests.cs

@@ -4,6 +4,7 @@
 using CrashKonijn.Goap.Runtime;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
@@ -29,6 +30,11 @@ public void SetUp()
             this.mockWorldData = Substitute.For<IWorldData>();
             this.mockMonoAgent = Substitute.For<IMonoAgent>();
             this.mockComponentReference = Substitute.For<IComponentReference>();
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/ProactiveControllerTests.cs

@@ -3,6 +3,7 @@
 using CrashKonijn.Goap.Runtime;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
@@ -40,6 +41,11 @@ public void SetUp()
             this.goap.Events.Returns(this.events);
 
             this.proactiveController = new ProactiveController();
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/ReactiveControllerTests.cs

@@ -3,6 +3,7 @@
 using CrashKonijn.Goap.Runtime;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
@@ -39,6 +40,11 @@ public void SetUp()
             this.goap.Events.Returns(this.events);
 
             this.controller = new ReactiveController();
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/SensorRunnerTests.cs

@@ -6,11 +6,21 @@
 using CrashKonijn.Goap.UnitTests.Classes;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
     public class SensorRunnerTests
     {
+        [SetUp]
+        public void Init()
+        {
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
+        }
+
         [Test]
         public void SenseGlobal_GlobalSensor_CallsSense()
         {

Package/Tests/CrashKonijn.Goap.Tests/UnitTests/WorldDataBaseTests.cs

@@ -4,6 +4,7 @@
 using CrashKonijn.Goap.Runtime;
 using NSubstitute;
 using NUnit.Framework;
+using Unity.Collections;
 
 namespace CrashKonijn.Goap.UnitTests
 {
@@ -16,6 +17,11 @@ public class WorldDataBaseTests
         public void SetUp()
         {
             this.worldDataBase = new LocalWorldData();
+
+            // Unity sometimes thinks that a temporary job is leaking memory
+            // This is not the case, so we ignore the message
+            // This can trigger in any test, even the ones that don't use the Job system
+            NativeLeakDetection.Mode = NativeLeakDetectionMode.Disabled;
         }
 
         [Test]