fix: update binary encoding for levelup 2

License: MIT
Signed-off-by: Jacob Heun <[email protected]>

Author: jacobheun

package.json

@@ -41,6 +41,7 @@
   "homepage": "https://github.com/ipfs/js-datastore-level#readme",
   "dependencies": {
     "datastore-core": "~0.4.0",
+    "encoding-down": "^5.0.2",
     "interface-datastore": "~0.4.1",
     "level-js": "github:timkuijsten/level.js#idbunwrapper",
     "leveldown": "^3.0.2",

src/index.js

@@ -9,6 +9,7 @@ const levelup = require('levelup')
 const asyncFilter = require('interface-datastore').utils.asyncFilter
 const asyncSort = require('interface-datastore').utils.asyncSort
 const Key = require('interface-datastore').Key
+const encode = require('encoding-down')
 
 /**
  * A datastore backed by leveldb.
@@ -35,10 +36,9 @@ class LevelDatastore {
     }
 
     this.db = levelup(
-      database(path),
+      encode(database(path), { valueEncoding: 'binary' }),
       Object.assign({}, opts, {
-        compression: false, // same default as go
-        valueEncoding: 'binary'
+        compression: false // same default as go
       }),
       (err) => {
         // Prevent an uncaught exception error on duplicate locks

test/index.spec.js

@@ -18,11 +18,14 @@ describe('LevelDatastore', () => {
 
       levelStore.open((err) => {
         expect(err).to.not.exist()
-        expect(levelStore.db.db instanceof LevelDown).to.equal(true)
+        expect(levelStore.db.db.db instanceof LevelDown).to.equal(true)
         expect(levelStore.db.options).to.include({
           createIfMissing: true,
           errorIfExists: false
         })
+        expect(levelStore.db.db.codec.opts).to.include({
+          valueEncoding: 'binary'
+        })
         done()
       })
     })
@@ -36,7 +39,7 @@ describe('LevelDatastore', () => {
 
       levelStore.open((err) => {
         expect(err).to.not.exist()
-        expect(levelStore.db.db instanceof memdown).to.equal(true)
+        expect(levelStore.db.db.db instanceof memdown).to.equal(true)
         expect(levelStore.db.options).to.include({
           createIfMissing: true,
           errorIfExists: true