Nice catch, we weren't checking for numbers being present. I had started with the initial version instead of the other forks from JS and focused on translation instead of being more diligent.
I wouldn't consider all caps as a problem. You are welcome to add the test, but this was not an oversight on my part. Thank you for your participation.
It hasn't been listed as a test case, but would the case where no lower case letters are used be a problem? It's not defined as such in the kata description, but may have been assumed.
Example Test case:
Test.assertEquals(testPassword("ALLTH3TH!NGS"), true);
Weird, I get 40 vs 41 lines (not instructions) with your code vs the previous post with "-O6" option.
Nice catch, we weren't checking for numbers being present. I had started with the initial version instead of the other forks from JS and focused on translation instead of being more diligent.
Thanks for the response. I added a fork for C++ translation and added that as a test case.
I wouldn't consider all caps as a problem. You are welcome to add the test, but this was not an oversight on my part. Thank you for your participation.
It hasn't been listed as a test case, but would the case where no lower case letters are used be a problem? It's not defined as such in the kata description, but may have been assumed.
Example Test case:
Test.assertEquals(testPassword("ALLTH3TH!NGS"), true);