-
-
Notifications
You must be signed in to change notification settings - Fork 540
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
word-count: Add test for multiple whitespace #1023
Conversation
@@ -111,6 +111,15 @@ | |||
"large": 2, | |||
"and": 1 | |||
} | |||
}, | |||
{ | |||
"description": "with multiple whitespace", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A more helpful description would be nice.
What is it about multiple whitespace that makes it interesting to test?
"space surrounded by spaces not detected as a word"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks - a valid point. I fell into the trap of just imitating the overly brief descriptions from some of the prior test cases...
Thanks @ErikSchierboom! |
Added two tests: * Multiple whitespace - exercism/problem-specifications#1023 * Empty words - exercism/problem-specifications#1446
Added two tests: * Multiple whitespace - exercism/problem-specifications#1023 * Empty words - exercism/problem-specifications#1446
Adds three test cases: * substrings_from_the_beginning (exercism/problem-specifications#1557) * multiple_spaces_not_detected_as_a_word (exercism/problem-specifications#1023) * alternating_word_separators_not_detected_as_a_word (exercism/problem-specifications#1446)
* Word-count - update tests specification to version 1.4.0 Adds three test cases: * substrings_from_the_beginning (exercism/problem-specifications#1557) * multiple_spaces_not_detected_as_a_word (exercism/problem-specifications#1023) * alternating_word_separators_not_detected_as_a_word (exercism/problem-specifications#1446) * Word-count - change test cases to be more like the specification (1.4.0) This reorders and removes some tests to match the specification in version 1.4.0 * Word-count - make empty lines uniform in test cases
See #1022 for reference