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

Scale back on SwiftCheck for some tests #68

Open
brentleyjones opened this issue Aug 6, 2015 · 2 comments
Open

Scale back on SwiftCheck for some tests #68

brentleyjones opened this issue Aug 6, 2015 · 2 comments

Comments

@brentleyjones
Copy link
Member

For certain tests SwiftCheck is too heavy/unneeded. Equatable tests for example don't need to check different permutations since they don't have code paths in them.

Resolving this issue with speed up our tests by a good amount (especially the soon-to-come FrequencyDistribution and ProbabilityMass tests).

@brentleyjones
Copy link
Member Author

Starting to get worse in #71:

Executed 181 tests, with 0 failures (0 unexpected) in 6.868 (7.005) seconds

If we could combine some of the SwiftCheck tests together, while still knowing which part failed, we would get a big speed up (e.g. Equatable tests). So far when trying to do so I've failed to have the failure output point to which part fail. It still says which input failed, so maybe that is enough (you could debug with that input).

@brentleyjones
Copy link
Member Author

I opened an issue about being able to have meaningful sub-property failures: typelift/SwiftCheck#74

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant