Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

catkin_test_results does not show disabled tests #839

Closed
rsinnet opened this issue Dec 23, 2016 · 1 comment
Closed

catkin_test_results does not show disabled tests #839

rsinnet opened this issue Dec 23, 2016 · 1 comment
Labels
enhancement

Comments

@rsinnet
Copy link

rsinnet commented Dec 23, 2016

In the Google Test framework, a test can be marked as disabled by prepending DISABLED_ to the test suite or test case name. In such a case the test is marked disabled in the output XML. catkin_test_results does not appear to handle disabled tests. It looks like it would only require a few lines in two files to add this feature. I'd be happy to add it.
@dirk-thomas
Copy link
Member

That would be great. You can also look at the recent changes to ament for adding the same functionality: ament/ament_tools@0c6003b and ament/ament_tools@8324706

dirk-thomas pushed a commit that referenced this issue Jan 25, 2017
@rsinnet @dirk-thomas
Fixes #839: Adding disabled tests to catkin_test_results summary.
b3a5033
dirk-thomas added a commit that referenced this issue Feb 28, 2017
@dirk-thomas
add skipped / disabled tests to catkin_test_results summary (#848)
df9e38e 
* Fixes #839: Adding disabled tests to catkin_test_results summary.

* maintain public API
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dirk-thomas @rsinnet