raszi / node-tmp

Temporary file and directory creator for node.js
MIT License
736 stars 92 forks source link

Test for issue #129 are not working as required #178

Closed silkentrance closed 5 years ago

silkentrance commented 5 years ago

We need to either

The latter is not an option as it requires us to reimplement a lot of code for sandboxing the imports.

The first option seems to be good as we can then call the method multiple times during our tests.

And even if one calls upon the internal, then we are still good as it will cause no harm.

silkentrance commented 5 years ago

Sigh, the tests are working as expected. Needed to get my head around them.