Handle circular references better

[fregante]

Handle circular references better

[koddsson]

Oh wow, let me take a look at this. Ping me if I forget. I can at least reproduce it so that's something:

  1) assert
       lengthOf doesn't crash:
     RangeError: Maximum call stack size exceeded
      at inspectObject (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/object.js:2:23)
      at inspectClass (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/class.js:13:20)
      at Object.inspect (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/index.js:118:14)
      at inspectProperty (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/helpers.js:144:19)
      at inspectList (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/helpers.js:108:28)
      at inspectObject (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/object.js:14:28)
      at inspectClass (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/class.js:13:20)
      at Object.inspect (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/index.js:118:14)
      at inspectProperty (file:///Users/cr91ew/src/koddsson/chai/node_modules/loupe/lib/helpers.js:144:19)

[fregante]

When generating the preview, would it make sense to automatically limit the size?

For example it's useful to see the message Expected [0, 2, 8] to have length 4, but it becomes unreadable when the array is huge or when it contains large objects.

An example limit for this specific case could look like arr.length * Object.keys(arr[0]).length < 20, although I know it might be difficult to translate it into real code. This might also be a choice of the caller rather than loupe's itself.

I know that Node's inspect won't try to go too deep into objects, so such a limit alone should solve circular references permanently.