felixdivo / ros2-easy-test

A Python test framework for ROS2 allowing simple and expressive assertions based on message interactions.
https://ros2-easy-test.readthedocs.io
MIT License
15 stars 5 forks source link

add parameters to with_launch_file decorator #29 #30

Closed schmidta83 closed 5 months ago

schmidta83 commented 6 months ago

Implements #29.

Timple commented 5 months ago

Wouldn't the term launch_arguments be more correct? You cannot add parameters to a launchfile. As it wouldn't know to which node to add the parameters.

felixdivo commented 5 months ago

Wouldn't the term launch_arguments be more correct? You cannot add parameters to a launchfile. As it wouldn't know to which node to add the parameters.

I think you are right. Thanks for checking this!

@schmidta83 We can keep all as-is; just need to rename the parameter and docstring above to call it "launch arguments passed to the launch file" instead of "parameters passed to the node". :)

schmidta83 commented 5 months ago

Sorry, was busy last weeks... @Timple , good point! Will change it.

@felixdivo Thanks for the information.

felixdivo commented 5 months ago

No worries, @schmidta83. I was unresponsive for weeks, too. 🙃

schmidta83 commented 5 months ago

@felixdivo All suggegtions are adapted. You can review again.