-
Notifications
You must be signed in to change notification settings - Fork 650
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
test(autoware_behavior_path_planner_common): add tests for calcInterpolatedPoseWithVelocity #8270
test(autoware_behavior_path_planner_common): add tests for calcInterpolatedPoseWithVelocity #8270
Conversation
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #8270 +/- ##
==========================================
- Coverage 25.02% 24.95% -0.08%
==========================================
Files 1322 1331 +9
Lines 98047 98355 +308
Branches 37795 37819 +24
==========================================
+ Hits 24536 24542 +6
- Misses 71020 71322 +302
Partials 2491 2491
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
78e4a92
into
autowarefoundation:main
…olatedPoseWithVelocity (autowarefoundation#8270) * test: add interpolated pose calculation function's test Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> * disabled SpecialCases test Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> --------- Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> Signed-off-by: emuemuJP <k.matsumoto.0807@gmail.com>
Description
Add test for calcInterpolatedPoseWithVelocity in the
autoware_behavior_path_planner_common
packageCoverage is improved a bit.
How was this PR tested?
conductin colcon test in local environment
get following result
Notes for reviewers
The tests defined in DISABLED_SpecialCases are temporarily disabled because same_time_result should return nullopt for interpolation of consecutive points at the same time. The implementation will be modified in different PR.
Effects on system behavior
None.