Merge pull request #20 from allevo/fix-unknown-method

Only http.METHODS

Author: delvedor

index.js

@@ -1,6 +1,7 @@
 'use strict'
 
 const assert = require('assert')
+const http = require('http')
 const Ajv = require('ajv')
 const Request = require('./lib/request')
 const Response = require('./lib/response')
@@ -41,7 +42,7 @@ const schema = {
     },
     authority: { type: 'string' },
     remoteAddress: { type: 'string' },
-    method: { type: 'string' },
+    method: { type: 'string', enum: http.METHODS.concat(http.METHODS.map(toLowerCase)) },
     validate: { type: 'boolean' }
     // payload type => any
   },
@@ -83,5 +84,7 @@ function isInjection (obj) {
   return (obj instanceof Request || obj instanceof Response)
 }
 
+function toLowerCase (m) { return m.toLowerCase() }
+
 module.exports = inject
 module.exports.isInjection = isInjection

package.json

@@ -14,7 +14,10 @@
     "tap": "^11.0.1"
   },
   "scripts": {
-    "test": "standard && tap test/test.js"
+    "test": "npm run lint && npm run unit",
+    "lint": "standard",
+    "unit": "tap test/test.js",
+    "coverage": "npm run unit -- --cov --coverage-report=html"
   },
   "repository": {
     "type": "git",

test/async-await.js

@@ -10,7 +10,7 @@ function asyncAwaitTest (t, inject) {
     }
 
     try {
-      const res = await inject(dispatch, { method: 'get', url: 'http://example.com:8080/hello' })
+      const res = await inject(dispatch, { method: 'GET', url: 'http://example.com:8080/hello' })
       t.equal(res.payload, 'hello')
     } catch (err) {
       t.fail(err)
@@ -23,7 +23,7 @@ function asyncAwaitTest (t, inject) {
     }
 
     try {
-      await inject(dispatch, { method: 'get', url: 'http://example.com:8080/hello' })
+      await inject(dispatch, { method: 'GET', url: 'http://example.com:8080/hello' })
       t.fail('should throw')
     } catch (err) {
       t.ok(err)