Skip to content

Commit

Permalink
alphametics: mark computationally intensive test as extra-credit
Browse files Browse the repository at this point in the history
While this test is canonical, it does not technically add additional coverage. This test serves as a test for efficiency (exercism/problem-specifications#1024 (comment)) of a solution, not completeness.

Furthermore, here are the run-times for this exercise from the [latest Travis build]() (at the time of this writing):
| Python Version | Run-time (seconds) |
| --- | --- |
| 2.7 | 3.155 |
| 3.3 | 2.461 |
| 3.4 | 3.567 |
| 3.5 | 7.270 |
| 3.6 | 0.774 |

Notice that the optimized example solution is only "fast" in 3.6.
  • Loading branch information
cmccandless committed Mar 5, 2018
1 parent e26b834 commit 52c9f4f
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions exercises/alphametics/alphametics_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,7 @@ def test_puzzle_with_ten_letters(self):
"S": 6,
"T": 9})

@unittest.skip("extra-credit")
def test_puzzle_with_ten_letters_and_199_addends(self):
self.assertEqual(
solve(
Expand Down

0 comments on commit 52c9f4f

Please sign in to comment.