fix passing multiple args to add_rostest

[dirk-thomas]

Fixes #790.

[efernandez]

👍 Exactly the fix I was thinking of.

I you give me a few minutes, I can checkout the code and give it a try. Not sure if the tests already check that.

BTW, any chance this could get backported to indigo?

[dirk-thomas]

Absolutely, please give it a try and yes, it will be backported since it is a clear bug fix.

[efernandez]

@dirk-thomas How can I test it? I've just checkout the repo on my workspace, but it still uses the system version.

[dirk-thomas]

After cloning this branch into your workspace and building it other packages in the workspace will use the rostest package from the workspace hence the custom version.

[efernandez]

Ok, let me catkin clean && catkin build then (not just catkin build).

[efernandez]

Confirmed. It works.

I tested directly with this branch on indigo, which works fine. I guess you'll cherry-pick the commit from this PR into indigo. I guess it'll work too, but please tag me on the PR against indigo if you want me to test it again.

[dirk-thomas]

Thanks for testing it.

Since the patch is straight forward I just cherry-picked it to indigo-devel: 7f251de

[efernandez]

Thanks for the extremely fast fix.

[efernandez]

@dirk-thomas When would the indigo-devel commit get into the next release? :)

[dirk-thomas]

While the previous release is in shadow-fixed there are a couple of regressions. So I will likely create a new release in the next days and after a shorter soak time it will be synced (hopefully 😉).