fix warnings

facontidavide · facontidavide · commit 2bd632669983 · 2022-11-23T10:50:28.000+01:00
diff --git a/tests/gtest_factory.cpp b/tests/gtest_factory.cpp
@@ -77,7 +77,7 @@ static const char* xml_text_subtree = R"(
 
 static const char* xml_text_subtree_part1 = R"(
 
-<root>
+<root BTCPP_format="4">
   <BehaviorTree ID="MainTree">
     <Fallback name="root_selector">
       <SubTree ID="DoorClosedSubtree" />
@@ -88,7 +88,7 @@ static const char* xml_text_subtree_part1 = R"(
 
 static const char* xml_text_subtree_part2 = R"(
 
-<root>
+<root BTCPP_format="4">
   <BehaviorTree ID="DoorClosedSubtree">
     <Sequence name="door_sequence">
       <Decorator ID="Inverter">
@@ -168,7 +168,7 @@ TEST(BehaviorTreeFactory, Subtree)
 TEST(BehaviorTreeFactory, Issue7)
 {
   const std::string xml_text_issue = R"(
-<root>
+<root BTCPP_format="4">
     <BehaviorTree ID="ReceiveGuest">
     </BehaviorTree>
 </root> )";
@@ -183,7 +183,7 @@ TEST(BehaviorTreeFactory, Issue7)
 
 static const char* xml_ports_subtree = R"(
 
-<root BTCPP_format="4" format="4" main_tree_to_execute="MainTree">
+<root BTCPP_format="4" main_tree_to_execute="MainTree">
 
   <BehaviorTree ID="TalkToMe">
     <Sequence>
