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

Request to add the CTest helper #14

Closed
Ignition opened this issue Jul 9, 2017 · 4 comments
Closed

Request to add the CTest helper #14

Ignition opened this issue Jul 9, 2017 · 4 comments

Comments

@Ignition
Copy link

Ignition commented Jul 9, 2017

contrib/ParseAndAddCatchTests.cmake in Catch's repo is a useful script to populate ctest with the tests defined in catch. I would like to do something like the following using your conan package.

add_executable(catch_testsuite tests/tests_for_x.cpp tests/main.cpp)
target_link_libraries(catch_testsuite CONAN_PKG::Catch MYLIB::MYLIB)
enable_testing()
include(${CONAN_CATCH_ROOT}/ParseAndAddCatchTests.cmake)
ParseAndAddCatchTests(catch_testsuite)
@uilianries
Copy link
Owner

@Ignition I appreciated your feature request. However, I'm trying to move Conan support for Catch project, as native. You could see here my recently issue.
For now, we created a recipe for the project, but is not uploaded to Bintray yet. I'll be very glad if you post this issue in #926 at Catch. We could discuss and implement this issue there.
Regards.

@solvingj
Copy link

I'm not sure but is the following issue an even more advanced mechanism for the Catch-CTest integration?

catchorg/Catch2#879

@uilianries
Copy link
Owner

Sure, and is very possible to include in the package. However, I prefer move this issue to Catch project. We have created a recipe to be maintained by Catch :)

@solvingj
Copy link

solvingj commented Jul 12, 2017

Actually, I'm saying this issue doesn't even need to go to Catch because Catch already merged in a better version of the feature requested. In that case, I would say that this issue could simply be closed.

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