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

[Alphametics] Add more easy cases. #425

Merged
merged 1 commit into from
Oct 27, 2016
Merged

Conversation

Insti
Copy link
Contributor

@Insti Insti commented Oct 24, 2016

Add a few more easy cases so people can be a little more sure their solution is working before they run up against memory/time constraints* when trying to solve the harder problems.

Re-order the problems in order of difficulty.

Rename tests to match number of letters in the result.

* The naive implementation of a solution tends to be O(n!)

Rename tests to match number of letters in the result.
Insti added a commit to Insti/exercism-ruby that referenced this pull request Oct 24, 2016
Ignore test cases based on the complexity of the expected result.

Hardcoded commented out versions of the hard(er) tests along with
instructions on why you might want to enable them.

Refactored lib/alphametics_cases.rb.
Use the common generator method names.
Tidied up expected value string generation code.

Re-generate tests based on version based on proposed changes.
exercism/problem-specifications#425

Update `example.rb` to pass the current version of tests.
@Insti Insti merged commit b760eb4 into exercism:master Oct 27, 2016
emcoding pushed a commit that referenced this pull request Nov 19, 2018
Date is not necessary for tests nor solution
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.

1 participant