Level / level-browserify

No longer maintained: superseded by level v5.0.0.
MIT License
63 stars 16 forks source link

Fix tests that depends on fs #1

Closed mcollina closed 10 years ago

mcollina commented 10 years ago

Some tests of Level depend on the fs module, and they do not pass with testling and in the browser

juliangruber commented 10 years ago

unless....you use brfs

rvagg commented 10 years ago

you can just tell it to not use those tests: https://github.com/Level/level-packager/blob/master/test.js#L85

See level-mem's tests for example: https://github.com/Level/level-mem/blob/master/test.js

If putting a "browser":{"fs":false} in level-packager's package.json will help then go do that too.

mcollina commented 10 years ago

Thanks! All is fixed!