Remove old test/common.js

Author: vweevers

.npmignore

@@ -1,5 +1,4 @@
 test
-buster.js
 .airtap.yml
 .travis.yml
 .dntrc

buster.js

@@ -10,7 +10,7 @@
     "test-browsers": "airtap --loopback airtap.local test/index.js",
     "test-browser-local": "airtap --local test/index.js",
     "hallmark": "hallmark --fix",
-    "dependency-check": "dependency-check . buster.js test/*.js",
+    "dependency-check": "dependency-check . test/*.js",
     "prepublishOnly": "npm run dependency-check"
   },
   "dependencies": {
@@ -24,7 +24,6 @@
     "airtap": "^2.0.0",
     "async-each": "^1.0.3",
     "browserify": "^16.0.0",
-    "bustermove": "^1.0.0",
     "concat-stream": "^2.0.0",
     "coveralls": "^3.0.2",
     "delayed": "^2.0.0",
@@ -37,7 +36,6 @@
     "memdown": "^5.0.0",
     "nyc": "^14.0.0",
     "pinkie": "^2.0.4",
-    "referee": "^1.2.0",
     "run-parallel": "^1.1.9",
     "run-series": "^1.1.8",
     "safe-buffer": "^5.1.0",

package.json

@@ -1,98 +1,29 @@
-var referee = require('referee')
-var assert = referee.assert
-var refute = referee.refute
-var each = require('async-each')
-var delayed = require('delayed').delayed
-var levelup = require('../lib/levelup.js')
-var errors = require('level-errors')
-var memdown = require('memdown')
-var encDown = require('encoding-down')
-
-assert(levelup.errors === errors)
-
-referee.add('isInstanceOf', {
-  assert: function (actual, expected) {
-    return actual instanceof expected
-  },
-  refute: function (actual, expected) {
-    return !(actual instanceof expected)
-  },
-  assertMessage: '${0} expected to be instance of ${1}', // eslint-disable-line
-  refuteMessage: '${0} expected not to be instance of ${1}' // eslint-disable-line
-})
-
-referee.add('isUndefined', {
-  assert: function (actual) {
-    return actual === undefined
-  },
-  refute: function (actual) {
-    return actual !== undefined
-  },
-  assertMessage: '${0} expected to be undefined', // eslint-disable-line
-  refuteMessage: '${0} expected not to be undefined' // eslint-disable-line
-})
-
-module.exports.openTestDatabase = function () {
-  var options = typeof arguments[0] === 'object' ? arguments[0] : {}
-  var callback = typeof arguments[0] === 'function' ? arguments[0] : arguments[1]
-
-  levelup(encDown(memdown(), options), function (err, db) {
-    refute(err)
-    if (!err) {
-      this.closeableDatabases.push(db)
-      callback(db)
-    }
-  }.bind(this))
-}
-
-module.exports.commonTearDown = function (done) {
-  each(this.closeableDatabases, function (db, callback) {
-    db.close(callback)
-  }, done)
-}
-
-module.exports.commonSetUp = function (done) {
-  this.closeableDatabases = []
-  this.openTestDatabase = module.exports.openTestDatabase.bind(this)
-  this.timeout = 10000
-  process.nextTick(done)
-}
-
-module.exports.readStreamSetUp = function (done) {
-  module.exports.commonSetUp.call(this, function () {
-    var i
-    var k
-
-    this.dataSpy = this.spy()
-    this.endSpy = this.spy()
-    this.sourceData = []
-
-    for (i = 0; i < 100; i++) {
-      k = (i < 10 ? '0' : '') + i
-      this.sourceData.push({
-        type: 'put',
-        key: k,
-        value: Math.random()
-      })
-    }
-
-    this.verify = delayed(function (rs, done, data) {
-      if (!data) data = this.sourceData // can pass alternative data array for verification
-      assert.equals(this.endSpy.callCount, 1, 'ReadStream emitted single "end" event')
-      assert.equals(this.dataSpy.callCount, data.length, 'ReadStream emitted correct number of "data" events')
-      data.forEach(function (d, i) {
-        var call = this.dataSpy.getCall(i)
-        if (call) {
-          assert.equals(call.args.length, 1, 'ReadStream "data" event #' + i + ' fired with 1 argument')
-          refute.isNull(call.args[0].key, 'ReadStream "data" event #' + i + ' argument has "key" property')
-          refute.isNull(call.args[0].value, 'ReadStream "data" event #' + i + ' argument has "value" property')
-          assert.equals(call.args[0].key, d.key, 'ReadStream "data" event #' + i + ' argument has correct "key"')
-          assert.equals(+call.args[0].value, +d.value, 'ReadStream "data" event #' + i + ' argument has correct "value"')
-        }
-      }.bind(this))
-      done()
-    }, 0.05, this)
-
-    done()
-  }.bind(this))
+// Same interface as abstract-leveldown's testCommon
+module.exports = function testCommon (options) {
+  var factory = options.factory
+  var test = options.test
+
+  if (typeof factory !== 'function') {
+    throw new TypeError('factory must be a function')
+  }
+
+  if (typeof test !== 'function') {
+    throw new TypeError('test must be a function')
+  }
+
+  return {
+    test: test,
+    factory: factory,
+
+    bufferKeys: options.bufferKeys !== false,
+    // createIfMissing: options.createIfMissing !== false,
+    // errorIfExists: options.errorIfExists !== false,
+    snapshots: options.snapshots !== false,
+    seek: options.seek !== false,
+    clear: !!options.clear,
+    deferredOpen: !!options.deferredOpen,
+    promises: !!options.promises,
+    streams: !!options.streams,
+    encodings: !!options.encodings
+  }
 }

test/common.js

@@ -8,7 +8,7 @@ var memdown = require('memdown')
 var encode = require('encoding-down')
 var levelup = require('../lib/levelup')
 
-var testCommon = require('./common2')({
+var testCommon = require('./common')({
   test: test,
   factory: function (options) {
     return levelup(encode(memdown(), options))