log_controller_test_case: Fix deepEqual returning true for primitive types

[jcpascual]

This PR fixes deepEqual to return the correct value when comparing primitive types.

When comparing obj1 and obj2, deepEqual first performs a standard === check. If that fails, it then proceeds to do a "deep comparison" with each targets' properties. However, since Object.keys(obj) always returns an empty array for primitive types, this means that deepEqual will always return true when comparing primitives as there are no properties to compare. (For example, deepEqual(true, false) will result in true.)

To avoid this, we only perform the deep comparison if both targets are of type object. If they are not, we return false.

Fixes #532.

[radiantshaw]

@dhh If this PR looks good to you, can we please fast-forward merging this? Cause I'll need this for the PR I'm working on for resolving #530.

[marcoroth]

Thank you @jcpascual and @radiantshaw!

This looks good to me, let's get it merged!