Quite an easy and simple kata.
C#: method name should be PascalCase (Please refer to implementation of backward compatibility here )
PascalCase
fixed
CoffeeScript: the range of random characters is 0 <= char code <= 1024 (but the description has 1023 as the upper bound).
0 <= char code <= 1024
1023
CoffeeScript uses randint to generate random characters and randint includes the upper bound.
randint
If more tweaks are required, we'll make another issue for those.
.
var
now that we'are at it...
it(`"${...}"`,...
I updated the test order.
so I guess I have an infinite loop somewhere...
I think it's still possible to push the changes, right?
Once back up, I'll rearrange to have all batches with smaller strings before other tests.
Loading collection data...
Quite an easy and simple kata.
C#: method name should be
PascalCase
(Please refer to implementation of backward compatibility here )fixed
CoffeeScript: the range of random characters is
0 <= char code <= 1024
(but the description has1023
as the upper bound).CoffeeScript uses
randint
to generate random characters andrandint
includes the upper bound.If more tweaks are required, we'll make another issue for those.
.
var
removed from code everywherefixed
now that we'are at it...
var
(neither the tests, but I don't see them, so...)it(`"${...}"`,...
messages?I updated the test order.
so I guess I have an infinite loop somewhere...
I think it's still possible to push the changes, right?
Once back up, I'll rearrange to have all batches with smaller strings before other tests.
Loading more items...