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

Re-enable the tests under SwiftPM. #12

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

samdeane
Copy link
Contributor

@samdeane samdeane commented Mar 7, 2018

Since we can't use a bundle to find the test data, we fall back on a relative path from the built executable.

This is a bit skanky, but it's a useable workaround, and better than not having the tests at all.

Since we can't use a bundle to find the test data, we fall back on a relative path from the built executable.
This is a bit skanky, but it's a useable workaround.
@samdeane samdeane mentioned this pull request Mar 8, 2018
@kovpas
Copy link
Contributor

kovpas commented Mar 9, 2018

Thanks! Could you please move this to a separate method, like you did in #14?

@samdeane
Copy link
Contributor Author

Yeah, no problem. I should have done it in the first place :)

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

Successfully merging this pull request may close these issues.

2 participants