Added test to verify reset hooks fire correctly

Author: ashblue

Assets/FluidBehaviorTree/Scripts/TaskParents/Composites/Editor/CompositeBaseTest.cs

@@ -1,16 +1,24 @@
 using Adnc.FluidBT.TaskParents.Composites;
-using Adnc.FluidBT.TaskParents;
+using Adnc.FluidBT.Tasks;
 using NSubstitute;
 using NUnit.Framework;
+using UnityEngine;
+using Tree = Adnc.FluidBT.Trees.Tree;
 
 namespace Adnc.FluidBT.Testing {
     public class CompositeBaseTest {
         private CompositeExample _composite;
 
         public class CompositeExample : CompositeBase {
+            public TaskStatus status;
+            
             public void SetChildIndex (int index) {
                 ChildIndex = index;
             }
+
+            protected override TaskStatus OnUpdate () {
+                return status;
+            }
         }
 
         [SetUp]
@@ -48,6 +56,18 @@ public void Calls_end_on_current_child () {
         }
 
         public class ResetMethod : CompositeBaseTest {
+            private GameObject _go;
+                
+            [SetUp]
+            public void BeforEach () {
+                _go = new GameObject();
+            }
+
+            [TearDown]
+            public void AfterEach () {
+                Object.DestroyImmediate(_go);
+            }
+            
             [Test]
             public void Resets_child_node_pointer () {
                 _composite.SetChildIndex(2);
@@ -56,6 +76,34 @@ public void Resets_child_node_pointer () {
 
                 Assert.AreEqual(0, _composite.ChildIndex);
             }
+
+            [Test]
+            public void Resets_pointer_if_tick_count_changes () {
+                var tree = new Tree(_go);
+                tree.AddNode(tree.Root, _composite);
+                tree.AddNode(_composite, Substitute.For<ITask>());
+                    
+                tree.Tick();
+                _composite.SetChildIndex(2);
+                tree.Tick();
+                    
+                Assert.AreEqual(0, _composite.ChildIndex);
+            }
+
+            [Test]
+            public void Does_not_reset_pointer_if_tick_count_does_not_change () {
+                var task = Substitute.For<ITask>();
+                var tree = new Tree(_go);
+                tree.AddNode(tree.Root, _composite);
+                tree.AddNode(_composite, task);
+                _composite.status = TaskStatus.Continue;
+                    
+                tree.Tick();
+                _composite.SetChildIndex(1);
+                tree.Tick();
+                    
+                Assert.AreEqual(1, _composite.ChildIndex);
+            }
         }
     }
 }

Assets/FluidBehaviorTree/Scripts/Tasks/Editor/TaskTest.cs

@@ -1,6 +1,8 @@
 using Adnc.FluidBT.Tasks;
 using Adnc.FluidBT.Tasks.Actions;
 using NUnit.Framework;
+using UnityEngine;
+using Tree = Adnc.FluidBT.Trees.Tree;
 
 namespace Adnc.FluidBT.Testing {
     public class TaskTest {
@@ -35,6 +37,44 @@ public void SetUpNode () {
                 node = new TaskExample();
             }
 
+            public class TreeTickCountChange : UpdateMethod {
+                private GameObject _go;
+                
+                [SetUp]
+                public void BeforEach () {
+                    _go = new GameObject();
+                }
+
+                [TearDown]
+                public void AfterEach () {
+                    Object.DestroyImmediate(_go);
+                }
+                
+                [Test]
+                public void Retriggers_start_if_tick_count_changes () {
+                    var tree = new Tree(_go);
+                    tree.AddNode(tree.Root, node);
+                    
+                    tree.Tick();
+                    tree.Tick();
+                    
+                    Assert.AreEqual(2, node.StartCount);
+                }
+                
+                [Test]
+                public void Does_not_retrigger_start_if_tick_count_stays_the_same () {
+                    node.status = TaskStatus.Continue;
+                    
+                    var tree = new Tree(_go);
+                    tree.AddNode(tree.Root, node);
+                    
+                    tree.Tick();
+                    tree.Tick();
+                    
+                    Assert.AreEqual(1, node.StartCount);
+                }
+            }
+
             public class StartEvent : UpdateMethod {
                 [Test]
                 public void Trigger_on_1st_run () {