fix unit test

Author: Davide Faconti

src/controls/reactive_fallback.cpp

@@ -18,7 +18,6 @@ namespace BT
 NodeStatus ReactiveFallback::tick()
 {
     size_t failure_count = 0;
-    size_t running_count = 0;
 
     for (size_t index = 0; index < childrenCount(); index++)
     {
@@ -29,7 +28,7 @@ NodeStatus ReactiveFallback::tick()
         {
             case NodeStatus::RUNNING:
             {
-                haltChildren(0);
+                haltChildren(index+1);
                 return NodeStatus::RUNNING;
             }
 

tests/gtest_fallback.cpp

@@ -38,14 +38,14 @@ struct SimpleFallbackTest : testing::Test
     }
 };
 
-struct ParallelOneTest : testing::Test
+struct ReactiveFallbackTest : testing::Test
 {
     BT::ReactiveFallback root;
     BT::ConditionTestNode condition_1;
     BT::ConditionTestNode condition_2;
     BT::AsyncActionTest action_1;
 
-    ParallelOneTest()
+    ReactiveFallbackTest()
       : root("root_first")
       , condition_1("condition_1")
       , condition_2("condition_2")
@@ -55,7 +55,7 @@ struct ParallelOneTest : testing::Test
         root.addChild(&condition_2);
         root.addChild(&action_1);
     }
-    ~ParallelOneTest()
+    ~ReactiveFallbackTest()
     {
         haltAllActions(&root);
     }
@@ -152,7 +152,7 @@ TEST_F(SimpleFallbackTest, ConditionChangeWhileRunning)
     ASSERT_EQ(NodeStatus::RUNNING, action.status());
 }
 
-TEST_F(ParallelOneTest, Condition1ToTrue)
+TEST_F(ReactiveFallbackTest, Condition1ToTrue)
 {
     condition_1.setBoolean(false);
     condition_2.setBoolean(false);
@@ -174,7 +174,7 @@ TEST_F(ParallelOneTest, Condition1ToTrue)
     ASSERT_EQ(NodeStatus::IDLE, action_1.status());
 }
 
-TEST_F(ParallelOneTest, Condition2ToTrue)
+TEST_F(ReactiveFallbackTest, Condition2ToTrue)
 {
     condition_1.setBoolean(false);
     condition_2.setBoolean(false);