Approved.

Removed TestFactory because it is too loud when tests are failing. Approved.

It's also got best practice votes so I interpret it as "a good combination of being maintainable and efficient but not something you would expect to see in production code."

Ali, the only notable aspect of this answer is it's verbosity. It's not clever...

check here please: https://docs.codewars.com/getting-started/solutions/#clever

Approved

Should return false with [3, 2, 5, 0, 5, 8, 2, 6] and [5, 8, 0, 6, 2, 3, 2, 5] ==> expected: <false> but was: <true>

i think your algorithm is not reliable: it treats 3 the same as "3", which goes against the description

Clever doesn't necessarily mean best practices. I once saw a solution that was worded in the longest way possible (like, absurdly long) and it got clever because most people got a laugh out of it, lol.

merge conflict

Needs to be updated to chai+assert before approval

Should use chai+assert, not test-compat

added, but i don't know any of the other languages. Can someone make sure it's correct? Thanks.

CoffeeScript fork fixing issue with undefined in random tests

Yes and that is because I am a numnnutt who forgot I have to make a fork that have to be approved again.