Merge pull request #23 from cemremengu/patch-1

Check duplicate connection names

Author: mcollina

index.js

@@ -29,6 +29,10 @@ function fastifyPostgres (fastify, options, next) {
       fastify.decorate('pg', {})
     }
 
+    if (fastify.pg[name]) {
+      return next(new Error('Connection name has already been registered: ' + name))
+    }
+
     fastify.pg[name] = db
   } else {
     if (fastify.pg) {

test.js

@@ -234,3 +234,22 @@ test('fastify.pg.test use native module', t => {
       })
   })
 })
+
+test('fastify.pg.test should throw with duplicate connection names', t => {
+  t.plan(1)
+
+  const fastify = Fastify()
+
+  fastify.register(fastifyPostgres, {
+    name: 'test',
+    connectionString: 'postgres://postgres@localhost/postgres'
+  })
+    .register(fastifyPostgres, {
+      name: 'test',
+      connectionString: 'postgres://postgres@localhost/postgres'
+    })
+
+  fastify.ready(err => {
+    t.is(err.message, 'Connection name has already been registered: test')
+  })
+})