-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Sprint 11/PI - 154] Akushon Unit Test #31
base: master
Are you sure you want to change the base?
Conversation
test/action/interpolator_test.cpp
Outdated
std::vector<tachimawari::joint::Joint> action_manager_joints = action_manager.get_joints(); | ||
|
||
for (int i = 0; i < 20; i++){ | ||
ASSERT_TRUE(expected_positions[0][i] + max_error > action_manager_joints[i].get_position() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use EXPECT_NEAR
instead, see this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another similar usage as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
okay done
test/action/interpolator_test.cpp
Outdated
&& expected_positions[0][i] - max_error < action_manager_joints[i].get_position()); | ||
} | ||
|
||
for (int i = 0; i < 1100; i++) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why did you use 1100
for the looping limit?
Can you explain about this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the delay for the action is 1 second, so i used 1100
to test whether the action would be done in at least 0.1 seconds after the delay. should i change the value or assign it to a variable?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are request change and question
My guess was the |
Maybe mas @threeal, any suggestion or comment? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good for me
btw pls resolve the warnings @Alffaza |
…kushon-unit-tests
…ushon into feature/akushon-unit-tests
…eature/akushon-unit-tests
Jira Link:
https://ichiro-its.atlassian.net/browse/PI-154
Description
create unit test for node initialization, interpolator, and action creation
Type of Change
How Has This Been Tested?
Checklist:
feature/JIRA-ID-SHORT-DESCRIPTION
if has a JIRA ticketenhancement/SHORT-DESCRIPTION
if has/has no JIRA ticket and contain enhancementhotfix/SHORT-DESCRIPTION
if the change doesn't need to be tested (urgent)