Skip to content
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.

Sign up for GitHub

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

Jump to bottom

test(costmap_generator): unit test implementation for costmap generator #9149

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

test(costmap_generator): unit test implementation for costmap generator #9149

wants to merge 9 commits into from

Conversation

mkquda
Copy link
Contributor

@mkquda mkquda commented Oct 24, 2024

Description

Add test for costmap_generator package to improve code coverage and prevent future bugs.

Changes

  • modify costmap generator directory structure
  • update CMakeLists.txt
  • add test case for object_to_costmap utils
  • add unit test for object_map_utils
  • add unit test for CostmapGeneratorNode class

Related links

None.

How was this PR tested?

  • Build successfully
  • Run unit test
  • Generate code coverage report

Screenshot from 2024-10-24 15-55-16

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

@mkquda
modify costmap generator directory structure
cb22060 
Signed-off-by: mohammad alqudah <[email protected]>
@mkquda
rename class CostmapGenerator to CostmapGeneratorNode
e49a5b5 
Signed-off-by: mohammad alqudah <[email protected]>
@mkquda
Merge branch 'awf-latest' into RT1-7963-unit-test-implementation-for-…
d44fe33 
…costmap-generator
@mkquda
unit test for object_map_utils
82585a4 
Signed-off-by: mohammad alqudah <[email protected]>
@mkquda
catch error from lookupTransform
9f5ae50 
Signed-off-by: mohammad alqudah <[email protected]>
@mkquda
Merge branch 'awf-latest' into RT1-7963-unit-test-implementation-for-…
9965bde 
…costmap-generator
@mkquda
use polling subscriber in costmap generator node
e4c09fb 
Signed-off-by: mohammad alqudah <[email protected]>
@mkquda
Merge branch 'awf-latest' into RT1-7963-unit-test-implementation-for-…
24d9e62 
…costmap-generator
@mkquda
add test for costmap generator node
162ddaa 
Signed-off-by: mohammad alqudah <[email protected]>
@github-actions github-actions bot added component:planning Route planning, decision-making, and navigation. (auto-assigned) component:launch Launch files, scripts and initialization tools. (auto-assigned) component:common Common packages from the autoware-common repository. (auto-assigned) labels Oct 24, 2024
@github-actions GitHub Actions
Copy link

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kosuke55 kosuke55 Awaiting requested review from kosuke55 kosuke55 will be requested when the pull request is marked ready for review kosuke55 is a code owner

@TakaHoribe TakaHoribe Awaiting requested review from TakaHoribe TakaHoribe will be requested when the pull request is marked ready for review TakaHoribe is a code owner

@takayuki5168 takayuki5168 Awaiting requested review from takayuki5168 takayuki5168 will be requested when the pull request is marked ready for review takayuki5168 is a code owner

@rej55 rej55 Awaiting requested review from rej55 rej55 will be requested when the pull request is marked ready for review rej55 is a code owner

@kyoichi-sugahara kyoichi-sugahara Awaiting requested review from kyoichi-sugahara kyoichi-sugahara will be requested when the pull request is marked ready for review kyoichi-sugahara is a code owner

@mkuri mkuri Awaiting requested review from mkuri mkuri will be requested when the pull request is marked ready for review mkuri is a code owner

@soblin soblin Awaiting requested review from soblin soblin will be requested when the pull request is marked ready for review soblin is a code owner

@satoshi-ota satoshi-ota Awaiting requested review from satoshi-ota satoshi-ota will be requested when the pull request is marked ready for review satoshi-ota is a code owner

@shmpwk shmpwk Awaiting requested review from shmpwk shmpwk will be requested when the pull request is marked ready for review shmpwk is a code owner

@taikitanaka3 taikitanaka3 Awaiting requested review from taikitanaka3 taikitanaka3 will be requested when the pull request is marked ready for review taikitanaka3 is a code owner

@TomohitoAndo TomohitoAndo Awaiting requested review from TomohitoAndo TomohitoAndo will be requested when the pull request is marked ready for review TomohitoAndo is a code owner

@tkimura4 tkimura4 Awaiting requested review from tkimura4 tkimura4 will be requested when the pull request is marked ready for review tkimura4 is a code owner

@zulfaqar-azmi-t4 zulfaqar-azmi-t4 Awaiting requested review from zulfaqar-azmi-t4 zulfaqar-azmi-t4 will be requested when the pull request is marked ready for review zulfaqar-azmi-t4 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
component:common Common packages from the autoware-common repository. (auto-assigned) component:launch Launch files, scripts and initialization tools. (auto-assigned) component:planning Route planning, decision-making, and navigation. (auto-assigned)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mkquda