Skip to content

Add test for macros with multiple arguments#69

Merged
scpeters merged 1 commit intoros:masterfrom
scpeters:multiple_arguments_test
May 6, 2020
Merged

Add test for macros with multiple arguments#69
scpeters merged 1 commit intoros:masterfrom
scpeters:multiple_arguments_test

Conversation

@scpeters
Copy link
Copy Markdown
Contributor

@scpeters scpeters commented May 6, 2020

I wrote the following test while reviewing #64 but added it to a separate pull request to keep things simple.

On my machine it prints:

[ RUN      ] ConsoleTest.MultipleArguments
Error:   no extra parameters
         at line 32 in /Users/scpeters/clone/console_bridge/test/console_TEST.cc
Error:   one integer: 42
         at line 33 in /Users/scpeters/clone/console_bridge/test/console_TEST.cc
Error:   two floats: 42.010000, 0.33
         at line 34 in /Users/scpeters/clone/console_bridge/test/console_TEST.cc
[       OK ] ConsoleTest.MultipleArguments (0 ms)

@scpeters scpeters requested a review from clalancette May 6, 2020 21:26
@scpeters scpeters merged commit bf4346b into ros:master May 6, 2020
@scpeters scpeters deleted the multiple_arguments_test branch May 6, 2020 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants