search

Level / abstract-leveldown

An abstract prototype matching the leveldown API.
MIT License
146 stars 53 forks source link

Lift out data to file scope to get rid of extra parameter #275

Closed ralphtheninja closed 6 years ago

ralphtheninja commented 6 years ago

Closes https://github.com/Level/abstract-leveldown/issues/270

I solved this differently, since test/self.js is calling setUp() and tearDown() we can't export a single .all() method.

Upgrade guide needs updating, because now all test methods have the same api.

vweevers commented 6 years ago

Upgrade guide needs updating, because now all test methods have the same api.

I figure we can just remove that section, because abstract-leveldown/test is now the preferred entry point. The internals no longer matter for consumers.