search

Level / level-js

An abstract-leveldown compliant store on top of IndexedDB.
MIT License
544 stars 44 forks source link

Remove test/levelup-test.js #134

Closed ralphtheninja closed 6 years ago

ralphtheninja commented 6 years ago

Closes https://github.com/Level/level-js/issues/96

ralphtheninja commented 6 years ago

Also see https://github.com/Level/level-browserify/pull/47

vweevers commented 6 years ago

We can also rewrite the destroy tests to not use levelup. This:

https://github.com/Level/level-js/blob/2666a50887e3ff2136252f8a449aba0a9bcce9df/test/custom-test.js#L168

And

https://github.com/Level/level-js/blob/2666a50887e3ff2136252f8a449aba0a9bcce9df/test/custom-test.js#L191

Then we can remove the levelup dependency as well as the pinkie Promise polyfill.

ralphtheninja commented 6 years ago

@vweevers Yes, let's do that.