two-fer: Add test for handling empty string param#2107
two-fer: Add test for handling empty string param#2107nguyendat wants to merge 1 commit intoexercism:masterfrom
Conversation
cmccandless
left a comment
There was a problem hiding this comment.
Test cases for this exercise are defined in the canonical-data. For more information on the structure of Exercism track repositories, please see the Contributing Guide
|
@nguyendat I'm afraid this test case doesn't really make sense, and neither does the discussion on that exercise: the purpose of a default value is to be used only if no argument whatsoever is supplied for that parameter ... if any string-convertible argument is provided the argument should be used, even if it's nonsensical, otherwise you're second guessing your user's intent, and the user is allowed to have nonsensical intent. So the "correct" response for I'm going to close this as wontfix for that reason, but thank you for the contribution. |
Ref: https://exercism.io/solutions/a1ec5f564aba409b8377f25ef1b80cee/comments/162304