test(decorators): move a repeater test to the DecoratorTest class

Now that DecoratorBase returns failure if it has no child, Does_not_crash_if_no_child can be moved
to the base decorator tests

Author: JesseTG

Assets/FluidBehaviorTree/Tests/Editor/Decorators/DecoratorTest.cs

@@ -1,4 +1,4 @@
-using CleverCrow.Fluid.BTs.Decorators;
+using CleverCrow.Fluid.BTs.Decorators;
 using CleverCrow.Fluid.BTs.Tasks;
 using NSubstitute;
 using NUnit.Framework;
@@ -50,6 +50,13 @@ public void It_should_return_failure_if_a_child_is_missing () {
                 _decorator.Children.RemoveAt(0);
                 Assert.AreEqual(TaskStatus.Failure, _decorator.Update());
             }
+
+            [Test]
+            public void Does_not_crash_if_no_child () {
+                _decorator.Children.RemoveAt(0);
+
+                Assert.DoesNotThrow(() => _decorator.Update());
+            }
         }
 
         public class EndMethod : DecoratorTest {

Assets/FluidBehaviorTree/Tests/Editor/Decorators/RepeatForeverTest.cs

@@ -28,13 +28,6 @@ public void Returns_continue_on_child_continue () {
 
                 Assert.AreEqual(TaskStatus.Continue, repeater.Update());
             }
-
-            [Test]
-            public void Does_not_crash_if_no_child () {
-                var repeater = new RepeatForever();
-
-                Assert.DoesNotThrow(() => repeater.Update());
-            }
         }
     }
 }

Assets/FluidBehaviorTree/Tests/Editor/Decorators/RepeatUntilFailureTest.cs

@@ -28,13 +28,6 @@ public void Returns_continue_on_child_continue () {
 
                 Assert.AreEqual(TaskStatus.Continue, repeater.Update());
             }
-
-            [Test]
-            public void Does_not_crash_if_no_child () {
-                var repeater = new RepeatUntilFailure();
-
-                Assert.DoesNotThrow(() => repeater.Update());
-            }
         }
     }
 }

Assets/FluidBehaviorTree/Tests/Editor/Decorators/RepeatUntilSuccessTest.cs

@@ -28,13 +28,6 @@ public void Returns_continue_on_child_continue () {
 
                 Assert.AreEqual(TaskStatus.Continue, repeater.Update());
             }
-
-            [Test]
-            public void Does_not_crash_if_no_child () {
-                var repeater = new RepeatUntilSuccess();
-
-                Assert.DoesNotThrow(() => repeater.Update());
-            }
         }
     }
 }