Is there perhaps an error in the aBunchInARowTest for Java? When I "attempt" the kata all tests pass except this one which returns: #baf46 expected:true but was: false
Now, I am not sure how the test class has been written but #baf46 looks like invalid output to me so shouldn't it be false?
I talking about uppercase just because you write tweet.toLowerCase() in your code(I don't know what's the testcase) ;-)
Perhaps the real bug is: tweet.replaceAll("coverage", "covfefe"); --> tweet=tweet.replaceAll("coverage", "covfefe");
I fixed it. It must have something to do with the way the string is stored temporarily. Putting return inside the if/else statement fixes it and passes all tests.
No need to handle uppercase "Coverage"/"COVERAGE" by the way.
Is there perhaps an error in the aBunchInARowTest for Java? When I "attempt" the kata all tests pass except this one which returns: #baf46 expected:true but was: false
Now, I am not sure how the test class has been written but #baf46 looks like invalid output to me so shouldn't it be false?
This comment is hidden because it contains spoiler information about the solution
I completely agree. This is undoubtedly a good kata but oh boy is the description confusing.
I talking about
uppercase
just because you writetweet.toLowerCase()
in your code(I don't know what's the testcase) ;-)Perhaps the real bug is:
tweet.replaceAll("coverage", "covfefe");
-->tweet=tweet.replaceAll("coverage", "covfefe");
I fixed it. It must have something to do with the way the string is stored temporarily. Putting return inside the if/else statement fixes it and passes all tests.
No need to handle uppercase "Coverage"/"COVERAGE" by the way.
True, it can't. But shouldn't the basic tests pass anyway seeing that no uppercase is found in the first couple of test cases?
your code can not handle uppercase such as "Coverage". using regular expression is a better way ;-)
This comment is hidden because it contains spoiler information about the solution