From afb6051282b4317f449e49102aa31ac2ea42ecfe Mon Sep 17 00:00:00 2001 From: "greenkeeper[bot]" <23040076+greenkeeper[bot]@users.noreply.github.com> Date: Thu, 11 Jul 2019 21:36:05 +0000 Subject: [PATCH 1/2] chore(package): update standard to version 13.0.1 --- package.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/package.json b/package.json index a31c36b0..02fe8570 100644 --- a/package.json +++ b/package.json @@ -40,7 +40,7 @@ "run-parallel": "^1.1.9", "run-series": "^1.1.8", "safe-buffer": "^5.1.0", - "standard": "^12.0.0", + "standard": "^13.0.1", "tape": "^4.7.0", "trickle": "0.0.2" }, From 9f880b749d6fa8291572808ba58b65fda13e237c Mon Sep 17 00:00:00 2001 From: Vincent Weevers Date: Fri, 12 Jul 2019 09:45:32 +0200 Subject: [PATCH 2/2] standard --fix --- buster.js | 4 ++-- test/argument-checking-test.js | 4 ++-- test/batch-test.js | 16 ++++++------- test/benchmarks/index.js | 4 ++-- test/benchmarks/tests/index.js | 24 +++++++++---------- test/binary-test.js | 4 ++-- test/create-stream-vs-put-racecondition.js | 4 ++-- test/deferred-open-test.js | 6 ++--- test/get-put-del-test.js | 4 ++-- test/idempotent-test.js | 4 ++-- test/init-test.js | 4 ++-- test/inject-encoding-test.js | 10 ++++---- test/iterator-test.js | 4 ++-- test/json-test.js | 8 +++---- test/key-value-streams-test.js | 4 ++-- test/maybe-error-test.js | 4 ++-- test/no-encoding-test.js | 4 ++-- test/null-and-undefined-test.js | 6 ++--- test/open-patchsafe-test.js | 4 ++-- test/read-stream-test.js | 28 +++++++++++----------- test/snapshot-test.js | 4 ++-- 21 files changed, 77 insertions(+), 77 deletions(-) diff --git a/buster.js b/buster.js index 0e20f166..0a233665 100644 --- a/buster.js +++ b/buster.js @@ -2,6 +2,6 @@ var config = module.exports config['unit'] = { environment: 'node', - tests: [ 'test/*-test.js' ], - libs: [ 'test/common.js' ] + tests: ['test/*-test.js'], + libs: ['test/common.js'] } diff --git a/test/argument-checking-test.js b/test/argument-checking-test.js index 16c84201..1017be50 100644 --- a/test/argument-checking-test.js +++ b/test/argument-checking-test.js @@ -3,8 +3,8 @@ var assert = require('referee').assert var buster = require('bustermove') buster.testCase('Argument checking', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'test get() throwables': function (done) { this.openTestDatabase(function (db) { diff --git a/test/batch-test.js b/test/batch-test.js index ff722177..0f08f2ce 100644 --- a/test/batch-test.js +++ b/test/batch-test.js @@ -8,8 +8,8 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('batch()', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'batch() with multiple puts': function (done) { this.openTestDatabase(function (db) { @@ -111,9 +111,9 @@ buster.testCase('batch()', { .write(function (err) { refute(err) - each([ 'one', 'three', '1', '2', '3' ], function (key, callback) { + each(['one', 'three', '1', '2', '3'], function (key, callback) { db.get(key, function (err) { - if ([ 'one', 'three', '1', '3' ].indexOf(key) > -1) { assert(err) } else { refute(err) } + if (['one', 'three', '1', '3'].indexOf(key) > -1) { assert(err) } else { refute(err) } callback() }) }, done) @@ -173,9 +173,9 @@ buster.testCase('batch()', { .del('3') .write() .then(function () { - each([ 'one', 'three', '1', '2', '3' ], function (key, callback) { + each(['one', 'three', '1', '2', '3'], function (key, callback) { db.get(key, function (err) { - if ([ 'one', 'three', '1', '3' ].indexOf(key) > -1) { assert(err) } else { refute(err) } + if (['one', 'three', '1', '3'].indexOf(key) > -1) { assert(err) } else { refute(err) } callback() }) }, done) @@ -281,7 +281,7 @@ buster.testCase('batch()', { }, 'chained batch() arguments': { - 'setUp': function (done) { + setUp: function (done) { this.openTestDatabase(function (db) { this.db = db this.batch = db.batch() @@ -357,7 +357,7 @@ buster.testCase('batch()', { }, 'test batch operations after write()': { - 'setUp': function (done) { + setUp: function (done) { this.batch.put('foo', 'bar').put('boom', 'bang').del('foo').write(done) this.verify = function (cb) { assert.exception(cb, function (err) { diff --git a/test/benchmarks/index.js b/test/benchmarks/index.js index fe8b1ebb..ecac837c 100644 --- a/test/benchmarks/index.js +++ b/test/benchmarks/index.js @@ -29,8 +29,8 @@ var rmdb = function (engine, db, location, callback) { var run = function (db, name, fn, color, cb) { var exec = function () { new Benchmark(name, { - 'defer': true, - 'fn': function (deferred) { + defer: true, + fn: function (deferred) { fn(db, deferred.resolve.bind(deferred)) } }) diff --git a/test/benchmarks/tests/index.js b/test/benchmarks/tests/index.js index ab9c6367..194ff36e 100644 --- a/test/benchmarks/tests/index.js +++ b/test/benchmarks/tests/index.js @@ -4,78 +4,78 @@ module.exports = { 'put(int, string) x 10': { - 'LevelUP': require('./put_int_string_x10_levelup'), + LevelUP: require('./put_int_string_x10_levelup'), 'LevelUP (release)': require('./put_int_string_x10_levelup'), 'LevelUP (no Snappy)': require('./put_int_string_x10_levelup') // , 'SQLite3' : require('./put_int_string_x10_sqlite3') }, 'put(int, string) x 1000': { - 'LevelUP': require('./put_int_string_x1000_levelup'), + LevelUP: require('./put_int_string_x1000_levelup'), 'LevelUP (release)': require('./put_int_string_x1000_levelup'), 'LevelUP (no Snappy)': require('./put_int_string_x1000_levelup') // , 'SQLite3' : require('./put_int_string_x1000_sqlite3') }, 'put(int, string) x 100,000': { - 'LevelUP': require('./put_int_string_x100000_levelup'), + LevelUP: require('./put_int_string_x100000_levelup'), 'LevelUP (release)': require('./put_int_string_x100000_levelup'), 'LevelUP (no Snappy)': require('./put_int_string_x100000_levelup') // too slow , 'SQLite3' : require('./put_int_string_x100000_sqlite3') }, 'get(int):string x 10': { - 'LevelUP': require('./get_int_string_x10_levelup'), + LevelUP: require('./get_int_string_x10_levelup'), 'LevelUP (release)': require('./get_int_string_x10_levelup'), 'LevelUP (no Snappy)': require('./get_int_string_x10_levelup') //, 'SQLite3' : require('./get_int_string_x10_sqlite3') }, 'get(int):string x 1000': { - 'LevelUP': require('./get_int_string_x1000_levelup'), + LevelUP: require('./get_int_string_x1000_levelup'), 'LevelUP (release)': require('./get_int_string_x1000_levelup'), 'LevelUP (no Snappy)': require('./get_int_string_x1000_levelup') //, 'SQLite3' : require('./get_int_string_x1000_sqlite3') }, 'batch(int, string) x 1000': { - 'LevelUP': require('./batch_int_string_x1000_levelup'), + LevelUP: require('./batch_int_string_x1000_levelup'), 'LevelUP (release)': require('./batch_int_string_x1000_levelup'), 'LevelUP (no Snappy)': require('./batch_int_string_x1000_levelup') }, 'batch(int, string) x 100,000': { - 'LevelUP': require('./batch_int_string_x100000_levelup'), + LevelUP: require('./batch_int_string_x100000_levelup'), 'LevelUP (release)': require('./batch_int_string_x100000_levelup'), 'LevelUP (no Snappy)': require('./batch_int_string_x100000_levelup') }, 'readStream x 10': { - 'LevelUP': require('./readStream_x10_levelup'), + LevelUP: require('./readStream_x10_levelup'), 'LevelUP (release)': require('./readStream_x10_levelup'), 'LevelUP (no Snappy)': require('./readStream_x10_levelup') }, 'readStream x 100': { - 'LevelUP': require('./readStream_x100_levelup'), + LevelUP: require('./readStream_x100_levelup'), 'LevelUP (release)': require('./readStream_x100_levelup'), 'LevelUP (no Snappy)': require('./readStream_x100_levelup') }, 'readStream x 1,000': { - 'LevelUP': require('./readStream_x1000_levelup'), + LevelUP: require('./readStream_x1000_levelup'), 'LevelUP (release)': require('./readStream_x1000_levelup'), 'LevelUP (no Snappy)': require('./readStream_x1000_levelup') }, 'readStream x 10,000': { - 'LevelUP': require('./readStream_x10000_levelup'), + LevelUP: require('./readStream_x10000_levelup'), 'LevelUP (release)': require('./readStream_x10000_levelup'), 'LevelUP (no Snappy)': require('./readStream_x10000_levelup') }, 'readStream x 100,000': { - 'LevelUP': require('./readStream_x100000_levelup'), + LevelUP: require('./readStream_x100000_levelup'), 'LevelUP (release)': require('./readStream_x100000_levelup'), 'LevelUP (no Snappy)': require('./readStream_x100000_levelup') } diff --git a/test/binary-test.js b/test/binary-test.js index 2ba9cd4c..f4bd78c4 100644 --- a/test/binary-test.js +++ b/test/binary-test.js @@ -5,14 +5,14 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('Binary API', { - 'setUp': function (done) { + setUp: function (done) { common.commonSetUp.call(this, function () { this.testData = loadData() done() }.bind(this)) }, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, 'sanity check on test data': function (done) { assert(Buffer.isBuffer(this.testData)) diff --git a/test/create-stream-vs-put-racecondition.js b/test/create-stream-vs-put-racecondition.js index 4f0ba379..d0f24237 100644 --- a/test/create-stream-vs-put-racecondition.js +++ b/test/create-stream-vs-put-racecondition.js @@ -57,9 +57,9 @@ function makeTest (db, delay, done) { } buster.testCase('ReadStream', { - 'setUp': common.readStreamSetUp, + setUp: common.readStreamSetUp, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, // TODO: test various encodings 'readStream and then put in nextTick': function (done) { diff --git a/test/deferred-open-test.js b/test/deferred-open-test.js index e789c790..9e419653 100644 --- a/test/deferred-open-test.js +++ b/test/deferred-open-test.js @@ -10,8 +10,8 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('Deferred open()', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'put() and get() on pre-opened database': function (done) { // 1) open database without callback, opens in next tick @@ -119,7 +119,7 @@ buster.testCase('Deferred open()', { }, 'test deferred ReadStream': { - 'setUp': common.readStreamSetUp, + setUp: common.readStreamSetUp, 'simple ReadStream': function (done) { var db = levelup(encDown(memdown())) diff --git a/test/get-put-del-test.js b/test/get-put-del-test.js index f1d8d7ac..a9335585 100644 --- a/test/get-put-del-test.js +++ b/test/get-put-del-test.js @@ -8,8 +8,8 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('get() / put() / del()', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'Simple operations': { 'get() on empty database causes error': function (done) { diff --git a/test/idempotent-test.js b/test/idempotent-test.js index 4ef3391e..42648c59 100644 --- a/test/idempotent-test.js +++ b/test/idempotent-test.js @@ -5,9 +5,9 @@ var assert = require('referee').assert var buster = require('bustermove') buster.testCase('Idempotent open & close', { - 'setUp': common.readStreamSetUp, + setUp: common.readStreamSetUp, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, 'call open twice, should emit "open" once': function (done) { var n = 0 diff --git a/test/init-test.js b/test/init-test.js index cc01300c..6cfd38e8 100644 --- a/test/init-test.js +++ b/test/init-test.js @@ -6,8 +6,8 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('Init & open()', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'levelup()': function () { assert.isFunction(levelup) diff --git a/test/inject-encoding-test.js b/test/inject-encoding-test.js index 794ace72..b323335f 100644 --- a/test/inject-encoding-test.js +++ b/test/inject-encoding-test.js @@ -9,7 +9,7 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('custom encoding', { - 'setUp': function (done) { + setUp: function (done) { common.commonSetUp.call(this, function () { this.runTest = function (testData, assertType, done) { var customEncoding = { @@ -46,7 +46,7 @@ buster.testCase('custom encoding', { }.bind(this)) }, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, 'simple-object values in "json" encoding': function (done) { this.runTest([ @@ -76,7 +76,7 @@ buster.testCase('custom encoding', { key: '0', value: { foo: 'bar', - bar: [ 1, 2, 3 ], + bar: [1, 2, 3], bang: { yes: true, no: false } } } @@ -91,7 +91,7 @@ buster.testCase('custom encoding', { value: '0', key: { foo: 'bar', - bar: [ 1, 2, 3 ], + bar: [1, 2, 3], bang: { yes: true, no: false } } }, @@ -99,7 +99,7 @@ buster.testCase('custom encoding', { value: '1', key: { foo: 'different', - bar: [ 1, 2, 3 ], + bar: [1, 2, 3], bang: { yes: true, no: false } } } diff --git a/test/iterator-test.js b/test/iterator-test.js index 2be707f3..0f5df3b7 100644 --- a/test/iterator-test.js +++ b/test/iterator-test.js @@ -7,8 +7,8 @@ var levelup = require('../lib/levelup') var common = require('./common') buster.testCase('iterator', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'test simple iterator': function (done) { var db = levelup(memdown()) diff --git a/test/json-test.js b/test/json-test.js index ece8c513..fc9a42f9 100644 --- a/test/json-test.js +++ b/test/json-test.js @@ -10,7 +10,7 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('JSON encoding', { - 'setUp': function (done) { + setUp: function (done) { common.commonSetUp.call(this, function () { this.runTest = function (testData, assertType, done) { levelup(encDown(memdown(), { @@ -51,7 +51,7 @@ buster.testCase('JSON encoding', { }.bind(this)) }, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, 'simple-object values in "json" encoding': function (done) { this.runTest([ @@ -79,7 +79,7 @@ buster.testCase('JSON encoding', { key: '0', value: { foo: 'bar', - bar: [ 1, 2, 3 ], + bar: [1, 2, 3], bang: { yes: true, no: false } } } @@ -92,7 +92,7 @@ buster.testCase('JSON encoding', { value: '0', key: { foo: 'bar', - bar: [ 1, 2, 3 ], + bar: [1, 2, 3], bang: { yes: true, no: false } } } diff --git a/test/key-value-streams-test.js b/test/key-value-streams-test.js index 3de2931c..9f1fb6b7 100644 --- a/test/key-value-streams-test.js +++ b/test/key-value-streams-test.js @@ -5,7 +5,7 @@ var buster = require('bustermove') var delayed = require('delayed').delayed buster.testCase('Key and Value Streams', { - 'setUp': function (done) { + setUp: function (done) { common.commonSetUp.call(this, function () { this.dataSpy = this.spy() this.endSpy = this.spy() @@ -43,7 +43,7 @@ buster.testCase('Key and Value Streams', { }.bind(this)) }, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, 'test .keyStream()': function (done) { this.openTestDatabase(function (db) { diff --git a/test/maybe-error-test.js b/test/maybe-error-test.js index 4f8ffaf4..104509e6 100644 --- a/test/maybe-error-test.js +++ b/test/maybe-error-test.js @@ -6,8 +6,8 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('maybeError() should be called async', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'put()': function (done) { this.openTestDatabase(function (db) { diff --git a/test/no-encoding-test.js b/test/no-encoding-test.js index 1df55941..88cbf3b6 100644 --- a/test/no-encoding-test.js +++ b/test/no-encoding-test.js @@ -6,8 +6,8 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('without encoding-down', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'serializes key': function (done) { var down = memdown() diff --git a/test/null-and-undefined-test.js b/test/null-and-undefined-test.js index 4ca19892..253b8684 100644 --- a/test/null-and-undefined-test.js +++ b/test/null-and-undefined-test.js @@ -7,11 +7,11 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('null & undefined keys & values', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'null and undefined': { - 'setUp': function (done) { + setUp: function (done) { levelup(memdown(), function (err, db) { refute(err) // sanity this.closeableDatabases.push(db) diff --git a/test/open-patchsafe-test.js b/test/open-patchsafe-test.js index 9f100809..dc58ebca 100644 --- a/test/open-patchsafe-test.js +++ b/test/open-patchsafe-test.js @@ -21,8 +21,8 @@ function test (fun) { } buster.testCase('Deferred open() is patch-safe', { - 'setUp': common.commonSetUp, - 'tearDown': common.commonTearDown, + setUp: common.commonSetUp, + tearDown: common.commonTearDown, 'put() on pre-opened database': test(function (db, done) { var put = db.put diff --git a/test/read-stream-test.js b/test/read-stream-test.js index 9f4559b9..4ffc8070 100644 --- a/test/read-stream-test.js +++ b/test/read-stream-test.js @@ -8,9 +8,9 @@ var buster = require('bustermove') var bigBlob = Array.apply(null, Array(1024 * 100)).map(function () { return 'aaaaaaaaaa' }).join('') buster.testCase('ReadStream', { - 'setUp': common.readStreamSetUp, + setUp: common.readStreamSetUp, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, // TODO: test various encodings @@ -381,14 +381,14 @@ buster.testCase('ReadStream', { var options = { keyEncoding: 'utf8', valueEncoding: 'json' } var data = [ { type: 'put', key: 'aa', value: { a: 'complex', obj: 100 } }, - { type: 'put', key: 'ab', value: { b: 'foo', bar: [ 1, 2, 3 ] } }, - { type: 'put', key: 'ac', value: { c: 'w00t', d: { e: [ 0, 10, 20, 30 ], f: 1, g: 'wow' } } }, + { type: 'put', key: 'ab', value: { b: 'foo', bar: [1, 2, 3] } }, + { type: 'put', key: 'ac', value: { c: 'w00t', d: { e: [0, 10, 20, 30], f: 1, g: 'wow' } } }, { type: 'put', key: 'ba', value: { a: 'complex', obj: 100 } }, - { type: 'put', key: 'bb', value: { b: 'foo', bar: [ 1, 2, 3 ] } }, - { type: 'put', key: 'bc', value: { c: 'w00t', d: { e: [ 0, 10, 20, 30 ], f: 1, g: 'wow' } } }, + { type: 'put', key: 'bb', value: { b: 'foo', bar: [1, 2, 3] } }, + { type: 'put', key: 'bc', value: { c: 'w00t', d: { e: [0, 10, 20, 30], f: 1, g: 'wow' } } }, { type: 'put', key: 'ca', value: { a: 'complex', obj: 100 } }, - { type: 'put', key: 'cb', value: { b: 'foo', bar: [ 1, 2, 3 ] } }, - { type: 'put', key: 'cc', value: { c: 'w00t', d: { e: [ 0, 10, 20, 30 ], f: 1, g: 'wow' } } } + { type: 'put', key: 'cb', value: { b: 'foo', bar: [1, 2, 3] } }, + { type: 'put', key: 'cc', value: { c: 'w00t', d: { e: [0, 10, 20, 30], f: 1, g: 'wow' } } } ] this.openTestDatabase(options, function (db) { @@ -414,14 +414,14 @@ buster.testCase('ReadStream', { } var data = [ { type: 'put', key: 'aa', value: { a: 'complex', obj: 100 } }, - { type: 'put', key: 'ab', value: { b: 'foo', bar: [ 1, 2, 3 ] } }, - { type: 'put', key: 'ac', value: { c: 'w00t', d: { e: [ 0, 10, 20, 30 ], f: 1, g: 'wow' } } }, + { type: 'put', key: 'ab', value: { b: 'foo', bar: [1, 2, 3] } }, + { type: 'put', key: 'ac', value: { c: 'w00t', d: { e: [0, 10, 20, 30], f: 1, g: 'wow' } } }, { type: 'put', key: 'ba', value: { a: 'complex', obj: 100 } }, - { type: 'put', key: 'bb', value: { b: 'foo', bar: [ 1, 2, 3 ] } }, - { type: 'put', key: 'bc', value: { c: 'w00t', d: { e: [ 0, 10, 20, 30 ], f: 1, g: 'wow' } } }, + { type: 'put', key: 'bb', value: { b: 'foo', bar: [1, 2, 3] } }, + { type: 'put', key: 'bc', value: { c: 'w00t', d: { e: [0, 10, 20, 30], f: 1, g: 'wow' } } }, { type: 'put', key: 'ca', value: { a: 'complex', obj: 100 } }, - { type: 'put', key: 'cb', value: { b: 'foo', bar: [ 1, 2, 3 ] } }, - { type: 'put', key: 'cc', value: { c: 'w00t', d: { e: [ 0, 10, 20, 30 ], f: 1, g: 'wow' } } } + { type: 'put', key: 'cb', value: { b: 'foo', bar: [1, 2, 3] } }, + { type: 'put', key: 'cc', value: { c: 'w00t', d: { e: [0, 10, 20, 30], f: 1, g: 'wow' } } } ] this.openTestDatabase(options, function (db) { diff --git a/test/snapshot-test.js b/test/snapshot-test.js index 68c33e2b..16ca19da 100644 --- a/test/snapshot-test.js +++ b/test/snapshot-test.js @@ -5,9 +5,9 @@ var refute = require('referee').refute var buster = require('bustermove') buster.testCase('Snapshots', { - 'setUp': common.readStreamSetUp, + setUp: common.readStreamSetUp, - 'tearDown': common.commonTearDown, + tearDown: common.commonTearDown, 'test ReadStream implicit snapshot': function (done) { this.openTestDatabase(function (db) {