search

Level / abstract-leveldown

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

Remove sync test #300

Closed ralphtheninja closed 6 years ago

ralphtheninja commented 6 years ago

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

vweevers commented 6 years ago

@ralphtheninja reading https://github.com/Level/leveldown/issues/157 and the PR that introduced these sync tests (https://github.com/Level/abstract-leveldown/pull/58), I now understand they were about triggering segfaults. I just tested it on latest leveldown, still segfaults. So I'll add the test to leveldown instead, but skip it, and add a comment with some background.