ROS 2 remove obsolate tests in BT (#333)

* remove unnecessary tests * remove unnecessary brackets
husarion · Jun 10, 2024 · c0b1514 · c0b1514
1 parent cd0c334
commit c0b1514
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 37 deletions.
diff --git a/panther_manager/behavior_trees/lights.xml b/panther_manager/behavior_trees/lights.xml
@@ -49,7 +49,7 @@ current_anim_id = CHARGING_BATTERY_ANIM_ID"/>
                                     param=""
                                     repeating="true"
                                     service_name="lights/controller/set/animation"
-                                    _skipIf="(!e_stop_state) || (current_anim_id == E_STOP_ANIM_ID)"
+                                    _skipIf="(!e_stop_state) || current_anim_id == E_STOP_ANIM_ID"
                                     _onSuccess="current_anim_id = E_STOP_ANIM_ID"/>
         <Sequence name="BatteryStatusSequence">
           <TickAfterTimeout timeout="{LOW_BATTERY_ANIM_PERIOD}"

diff --git a/panther_manager/test/plugins/test_call_set_bool_service_node.cpp b/panther_manager/test/plugins/test_call_set_bool_service_node.cpp
@@ -135,22 +135,6 @@ TEST_F(TestCallSetBoolService, WrongSetBoolCallServiceFailure)
   EXPECT_EQ(status, BT::NodeStatus::FAILURE);
 }
 
-TEST_F(TestCallSetBoolService, WrongServiceValueDefined)
-{
-  std::map<std::string, std::string> service = {
-    {"service_name", "set_bool"}, {"data", "wrong_bool"}};
-
-  using std_srvs::srv::SetBool;
-  CreateService<SetBool>(
-    "set_bool",
-    [&](const SetBool::Request::SharedPtr request, SetBool::Response::SharedPtr response) {
-      ServiceCallback(request, response, true, true);
-    });
-  RegisterNodeWithParams<panther_manager::CallSetBoolService>("CallSetBoolService");
-
-  EXPECT_THROW({ CreateTree("CallSetBoolService", service); }, BT::LogicError);
-}
-
 int main(int argc, char ** argv)
 {
   testing::InitGoogleTest(&argc, argv);

diff --git a/panther_manager/test/plugins/test_call_set_led_animation_service_node.cpp b/panther_manager/test/plugins/test_call_set_led_animation_service_node.cpp
@@ -171,26 +171,6 @@ TEST_F(TestCallSetLedAnimationService, WrongSetLedAnimationCallServiceFailure)
   EXPECT_EQ(status, BT::NodeStatus::FAILURE);
 }
 
-TEST_F(TestCallSetLedAnimationService, WrongRepeatingServiceValueDefined)
-{
-  std::map<std::string, std::string> service = {
-    {"service_name", "set_led_animation"}, {"id", "0"}, {"param", ""}, {"repeating", "wrong_bool"}};
-
-  RegisterNodeWithParams<panther_manager::CallSetLedAnimationService>("CallSetLedAnimationService");
-
-  EXPECT_THROW({ CreateTree("CallSetLedAnimationService", service); }, BT::LogicError);
-}
-
-TEST_F(TestCallSetLedAnimationService, WrongIdServiceValueDefined)
-{
-  std::map<std::string, std::string> service = {
-    {"service_name", "set_led_animation"}, {"id", "-5"}, {"param", ""}, {"repeating", "true"}};
-
-  RegisterNodeWithParams<panther_manager::CallSetLedAnimationService>("CallSetLedAnimationService");
-
-  EXPECT_THROW({ CreateTree("CallSetLedAnimationService", service); }, BT::LogicError);
-}
-
 int main(int argc, char ** argv)
 {
   testing::InitGoogleTest(&argc, argv);