Add NOCAPTURE option to ament_add_pytest_test

[jacobperron]

This lets the user pass the '-s' option to pytest so we can see all test process output in the console.

Can be useful for debugging.

[wjwwood]

This seems fine to me, but for the purposes of debugging, it seems like it would be more useful to enable/disable this when invoking the test runner somehow, rather than changing a CMake variable, thus requiring you to rerun cmake.

I'm not sure on what the correct mechanism should be, but maybe a flag given to ctest via colcon's --ctest-args option, or maybe an environment variable?

But, that's more of a "yes and" suggestion, so this lgtm

[jacobperron]

Yes, I agree. It would be nice to be able to pass pytest args from colcon, but I'm also not sure on the mechanism. I guess ament could expose a set of well known cmake arguments, e.g. -DAMENT_CMAKE_PYTEST="-s -We". Or they could be very specific, like -DAMENT_CMAKE_PYTEST_NO_CAPTURE=ON.

[jacobperron]

• Linux
• Linux-aarch64
• Windows