diff --git a/tests/gtest_logger_zmq.cpp b/tests/gtest_logger_zmq.cpp
@@ -7,7 +7,7 @@ TEST(LoggerZMQ, ZMQLoggerDeletesCleanlyAfterTickingTree)
 {
   // GIVEN we create a behavior tree through the BT factory and attach a ZMQ publisher to it
   static constexpr auto XML = R"(
-<root>
+<root BTCPP_format="4">
     <BehaviorTree>
     <SetBlackboard output_key="arg1" value="1" />
     </BehaviorTree>
@@ -30,7 +30,7 @@ TEST(LoggerZMQ, ZMQLoggerDeletesCleanlyAfterNotTickingTree)
 {
   // GIVEN we create a behavior tree through the BT factory and attach a ZMQ publisher to it
   static constexpr auto XML = R"(
-<root>
+<root BTCPP_format="4">
     <BehaviorTree>
     <SetBlackboard output_key="arg1" value="1" />
     </BehaviorTree>
diff --git a/tests/gtest_wakeup.cpp b/tests/gtest_wakeup.cpp
@@ -28,7 +28,7 @@ TEST(WakeUp, BasicTest)
 {
   static const char* xml_text = R"(
 
-<root>
+<root BTCPP_format="4">
     <BehaviorTree ID="MainTree">
         <FastAction/>
     </BehaviorTree>
diff --git a/tests/navigation_test.cpp b/tests/navigation_test.cpp
@@ -7,7 +7,7 @@ using namespace BT;
 // clang-format off
 static const char* xml_text = R"(
 
-<root main_tree_to_execute="BehaviorTree">
+<root  BTCPP_format="4" main_tree_to_execute="BehaviorTree">
     <BehaviorTree ID="BehaviorTree">
         <Fallback name="root">
 
diff --git a/tests/trees/child/child/child_child_no_include.xml b/tests/trees/child/child/child_child_no_include.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ChildChildNoInclude">
+<root BTCPP_format="4" main_tree_to_execute = "ChildChildNoInclude">
   <BehaviorTree ID="ChildChildNoInclude">
     <AlwaysSuccess />
   </BehaviorTree>
diff --git a/tests/trees/child/child_include_child.xml b/tests/trees/child/child_include_child.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ChildIncludeChild">
+<root BTCPP_format="4" main_tree_to_execute = "ChildIncludeChild">
   <include path="child/child_child_no_include.xml" />
 
   <BehaviorTree ID="ChildIncludeChild">
diff --git a/tests/trees/child/child_include_parent.xml b/tests/trees/child/child_include_parent.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ChildIncludeParent">
+<root BTCPP_format="4" main_tree_to_execute = "ChildIncludeParent">
   <include path="../parent_no_include.xml" />
 
   <BehaviorTree ID="ChildIncludeParent">
diff --git a/tests/trees/child/child_include_sibling.xml b/tests/trees/child/child_include_sibling.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ChildIncludeSibling">
+<root BTCPP_format="4" main_tree_to_execute = "ChildIncludeSibling">
   <include path="child_no_include.xml" />
 
   <BehaviorTree ID="ChildIncludeSibling">
diff --git a/tests/trees/child/child_no_include.xml b/tests/trees/child/child_no_include.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ChildNoInclude">
+<root BTCPP_format="4" main_tree_to_execute = "ChildNoInclude">
   <BehaviorTree ID="ChildNoInclude">
     <AlwaysSuccess />
   </BehaviorTree>
diff --git a/tests/trees/parent_include_child.xml b/tests/trees/parent_include_child.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ParentIncludeChild">
+<root BTCPP_format="4" main_tree_to_execute = "ParentIncludeChild">
   <include path="child/child_no_include.xml" />
 
   <BehaviorTree ID="ParentIncludeChild">
diff --git a/tests/trees/parent_include_child_include_child.xml b/tests/trees/parent_include_child_include_child.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ParentIncludeChildIncludeChild">
+<root BTCPP_format="4" main_tree_to_execute = "ParentIncludeChildIncludeChild">
   <include path="child/child_include_child.xml" />
 
   <BehaviorTree ID="ParentIncludeChildIncludeChild">
diff --git a/tests/trees/parent_include_child_include_parent.xml b/tests/trees/parent_include_child_include_parent.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ParentIncludeChildIncludeParent">
+<root BTCPP_format="4" main_tree_to_execute = "ParentIncludeChildIncludeParent">
   <include path="child/child_include_parent.xml" />
 
   <BehaviorTree ID="ParentIncludeChildIncludeParent">
diff --git a/tests/trees/parent_include_child_include_sibling.xml b/tests/trees/parent_include_child_include_sibling.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ParentIncludeChildIncludeSibling">
+<root BTCPP_format="4" main_tree_to_execute = "ParentIncludeChildIncludeSibling">
   <include path="child/child_include_sibling.xml" />
 
   <BehaviorTree ID="ParentIncludeChildIncludeSibling">
diff --git a/tests/trees/parent_no_include.xml b/tests/trees/parent_no_include.xml
@@ -1,4 +1,4 @@
-<root main_tree_to_execute = "ParentNoInclude">
+<root BTCPP_format="4" main_tree_to_execute = "ParentNoInclude">
   <BehaviorTree ID="ParentNoInclude">
     <AlwaysSuccess />
   </BehaviorTree>

Original file line number	Diff line number	Diff line change
`@@ -28,7 +28,7 @@ TEST(WakeUp, BasicTest)`
`28`	`28`	`{`
`29`	`29`	`static const char* xml_text = R"(`
`30`	`30`
`31`		`-<root>`
	`31`	`+<root BTCPP_format="4">`
`32`	`32`	`<BehaviorTree ID="MainTree">`
`33`	`33`	`<FastAction/>`
`34`	`34`	`</BehaviorTree>`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-<root main_tree_to_execute = "ChildChildNoInclude">`
	`1`	`+<root BTCPP_format="4" main_tree_to_execute = "ChildChildNoInclude">`
`2`	`2`	`<BehaviorTree ID="ChildChildNoInclude">`
`3`	`3`	`<AlwaysSuccess />`
`4`	`4`	`</BehaviorTree>`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-<root main_tree_to_execute = "ChildIncludeChild">`
	`1`	`+<root BTCPP_format="4" main_tree_to_execute = "ChildIncludeChild">`
`2`	`2`	`<include path="child/child_child_no_include.xml" />`
`3`	`3`
`4`	`4`	`<BehaviorTree ID="ChildIncludeChild">`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-<root main_tree_to_execute = "ChildNoInclude">`
	`1`	`+<root BTCPP_format="4" main_tree_to_execute = "ChildNoInclude">`
`2`	`2`	`<BehaviorTree ID="ChildNoInclude">`
`3`	`3`	`<AlwaysSuccess />`
`4`	`4`	`</BehaviorTree>`
Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-<root main_tree_to_execute = "ParentIncludeChild">`
	`1`	`+<root BTCPP_format="4" main_tree_to_execute = "ParentIncludeChild">`
`2`	`2`	`<include path="child/child_no_include.xml" />`
`3`	`3`
`4`	`4`	`<BehaviorTree ID="ParentIncludeChild">`