From 771d019ebb6849f4ff497d55f5d488db6d6e87dd Mon Sep 17 00:00:00 2001 From: delvedor Date: Fri, 6 Nov 2020 18:46:13 +0100 Subject: [PATCH] Standard --- index.js | 8 +++++--- test/errors.test.js | 20 ++++++++++---------- test/version.custom-versioning.test.js | 2 +- 3 files changed, 16 insertions(+), 14 deletions(-) diff --git a/index.js b/index.js index dff3f72..4126593 100644 --- a/index.js +++ b/index.js @@ -215,12 +215,14 @@ Router.prototype._insert = function _insert (method, path, kind, params, handler // let's split the node and add a new child if (len < prefixLen) { node = new Node( - { prefix: prefix.slice(len), + { + prefix: prefix.slice(len), children: currentNode.children, kind: currentNode.kind, handlers: new Node.Handlers(currentNode.handlers), regex: currentNode.regex, - versions: currentNode.versions } + versions: currentNode.versions + } ) if (currentNode.wildcardChild !== null) { node.wildcardChild = currentNode.wildcardChild @@ -585,7 +587,7 @@ Router.prototype.prettyPrint = function () { } for (var i in http.METHODS) { - if (!http.METHODS.hasOwnProperty(i)) continue + if (!http.METHODS.hasOwnProperty(i)) continue // eslint-disable-line const m = http.METHODS[i] const methodName = m.toLowerCase() diff --git a/test/errors.test.js b/test/errors.test.js index fb35536..7b794f1 100644 --- a/test/errors.test.js +++ b/test/errors.test.js @@ -181,7 +181,7 @@ test('Method already declared', t => { findMyWay.on('GET', '/test', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test\'') } }) @@ -198,14 +198,14 @@ test('Method already declared [ignoreTrailingSlash=true]', t => { findMyWay.on('GET', '/test', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test\'') } try { findMyWay.on('GET', '/test/', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test/'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test/\'') } }) @@ -219,14 +219,14 @@ test('Method already declared [ignoreTrailingSlash=true]', t => { findMyWay.on('GET', '/test', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test\'') } try { findMyWay.on('GET', '/test/', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test/'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test/\'') } }) }) @@ -243,7 +243,7 @@ test('Method already declared nested route', t => { findMyWay.on('GET', '/test/hello', () => {}) t.fail('method already delcared in nested route') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test/hello'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test/hello\'') } }) @@ -262,14 +262,14 @@ test('Method already declared nested route [ignoreTrailingSlash=true]', t => { findMyWay.on('GET', '/test/hello', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test/hello'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test/hello\'') } try { findMyWay.on('GET', '/test/hello/', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test/hello/'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test/hello/\'') } }) @@ -285,14 +285,14 @@ test('Method already declared nested route [ignoreTrailingSlash=true]', t => { findMyWay.on('GET', '/test/hello', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test/hello'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test/hello\'') } try { findMyWay.on('GET', '/test/hello/', () => {}) t.fail('method already declared') } catch (e) { - t.is(e.message, `Method 'GET' already declared for route '/test/hello/'`) + t.is(e.message, 'Method \'GET\' already declared for route \'/test/hello/\'') } }) }) diff --git a/test/version.custom-versioning.test.js b/test/version.custom-versioning.test.js index 2f70947..d4b18e2 100644 --- a/test/version.custom-versioning.test.js +++ b/test/version.custom-versioning.test.js @@ -17,7 +17,7 @@ const customVersioning = { } }, deriveVersion: (req, ctx) => { - return req.headers['accept'] + return req.headers.accept } }