Add tests for type_support functions - Githubissues

ros2 / rosidl_typesupport

Packages which provide the typesupport for ROS messages and services

Apache License 2.0

13 stars 34 forks source link

Add tests for type_support functions #63

Closed brawner closed 4 years ago

brawner commented 4 years ago

Adding unit tests to cover the packages public API as part of the effort to bring these packages to QL 1.

brawner commented 4 years ago

Linux
Linux-aarch64
macOS
Windows
Coverage

brawner commented 4 years ago

This is ready for review

brawner commented 4 years ago

Coverage test is unstable because I forgot to run uncrustify before running it. Otherwise coverage is now at 94%!

brawner commented 4 years ago

I have some work to do on Windows looks like:

brawner commented 4 years ago

Windows should be working now.

Linux
Linux-aarch64
macOS
Windows

brawner commented 4 years ago

Linux
Linux-aarch64
macOS
Windows

ahcorde commented 4 years ago

Windows

brawner commented 4 years ago

Trying to fix windows warning:

brawner commented 4 years ago

I think I worked out this windows issue. New updates to VS make it pickier than it was a month ago.

Linux
Linux-aarch64
macOS
Windows

brawner commented 4 years ago

Force pushed a fix for the white space issue.

brawner commented 4 years ago

Thank you Scott, for that addition, it really improves this PR significantly.

brawner commented 4 years ago

I think this is failed last ngiht:

https://ci.ros2.org/view/nightly/job/nightly_linux_release/lastCompletedBuild/testReport/(root)/projectroot/test_message_type_support/