Skip to content

Commit

Permalink
scrabble-score: add alphabet completeness test (#516)
Browse files Browse the repository at this point in the history
After the discussion in #509 , here is proposed addition to the
canonical data.

If we really wish to stick to "proper, scrabble-approved word", I could
make multiple cases for
"the"
"quick"
"brown"
"fox"
"jumps"
"over"
"lazy"
"dog"

But in my personal opinion that obscures the intended purpose. The
complete alphabet is the shortest way to express the complete alphabet,
and thereby makes the intent the clearest. Anything else just muddles
the waters (IMHO).

Closes #509
  • Loading branch information
juleskers authored and petertseng committed Jan 30, 2017
1 parent ab2a8d5 commit 9b580e8
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions exercises/scrabble-score/canonical-data.json
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,11 @@
"description": "empty input",
"input": "",
"expected": 0
},
{
"description": "entire alphabet available",
"input": "abcdefghijklmnopqrstuvwxyz",
"expected": 87
}
]
}

0 comments on commit 9b580e8

Please sign in to comment.