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.

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

Add CMake test support as catch_discover_tests() in Catch2 #171

Closed
ncihnegn opened this issue Jan 12, 2019 · 5 comments
Closed

Add CMake test support as catch_discover_tests() in Catch2 #171

ncihnegn opened this issue Jan 12, 2019 · 5 comments

Comments

@ncihnegn
Copy link
Contributor

With catch_discover_tests(), testcase names can be shown in 'make test'.

@onqtam
Copy link
Member

onqtam commented Jan 15, 2019

I'll see what can be done about this but no timeframe can be promised.

@reddwarf69
Copy link
Contributor

@ncihnegn @DPD85 do you want to try how well #236 covers your needs?

@onqtam
Copy link
Member

onqtam commented May 15, 2019

@reddwarf69 Glad to see someone else took the initiative for this work! I'm not sure when I'll get to reviewing it but since there is interest I guess it will eventually end up merged.

One observation I have is that often there are PRs and issues raised against Catch for this functionality so I hope it won't be just me maintaining it in the future.

@reddwarf69
Copy link
Contributor

One observation I have is that often there are PRs and issues raised against Catch for this functionality so I hope it won't be just me maintaining it in the future.

Honestly, I can't tell you "don't worry, I will take care of them". I am not that proficient with cmake and I'm short on free time.
But I'm not going to say "not my problem" either. If any bug is raised feel free to poke me and I will do my best.

@onqtam
Copy link
Member

onqtam commented Jun 2, 2019

closing - this has been added here: https://github.com/onqtam/doctest/blob/master/scripts/cmake/doctest.cmake (also needs the doctestAddTests.cmake file from the same directory) - read the comments there on how to use it.

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

No branches or pull requests

3 participants