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

Run tests asynchronously #1308

Closed
Leandros opened this issue Jun 7, 2018 · 3 comments
Closed

Run tests asynchronously #1308

Leandros opened this issue Jun 7, 2018 · 3 comments

Comments

@Leandros
Copy link

Leandros commented Jun 7, 2018

This is a big one, I know, but with increasing number of tests I found catch to become very slow.
Since the tests in each TEST_CASE are (or rather should) be self-contained, it would be great if there was a knob to turn which runs all tests in parallel (up to N, where N is usually number of cores).

This would require a couple of changes I presume, since not even REQUIRE on it's own is thread-safe.

@Leandros
Copy link
Author

Leandros commented Jun 7, 2018

I've seen this has been requested in the past, and is mentioned in the known limitations. I'd still like to nag you. The current solutions are not satisfying.

@horenmar
Copy link
Member

Not happening.

To expand on it a bit, we have decided that having parallel runner is out of scope for Catch, and getting multiple tests to run in parallel should instead be left for external test runners.

We do however plan to have (optionally?) thread-safe assertions at some point.

@horenmar horenmar added the Resolved - pending review Issue waiting for feedback from the original author label Jul 8, 2018
@JoeyGrajciar
Copy link
Contributor

Closing based on last comment

@horenmar horenmar removed the Resolved - pending review Issue waiting for feedback from the original author label Apr 10, 2019
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