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

CTest integration using Catch2 scripts does not work #110

Closed
cschreib opened this issue Jul 25, 2023 · 0 comments · Fixed by #111
Closed

CTest integration using Catch2 scripts does not work #110

cschreib opened this issue Jul 25, 2023 · 0 comments · Fixed by #111
Assignees
Labels
bug:confirmed Something isn't working (confirmed)
Milestone

Comments

@cschreib
Copy link
Member

Since snitch supports the same test listing and filtering CLI interface as Catch2, in principle we should be able to reuse the Catch2 CTest integration to run the snitch tests.

In practice it does not work because, currently, snitch applies test filtering on the base name of the test case only, and not the full name reported in --list-tests. The base name and full name differ for typed (templated) test cases, e.g. full name foo <int> vs base name foo. Therefore CTest gets the full name from --list-test and then tries to filter the tests with this full name, and finds nothing.

@cschreib cschreib added the bug:confirmed Something isn't working (confirmed) label Jul 25, 2023
@cschreib cschreib added this to the v1.2 milestone Jul 25, 2023
@cschreib cschreib self-assigned this Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:confirmed Something isn't working (confirmed)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant