Details states: "To make things simpler, it will only have to deal with simple values and objects and arrays containing strings, booleans and numbers, without taking into account regular expressiones, dates and functions"
One of the submit tests includes an array with an object.
[ 1, 2, null, undefined, { name: 'Joe' } ]
I recommend to 1) update the detail to state arrays containing objects, string, booleans, and numbers or 2) remove the test.
For the second test I think it would be helpful to just compare 1 and "string" (then we can directly understand what is the test about). It took me few second to understand that one was a string and another a number.
Otherwise good kata. Simple by the definition but more complex in the resolution.
JS: Node v12 should be used enabled along with its appropiate assertion tools (Mocha + Chai).
Pretty much a duplicate of these 2 katas.
The function name should be in camelCase.
The issue below is not fixed.
No random tests.
This comment is hidden because it contains spoiler information about the solution
Would be nice to have this test case:
Test.expect(deepCompare({name: 'Joe', test: undefined}, {test1: undefined, name: 'Joe'}));
Details states: "To make things simpler, it will only have to deal with simple values and objects and arrays containing strings, booleans and numbers, without taking into account regular expressiones, dates and functions"
One of the submit tests includes an array with an object.
[ 1, 2, null, undefined, { name: 'Joe' } ]
I recommend to 1) update the detail to state arrays containing objects, string, booleans, and numbers or 2) remove the test.
For the second test I think it would be helpful to just compare 1 and "string" (then we can directly understand what is the test about). It took me few second to understand that one was a string and another a number.
Otherwise good kata. Simple by the definition but more complex in the resolution.