npm / arborist / #374

push

Do not set fsParent to a link node

This was a weird side effect caused by the last bug, treating '..' as a
local node in the root's tree.  While that was fixed, it highlighted the
need for a better debug check on that sort of thing.

This adds a 'debug()' method, which takes a function.  The function is
ignored in normal operations, so it's perfectly fine to put expensive
and/or hazardously aggressive assertion crashes in there.  In tests,
methods passed to debug are always run, so we'll avoid creating new
scenarios where it could have been caught.

338 of 1913 branches covered (17.67%)

Branch coverage included in aggregate %.

518 of 2457 relevant lines covered (21.08%)

1.52 hits per line