Adding server-destroy

Author: Ian Walter

index.js

@@ -2,6 +2,7 @@ const http = require('http')
 const Koa = require('koa')
 const json = require('koa-json')
 const log = require('@ianwalter/log')
+const enableDestroy = require('server-destroy')
 
 const defaultOptions = { cors: false }
 
@@ -36,18 +37,14 @@ module.exports = function createTestServer (options = defaultOptions) {
   // it receives.
   const server = http.createServer(app.callback())
 
+  // Add a destroy method to the server instance.
+  // https://github.com/nodejs/node/issues/2642
+  enableDestroy(server)
+
   // Add a close method to the Koa app to allow the caller / receiver of the Koa
   // app to close the server when done with it.
   app.close = function close () {
-    return new Promise((resolve, reject) => {
-      server.close(err => {
-        if (err) {
-          reject(err)
-        } else {
-          resolve(app)
-        }
-      })
-    })
+    return new Promise(resolve => server.destroy(resolve))
   }
 
   // Return the Koa app instance when the server has started listening.

package.json

@@ -32,7 +32,8 @@
   "dependencies": {
     "@ianwalter/log": "^2.0.0",
     "koa": "^2.7.0",
-    "koa-json": "^2.0.2"
+    "koa-json": "^2.0.2",
+    "server-destroy": "^1.0.1"
   },
   "devDependencies": {
     "@ianwalter/puppeteer-helper": "^3.0.0",

yarn.lock

@@ -4819,6 +4819,11 @@ serialize-javascript@^1.4.0:
   resolved "https://registry.yarnpkg.com/serialize-javascript/-/serialize-javascript-1.6.1.tgz#4d1f697ec49429a847ca6f442a2a755126c4d879"
   integrity sha512-A5MOagrPFga4YaKQSWHryl7AXvbQkEqpw4NNYMTNYUNV51bA8ABHgYFpqKx+YFFrw59xMV1qGH1R4AgoNIVgCw==
 
+server-destroy@^1.0.1:
+  version "1.0.1"
+  resolved "https://registry.yarnpkg.com/server-destroy/-/server-destroy-1.0.1.tgz#f13bf928e42b9c3e79383e61cc3998b5d14e6cdd"
+  integrity sha1-8Tv5KOQrnD55OD5hzDmYtdFObN0=
+
 set-blocking@~2.0.0:
   version "2.0.0"
   resolved "https://registry.yarnpkg.com/set-blocking/-/set-blocking-2.0.0.tgz#045f9782d011ae9a6803ddd382b24392b3d890f7"