fix: move sql to error stack (ali-sdk#8)

node-modules · Jun 2, 2016 · 54349cd · 54349cd
1 parent 1e4c769
commit 54349cd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/operator.js b/lib/operator.js
@@ -49,7 +49,7 @@ proto.query = function* (sql, values) {
   try {
     return yield this._query(sql);
   } catch (err) {
-    err.message += ' (sql: ' + sql + ')';
+    err.stack = err.stack + '\n    sql: ' + sql;
     debug('query error: %s', err);
     throw err;
   }

diff --git a/test/client.test.js b/test/client.test.js
@@ -212,6 +212,7 @@ describe('test/client.test.js', function () {
         // error, rollback
         yield conn.rollback(); // rollback call won't throw err
         assert.equal(err.code, 'ER_PARSE_ERROR');
+        assert(/sql: insert into/.test(err.stack), err.stack);
       } finally {
         // should release connection whatever
         conn.release();