Make callback to pool.query optional

closes #585

Author: dougwilson

Changes.md

@@ -7,6 +7,7 @@ you spot any mistakes.
 ## HEAD
 
 * Use indexOf instead of for loops removing conn from pool #611
+* Make callback to `pool.query` optional like `conn.query` #585
 
 ## v2.1.1 (2014-03-13)
 

lib/Pool.js

@@ -128,6 +128,11 @@ Pool.prototype.query = function (sql, values, cb) {
     values = null;
   }
 
+  if (!cb) {
+    // Ignore results and errors if no cb supplied; matches connection.query
+    cb = function () {};
+  }
+
   var connection;
   var query = Connection.createQuery(sql, values, function (err, rows, fields) {
     connection.release();

test/integration/pool/test-query.js

@@ -8,7 +8,12 @@ pool.query('SELECT 1', function (err, _rows, _fields) {
   rows = _rows;
   fields = _fields;
 
-  pool.end();
+  // Should work without error
+  pool.query('SELECT SQL_ERROR');
+
+  process.nextTick(function () {
+    pool.end();
+  });
 });
 
 process.on('exit', function () {