Add test related to issue #539

Author: facontidavide

tests/gtest_postconditions.cpp

@@ -37,3 +37,28 @@ TEST(PostConditions, BasicTest)
   ASSERT_EQ(tree.rootBlackboard()->get<int>("C"), 42);
   ASSERT_EQ(tree.rootBlackboard()->get<int>("D"), 42);
 }
+
+TEST(PostConditions, Issue539)
+{
+  const std::string xml_text = R"(
+    <root BTCPP_format="4" >
+      <BehaviorTree ID="MainTree">
+        <Sequence>
+          <Script code = "x:=0; y:=0" />
+          <RetryUntilSuccessful num_attempts="5">
+            <AlwaysFailure _onFailure="x  += 1"  _post="y  += 1" />
+          </RetryUntilSuccessful>
+        </Sequence>
+      </BehaviorTree>
+    </root>)";
+
+  BehaviorTreeFactory factory;
+  auto tree = factory.createTreeFromText(xml_text);
+  const auto status = tree.tickWhileRunning();
+  ASSERT_EQ(status, NodeStatus::FAILURE);
+
+  ASSERT_EQ(tree.rootBlackboard()->get<int>("x"), 5);
+  ASSERT_EQ(tree.rootBlackboard()->get<int>("y"), 5);
+}
+
+

tests/script_parser_test.cpp

@@ -120,7 +120,13 @@ TEST(ParserTest, Equations)
   EXPECT_EQ(GetResult("x+=1").cast<double>(), 4.0);
   EXPECT_EQ(variables->get<double>("x"), 4.0);
 
-  EXPECT_EQ(GetResult("x-=1").cast<double>(), 3.0);
+  EXPECT_EQ(GetResult("x += 1").cast<double>(), 5.0);
+  EXPECT_EQ(variables->get<double>("x"), 5.0);
+
+  EXPECT_EQ(GetResult("x-=1").cast<double>(), 4.0);
+  EXPECT_EQ(variables->get<double>("x"), 4.0);
+
+  EXPECT_EQ(GetResult("x -= 1").cast<double>(), 3.0);
   EXPECT_EQ(variables->get<double>("x"), 3.0);
 
   EXPECT_EQ(GetResult("x*=2").cast<double>(), 6.0);