Only remove modules that are not *.node from the cache

[hsablonniere]

Removing a *.node module from the cache and requiring it a 2nd time causes an Error :
Error: Module did not self-register.

See nodejs/node#5016

Fixes #108.

[hsablonniere]

Let me know if you want me to add some more tests specific for this.

[bertrandgressier]

👍

[bendrucker]

Would love a test before merging this @hsablonniere

[bendrucker]

Are you able to add a test? The behavior is okay until Node can allow us to reintroduce cache manipulation support for modules with native deps.

[hsablonniere]

Hey Ben,

I don't have much time these days to get back on this. Feel free to tag it and to close it.

Thx ;-)

[sonicdoe]

I would like to add a test for this. @bendrucker Would you rather have a test which actually requires a native module or one which just fakes an item in require.cache?

[bendrucker]

A real world integration test would be great