do not quit given client on close (#25)

* do not quit given client on close * typo

Author: mcollina

README.md

@@ -12,7 +12,8 @@ npm i fastify-redis --save
 ```
 ## Usage
 Add it to your project with `register` and you are done!
-You can access the *Redis* client via `fastify.redis`.
+You can access the *Redis* client via `fastify.redis`. The client is
+automatically closed when the fastify instance is closed.
 
 ```js
 const fastify = require('fastify')
@@ -40,7 +41,9 @@ fastify.listen(3000, err => {
 ```
 
 You may also supply an existing *Redis* client instance by passing an options
-object with the `client` property set to the instance.
+object with the `client` property set to the instance. In this case,
+the client is not automatically closed when the Fastify instance is
+closed.
 
 ```js
 const fastify = Fastify()

index.js

@@ -12,11 +12,10 @@ function fastifyRedis (fastify, options, next) {
  } catch (err) {
  return next(err)
  }
+ fastify.addHook('onClose', close)
  }
 
- fastify
- .decorate('redis', client)
- .addHook('onClose', close)
+ fastify.decorate('redis', client)
 
  next()
 }

test.js

@@ -84,7 +84,7 @@ test('promises support', t => {
 })
 
 test('custom client', t => {
- t.plan(5)
+ t.plan(7)
  const fastify = Fastify()
  const redis = require('redis').createClient({ host: 'localhost', port: 6379 })
 
@@ -100,7 +100,12 @@ test('custom client', t => {
  t.error(err)
  t.equal(val, 'value')
 
- fastify.close()
+ fastify.close(function (err) {
+ t.error(err)
+ fastify.redis.quit(function (err) {
+ t.error(err)
+ })
+ })
  })
  })
  })