digidem / mapeo-core

Library for creating custom geo data and syncronizing via a peer to peer network
23 stars 2 forks source link

sync test failing with node 14.18.0 #127

Open achou11 opened 2 years ago

achou11 commented 2 years ago

Noticing in CI that running the tests on 14.18.0 (latest LTS version at the time of writing, released 2021-09-28) produces an error for this test case:

# sync: missing data still ends
ok 788 written no error
ok 789 written no error
ok 790 written no error
ok 791 written no error
ok 792 api 1 has peers
ok 793 api 2 has peers
ok 794 syncronizing 0
ok 795 started is true
ok 796 error on first ok
ok 797 emit down event on close
ok 798 not connected anymore
ok 799 closed first api
ok 800 syncronizing 1
ok 801 error message for missing data
ok 802 emit down event on close
ok 803 not connected anymore
ok 804 syncronizing 2
node: ../deps/leveldb/leveldb-1.20/db/version_set.cc:798: leveldb::VersionSet::~VersionSet(): Assertion `dummy_versions_.next_ == &dummy_versions_' failed.
Aborted (core dumped)
npm ERR! Test failed.  See above for more details.
Error: Process completed with exit code 1.

Was able to reproduce this locally and confirm that 14.18.0 introduced this error. The changelog for it is pretty massive but from a quick search, could be an issue with leveldown so may be a matter of updating some dep(s) to get this working

achou11 commented 2 years ago

same failure occurs for 12.19.X