inject function return normal value with non-ready app (#2417)

* inject return normal value with non-ready app

* remove async/await using in node 6 version

fastify.js

@@ -352,8 +352,15 @@ function build (options) {
         else lightMyRequest(httpHandler, opts, cb)
       })
     } else {
-      return this.ready()
-        .then(() => lightMyRequest(httpHandler, opts))
+      return lightMyRequest((req, res) => {
+        this.ready(function (err) {
+          if (err) {
+            res.emit('error', err)
+            return
+          }
+          httpHandler(req, res)
+        })
+      }, opts)
     }
   }
 

test/inject.test.js

@@ -409,3 +409,18 @@ test('should throw error if callback specified and if ready errors', t => {
     t.strictEqual(err, error)
   })
 })
+
+test('should support builder-style injection with non-ready app', (t) => {
+  t.plan(3)
+  const fastify = Fastify()
+  const payload = { hello: 'world' }
+
+  fastify.get('/', (req, reply) => {
+    reply.send(payload)
+  })
+  fastify.inject().get('/').end().then((res) => {
+    t.deepEqual(payload, JSON.parse(res.payload))
+    t.strictEqual(res.statusCode, 200)
+    t.strictEqual(res.headers['content-length'], '17')
+  })
+})