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.

Sign up for GitHub

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

luhn: tweak canonical tests #547

Merged
merged 2 commits into from
Feb 21, 2017
Merged

luhn: tweak canonical tests #547

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
aff180a
luhn: tweak canonical tests
stkent Feb 12, 2017
9d5b690
luhn: further adjustments to canonical test suite
stkent Feb 12, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 17 additions & 17 deletions exercises/luhn/canonical-data.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,18 +11,23 @@
"expected": false
},
{
"description": "simple valid sin",
"input": " 5 9 ",
"description": "a simple valid SIN that remains valid if reversed",
"input": "059",
"expected": true
},
{
"description": "valid Canadian SIN",
"input": "046 454 286",
"description": "a simple valid SIN that becomes invalid if reversed",
"input": "59",
"expected": true
},
{
"description": "a valid Canadian SIN",
"input": "055 444 285",
"expected": true
},
{
"description": "invalid Canadian SIN",
"input": "046 454 287",
"input": "055 444 286",
"expected": false
},
{
Expand All @@ -31,17 +36,17 @@
"expected": false
},
{
"description": "valid strings with a non-digit added become invalid",
"input": "046a 454 286",
"description": "valid strings with a non-digit included become invalid",
"input": "055a 444 285",
"expected": false
},
{
"description": "punctuation is not allowed",
"description": "valid strings with punctuation included become invalid",
"input": "055-444-285",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compare to valid case 055 444 285.

"expected": false
},
{
"description": "symbols are not allowed",
"description": "valid strings with symbols included become invalid",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Compare to valid case 055 444 285.

"input": "055£ 444$ 285",
"expected": false
},
Expand All @@ -51,17 +56,12 @@
"expected": false
},
{
"description": "lots of zeros are valid",
"input": " 00000",
"expected": true
},
{
"description": "another valid sin",
"input": "055 444 285",
"description": "more than a single zero is valid",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated to state the limitation here more clearly, and moved the space character to indicate that a "chain" of zeros is not required.

"input": "0000 0",
"expected": true
},
{
"description": "nine doubled is nine",
"description": "input digit 9 is correctly converted to output digit 9",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nine doubled is not nine; tried to make this a little more explicit!

"input": "091",
"expected": true
}
Expand Down