fix tests

BehaviorTree · Jun 18, 2024 · bcceae7 · bcceae7
1 parent ced041b
commit bcceae7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/src/xml_parsing.cpp b/src/xml_parsing.cpp
@@ -402,6 +402,13 @@ void VerifyXML(const std::string& xml_text,
   }
   //-------------------------------------------------
 
+  int behavior_tree_count = 0;
+  for(auto child = xml_root->FirstChildElement(); child != nullptr;
+      child = child->NextSiblingElement())
+  {
+    behavior_tree_count++;
+  }
+
   // function to be called recursively
   std::function<void(const XMLElement*)> recursiveStep;
 
@@ -482,7 +489,7 @@ void VerifyXML(const std::string& xml_text,
     }
     else if(name == "BehaviorTree")
     {
-      if(ID.empty())
+      if(ID.empty() && behavior_tree_count > 1)
       {
         ThrowError(line_number, "The tag <BehaviorTree> must have the "
                                 "attribute [ID]");

diff --git a/tests/gtest_ports.cpp b/tests/gtest_ports.cpp
@@ -104,11 +104,11 @@ TEST(PortTest, Descriptions)
         <BehaviorTree ID="MainTree" _description="this is my tree" >
             <Sequence>
                 <NodeWithPorts name="first"  in_port_B="66" _description="this is my action" />
-                <SubTree ID="SubTree" name="second" _description="this is a subtree"/>
+                <SubTree ID="mySubTree" name="second" _description="this is a subtree"/>
             </Sequence>
         </BehaviorTree>
 
-        <BehaviorTree ID="SubTree" _description="this is a subtree" >
+        <BehaviorTree ID="mySubTree" _description="this is a subtree" >
             <NodeWithPorts name="third" in_port_B="99" />
         </BehaviorTree>