Improve test coverage

Author: JoshMock

test/node-test/agent.js

@@ -773,13 +773,20 @@ test('stream: fails with invalid onInfo', async (t) => {
   }
 })
 
-test('constructor validations', t => {
-  const p = tspl(t, { plan: 5 })
+test('Agent constructor validations', t => {
+  const p = tspl(t, { plan: 3 })
   p.throws(() => new Agent({ factory: 'ASD' }), errors.InvalidArgumentError, 'throws on invalid opts argument')
   p.throws(() => new Agent({ maxOrigins: -1 }), errors.InvalidArgumentError, 'maxOrigins must be a number greater than 0')
   p.throws(() => new Agent({ maxOrigins: 'foo' }), errors.InvalidArgumentError, 'maxOrigins must be a number greater than 0')
-  p.throws(() => new Pool({ maxFree: -1 }), errors.InvalidArgumentError, 'maxFree must be a number greater than 0')
-  p.throws(() => new Pool({ maxFree: 'foo' }), errors.InvalidArgumentError, 'maxFree must be a number greater than 0')
+})
+
+test('Pool constructor validatons', t => {
+  const p = tspl(t, { plan: 5 })
+  p.throws(() => new Pool({ connections: 'ASD' }), errors.InvalidArgumentError, 'throws on non-number')
+  p.throws(() => new Pool({ connect: 'ASD' }), errors.InvalidArgumentError, 'throws if not a function or object')
+  p.throws(() => new Pool({ factory: 'ASD' }), errors.InvalidArgumentError, 'throws on non-function')
+  p.throws(() => new Pool({ maxFree: -1 }), errors.InvalidArgumentError, 'throws on negative number')
+  p.throws(() => new Pool({ maxFree: 'foo' }), errors.InvalidArgumentError, 'throws on non-number')
 })
 
 test('dispatch validations', async t => {