Handle null values when comparing objects

[posva]

Fix #1566

@callumacrae I couldn't see why the error was happening in circleci (infinite loading) and wanted to change some things, so I created a new PR. I hope this doesn't bother you.
To answer your question, the problem was there was a non-caught exception, so I tested that and equality as well. But I should test that as well, let me add a commit for that

[callumacrae]

What should happen with { foo: null } and { foo: undefined }? Should they be equal?

[posva]

IMO no, but I also wrote that code 😆
As a developer, you have to be consistent, I think, so either use null or undefined (I'd choose null)

[javoski]

Maybe a shared util function isObject will be helpful, it also can be reused in many places in this repo.

function isObject (val) {
  return val !== null && typeof val === 'object'
}

[anxolin]

+1

[posva]

ping @yyx990803
@javoski I haven't checked, but yeah, maybe we could refactor some code

[webrsb]

Resolve #1722