npm / arborist / #39

push

fix: avoid throwing type errors on inventory.add

Trying to load mock malformed testing data from the cli resulted in an
unexpected TypeError. The issue can be reproduced by having a falsy
funding property in a package.json, as seen here:
https://github.com/npm/cli/blob/abdf52879/test/tap/fund.js#L81

This commit fixes the problem by avoiding trying to retrieve the `type`
and `url` properties when having a `null` object reference.

PR-URL: https://github.com/npm/arborist/pull/76
Credit: @ruyadorno
Close: #76
Reviewed-by: @isaacs

1777 of 1777 branches covered (100.0%)

Branch coverage included in aggregate %.

2656 of 2656 relevant lines covered (100.0%)

231.87 hits per line