Should be fixed.

For c++.
The tests breaks if you remove "using namespace std;"
It should not depend on the user code having this using in it.

Either have the using namespace in the actual test. Or better, just type std:: where it is needed.

the C++ tests have trailing newlines

technically that is correct. there's nothing saying there won't be or that rows will be the same width.

however. it is poorly communicated (as in, misleading) and it is different from other languges and is thus indeed an issue.

....
....
....
...X
....
....
....
....
....
....
Expected: equal to "[3, 7]"
Actual: "[3, 6]"

If 0,0 is bottom left corner then this test case is wrong..

Enabled in this fork

JS fork resolving this issue

Awesome challenge, very fun!

Good challenge! One note: the random tests include n as a positive integer. Its easy enough to handle but the description does state:"Given an odd number, n". Again nice challenge!

Enabled in this fork

that's not an issue, she does already know it ;-)

Why does Vicky know the work 'thank' in sample test case eventhough she hasn't been taught,

fixed

please see this fork that should fix the issue

js fork (helped by katafix)

• uses mocha/chai assertions
• uses lodash for randomness
• adds random tests
• provides inputs on failure

Enabled in this fork