Skip to content

Commit

Permalink
Fix Warning on moveit#690
Browse files Browse the repository at this point in the history
  • Loading branch information
DLu authored and henningkayser committed Sep 21, 2021
1 parent c6a31e9 commit 8a26425
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions moveit_kinematics/test/test_kinematics_plugin.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -476,8 +476,8 @@ TEST_F(KinematicsTest, unitIK)
};

std::vector<double> ground_truth, pose_values;
constexpr char POSE_TYPE_RELATIVE[] = "relative";
constexpr char POSE_TYPE_ABSOLUTE[] = "absolute";
constexpr char post_type_relative[] = "relative";
constexpr char post_type_absolute[] = "absolute";

/* process tests definitions on parameter server of the form
pose_1:
Expand All @@ -492,7 +492,7 @@ TEST_F(KinematicsTest, unitIK)
for (size_t i = 0; i < expected_test_poses; ++i) // NOLINT(modernize-loop-convert)
{
const std::string pose_name = "pose_" + std::to_string(i);
const std::string pose_param = TEST_POSES_PARAM + "." + pose_name;
const std::string pose_param = TEST_POSES_PARAM.append(".").append(pose_name);
goal = initial; // reset goal to initial
ground_truth.clear();

Expand All @@ -510,15 +510,15 @@ TEST_F(KinematicsTest, unitIK)

Eigen::Isometry3d pose;
ASSERT_TRUE(parsePose(pose_values, pose)) << "Failed to parse 'pose' vector in: " << pose_name;
std::string pose_type = "POSE_TYPE_RELATIVE";
std::string pose_type = "post_type_relative";
node_->get_parameter_or(pose_param + ".type", pose_type, pose_type);
if (pose_type == POSE_TYPE_RELATIVE)
if (pose_type == post_type_relative)
goal = goal * pose;
else if (pose_type == POSE_TYPE_ABSOLUTE)
else if (pose_type == post_type_absolute)
goal = pose;
else
FAIL() << "Found invalid 'type' in " << pose_name << ": should be one of '" << POSE_TYPE_RELATIVE << "' or '"
<< POSE_TYPE_ABSOLUTE << "'";
FAIL() << "Found invalid 'type' in " << pose_name << ": should be one of '" << post_type_relative << "' or '"
<< post_type_absolute << "'";

std::string desc;
{
Expand Down

0 comments on commit 8a26425

Please sign in to comment.