Skip to content

Commit

Permalink
Added a positive case without unit coins available
Browse files Browse the repository at this point in the history
All the current positive test cases include 1 unit coin in available coins and it's could be confusing.

This commit proposes a new test case that is positive but not includes 1 unit coin in the input.
  • Loading branch information
weapp committed Mar 2, 2017
1 parent fd73e47 commit df9eb7b
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions exercises/change/canonical-data.json
Original file line number Diff line number Diff line change
Expand Up @@ -36,6 +36,12 @@
"target": 999,
"expected": [2, 2, 5, 20, 20, 50, 100, 100, 100, 100, 100, 100, 100, 100, 100]
},
{
"description": "possible change without unit coins available",
"coins": [2, 5, 10, 20, 50],
"target": 21,
"expected": [2, 2, 2, 5, 10]
},
{
"description": "no coins make 0 change",
"coins": [1, 5, 10, 21, 25],
Expand Down

0 comments on commit df9eb7b

Please sign in to comment.