bugsnag / bugsnag-node / 285

79%

master: 40%

push

travis-ci

Better preparation for cloned object serialization

When Utils.cloneObject() was called on an instance of a class,
such as the mongo driver's `ObjectId` instance, the copy would
bring across its enumerable private properties, but none of its
methods, resulting in a bad serialization like so:

      "id": {
        "id": {
          "0": 88,
          "1": 129,
          "2": 222,
          "3": 117,
          "4": 187,
          "5": 221,
          "6": 189,
          "7": 57,
          "8": 86,
          "9": 86,
          "10": 65,
          "11": 202
        },
        "bsontype": "ObjectID"
      }

Now just before cloning, objects are checked for a toJSON() method.
If they have one, the result of that is used for the clone resulting
in better serialization, such as:

      { "oid": "5a9045f5be123c8c5d811d35" }

331 of 439 branches covered (75.4%)

Branch coverage included in aggregate %.

459 of 557 relevant lines covered (82.41%)

717.44 hits per line