• it block of random tests should be within the describe block

• Random test should not be fixed to generate string of size 15 only

• Assertion messages could be more helpful in readability by adding () around the input string

Approved

Random tests have been lowered to 55, that is already a very low number. Node 18 enabled too, so your solution now runs 2x faster at the very least compared to Node 8, so closing then ^^

Approved

Approved

Perhaps some reading might help, though this kata does not require to handle default values, only handling of empty list as an edge case is needed.

Approved by someone

^^

Much better, now perhaps it would be much clearer if users know that Input1, Input2, Input3 correspond to s, w, h

Added by Johan above

No need...

Just --> Write a function that ...

As for examples -> You can use something like s = ..., w = ..., h = ... or in the form of Input1, Input2, Input3 --> Output

Use repr or wrap the input in quotes for it block, otherwise it can be easily misleaded as inputs being integers

Could you also remove make_grid and use something more generalized, to be compatible with other languages ?

2nd example is wrong, should be

"ABCDE", 3, 7

In the meantime, you should make description language-agnostic

The other kata is now available