fix(errors): clean up organization of errors to separate file (#37)

Co-Authored-By: Liran Tal <[email protected]>

errors/index.js

@@ -0,0 +1,11 @@
+'use strict'
+class RouteVersionUnmatchedError extends Error {
+  constructor (message) {
+    super(message)
+    this.name = this.constructor.name
+  }
+}
+
+module.exports = {
+  RouteVersionUnmatchedError
+}

index.js

@@ -1,19 +1,9 @@
 'use strict'
 
 const semver = require('semver')
-
-class RouteVersionUnmatchedError extends Error {
-  constructor (message) {
-    super(message)
-    this.name = this.constructor.name
-  }
-}
+const { RouteVersionUnmatchedError } = require('./errors')
 
 class versionRouter {
-  static get RouteVersionUnmatchedError () {
-    return RouteVersionUnmatchedError
-  }
-
   static route (versionsMap = new Map(), options = new Map()) {
     return (req, res, next) => {
       for (let [versionKey, versionRouter] of versionsMap) {

test/sanity.test.js

@@ -2,6 +2,7 @@
 
 const test = require('ava')
 const versionRouter = require('../index')
+const { RouteVersionUnmatchedError } = require('../errors')
 
 test('given a versioned router, match the route', t => {
   const v1 = '1.0'
@@ -64,7 +65,7 @@ test('given a versioned router, dont match the requestVersion and error out if n
   const result = middleware(req, resIn, nextHandler)
   t.falsy(resIn)
   t.truthy(result instanceof Error)
-  t.truthy(result instanceof versionRouter.RouteVersionUnmatchedError)
+  t.truthy(result instanceof RouteVersionUnmatchedError)
   t.truthy(result.name === 'RouteVersionUnmatchedError')
   t.truthy(result.message === `${requestedVersion} doesn't match any versions`)
 })