Skip to content

crypto-square: Fix invalid tests #64

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

Merged
merged 1 commit into from
Sep 18, 2015
Merged

Conversation

jmluy
Copy link
Contributor

@jmluy jmluy commented Sep 16, 2015

The existing tests were normalizing the strings based on the square size and not the actual sequence of characters from the columns.
You can refer to the source material where "If man was meant to stay on the
ground god would have given us roots" should be "imtgdvs fearwer mayoogo anouuio ntnnlvt wttddes aohghn sseoau" but were grouped by square size on the existing tests. This is a temporary fix until #55 uses the json file.

@jtigger jtigger self-assigned this Sep 17, 2015
jtigger added a commit that referenced this pull request Sep 18, 2015
crypto-square: Fix invalid tests
@jtigger jtigger merged commit 49bb252 into exercism:master Sep 18, 2015
@jtigger
Copy link
Contributor

jtigger commented Sep 18, 2015

Thanks again, John! Looks great.

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