Applies consistent code formatting

Adjusts line wrapping and whitespace across examples, headers, source files, and tests
to enhance readability and maintain a unified code style.

Author: pepeRossRobotics

examples/t15_nodes_mocking.cpp

@@ -104,10 +104,9 @@ int main(int /*argc*/, char** /*argv*/)
   // This is the configuration passed to the TestNode
   BT::TestNodeConfig test_config;
   // the returned status can also be computed from a script.
-    // Change mock_should_fail to true in the XML above to see the failure path.
+  // Change mock_should_fail to true in the XML above to see the failure path.
   test_config.return_status.reset();
-  test_config.return_status_script =
-      "(mock_should_fail == true) ? FAILURE : SUCCESS";
+  test_config.return_status_script = "(mock_should_fail == true) ? FAILURE : SUCCESS";
   // Convert the node in asynchronous and wait 2000 ms
   test_config.async_delay = std::chrono::milliseconds(2000);
   // Execute a different script depending on the resolved return status.

examples/t15_nodes_mocking_strict_failure.cpp

@@ -79,8 +79,7 @@ int main(int /*argc*/, char** /*argv*/)
 
   BT::TestNodeConfig test_config;
   test_config.return_status.reset();
-  test_config.return_status_script =
-      "(mock_should_fail == true) ? FAILURE : SUCCESS";
+  test_config.return_status_script = "(mock_should_fail == true) ? FAILURE : SUCCESS";
   test_config.async_delay = std::chrono::milliseconds(2000);
   test_config.success_script = "msg := 'message SUBSTITUTED' ";
   test_config.failure_script = "msg := 'message FAILURE branch' ";

include/behaviortree_cpp/actions/test_node.h

@@ -13,14 +13,14 @@
 
 #pragma once
 
-#include <memory>
-#include <optional>
-#include <string>
-
 #include "behaviortree_cpp/action_node.h"
 #include "behaviortree_cpp/scripting/script_parser.hpp"
 #include "behaviortree_cpp/utils/timer_queue.h"
 
+#include <memory>
+#include <optional>
+#include <string>
+
 namespace BT
 {
 

src/actions/test_node.cpp

@@ -33,8 +33,8 @@ void validateTestNodeStatus(NodeStatus status, StringView source)
 
 EnumsTablePtr createTestNodeEnums(const EnumsTablePtr& base_enums)
 {
-  auto enums = base_enums ? std::make_shared<EnumsTable>(*base_enums)
-                          : std::make_shared<EnumsTable>();
+  auto enums = base_enums ? std::make_shared<EnumsTable>(*base_enums) :
+                            std::make_shared<EnumsTable>();
 
   (*enums)["IDLE"] = static_cast<int>(NodeStatus::IDLE);
   (*enums)["RUNNING"] = static_cast<int>(NodeStatus::RUNNING);
@@ -90,8 +90,8 @@ TestNode::TestNode(const std::string& name, const NodeConfig& config,
   if(!_config->complete_func && !_config->return_status &&
      _config->return_status_script.empty())
   {
-    throw RuntimeError(
-        "TestNode requires one of complete_func, return_status, or return_status_script");
+    throw RuntimeError("TestNode requires one of complete_func, return_status, or "
+                       "return_status_script");
   }
 
   _script_enums = createTestNodeEnums(config.enums);
@@ -170,8 +170,8 @@ NodeStatus TestNode::onCompleted()
   }
   else
   {
-    throw RuntimeError(
-        "TestNode requires one of complete_func, return_status, or return_status_script");
+    throw RuntimeError("TestNode requires one of complete_func, return_status, or "
+                       "return_status_script");
   }
 
   validateTestNodeStatus(status, "completion");

src/bt_factory.cpp

@@ -540,7 +540,8 @@ void BehaviorTreeFactory::loadSubstitutionRuleFromJSON(const std::string& json_t
       }
       if(test_config.contains("return_status_script"))
       {
-        config.return_status_script = test_config["return_status_script"].get<std::string>();
+        config.return_status_script =
+            test_config["return_status_script"].get<std::string>();
       }
       if(test_config.contains("async_delay"))
       {

tests/gtest_substitution.cpp

@@ -65,8 +65,8 @@ TEST(Substitution, Parser)
 
   auto configScript = std::get_if<TestNodeConfig>(&rules.at("actionD"));
   ASSERT_FALSE(configScript->return_status.has_value());
-  ASSERT_EQ(configScript->return_status_script,
-            "(mock_should_fail == true) ? FAILURE : SUCCESS");
+  ASSERT_EQ(configScript->return_status_script, "(mock_should_fail == true) ? FAILURE : "
+                                                "SUCCESS");
   ASSERT_EQ(configScript->failure_script, "branch := 'failure'");
 
   ASSERT_EQ(*std::get_if<std::string>(&rules.at("actionC")), "NotAConfig");
@@ -148,8 +148,7 @@ TEST(Substitution, ScriptedReturnStatusOverridesFixedStatus)
 
   TestNodeConfig test_config;
   test_config.return_status = NodeStatus::SUCCESS;
-  test_config.return_status_script =
-      "(mock_should_fail == true) ? FAILURE : SUCCESS";
+  test_config.return_status_script = "(mock_should_fail == true) ? FAILURE : SUCCESS";
   test_config.failure_script = "branch := 'failure'";
   factory.addSubstitutionRule("action_A", test_config);
 
@@ -178,8 +177,7 @@ TEST(Substitution, ScriptedReturnStatusAsyncSubstitution)
 
   TestNodeConfig test_config;
   test_config.return_status.reset();
-  test_config.return_status_script =
-      "(mock_should_fail == true) ? FAILURE : SUCCESS";
+  test_config.return_status_script = "(mock_should_fail == true) ? FAILURE : SUCCESS";
   test_config.async_delay = std::chrono::milliseconds(50);
   factory.addSubstitutionRule("action_A", test_config);
 
@@ -188,8 +186,8 @@ TEST(Substitution, ScriptedReturnStatusAsyncSubstitution)
       std::async(std::launch::async, [&tree]() { return tree.tickWhileRunning(); });
 
   auto status = future.wait_for(std::chrono::seconds(5));
-  ASSERT_NE(status, std::future_status::timeout)
-      << "Tree hung! tickWhileRunning did not complete within 5 seconds";
+  ASSERT_NE(status, std::future_status::timeout) << "Tree hung! tickWhileRunning did not "
+                                                    "complete within 5 seconds";
   ASSERT_EQ(future.get(), NodeStatus::SUCCESS);
 }