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

Fix Wordcount unit test for Xcode 6 beta 4 #5

Merged
merged 1 commit into from
Aug 8, 2014

Conversation

edasque
Copy link
Contributor

@edasque edasque commented Jul 27, 2014

Fix for issue #4

I imagine that's a correct approach until Swift Dictionary become Equatable or XTCAssertEqualObjects makes a come back

@edasque
Copy link
Contributor Author

edasque commented Jul 28, 2014

Let me know if that works for you, I noticed that the test suite for ETL suffers of the same issue.

@hankturowski
Copy link
Contributor

This is a very nice solution to the missing XTCAssertEqualObjects problem. Hopefully Apple will fix the issue in a future beta and we can switch back, but for now I think your bridging approach is the right way to go.

@edasque
Copy link
Contributor Author

edasque commented Jul 28, 2014

Great, I'll create more pull requests as I work through the exercisms once you merge this. @hankturowski, don't hesitate to nitpick my swift submissions. My exercism team doesn't have Swift experience.

kytrinyx added a commit that referenced this pull request Aug 8, 2014
Fix Wordcount unit test for Xcode 6 beta 4
@kytrinyx kytrinyx merged commit bb89a77 into exercism:master Aug 8, 2014
@kytrinyx
Copy link
Member

kytrinyx commented Aug 8, 2014

Thank you!

@edasque
Copy link
Contributor Author

edasque commented Aug 8, 2014

You're welcome. I'll fix the next one as well. See you at E4E, @kytrinyx, looking fwd to your presentation

@kytrinyx
Copy link
Member

kytrinyx commented Aug 8, 2014

I'm very excited about E4E, see you there!

kytrinyx added a commit that referenced this pull request Oct 9, 2014
Fix Wordcount unit test for Xcode 6 beta 4
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.

3 participants