Merge pull request #1298 from dhensby/pulls/bulk-error

Catch errors from table._makeBulk()

Author: dhensby

lib/msnodesqlv8/request.js

@@ -146,7 +146,11 @@ class Request extends BaseRequest {
     super._bulk(table, options, err => {
       if (err) return callback(err)
 
-      table._makeBulk()
+      try {
+        table._makeBulk()
+      } catch (e) {
+        return callback(new RequestError(e, 'EREQUEST'))
+      }
 
       if (!table.name) {
         setImmediate(callback, new RequestError('Table name must be specified for bulk insert.', 'ENAME'))

lib/tedious/request.js

@@ -213,7 +213,11 @@ class Request extends BaseRequest {
     super._bulk(table, options, err => {
       if (err) return callback(err)
 
-      table._makeBulk()
+      try {
+        table._makeBulk()
+      } catch (e) {
+        return callback(new RequestError(e, 'EREQUEST'))
+      }
 
       if (!table.name) {
         return callback(new RequestError('Table name must be specified for bulk insert.', 'ENAME'))