Merge pull request #349 from sushantdhiman/geom-parse-issue

Failing test for geometry typeCast issue

Author: sidorares

lib/compile_text_parser.js

@@ -21,8 +21,8 @@ function compile (fields, options, config) {
       table: field.table,
       name: field.name,
       string: function() { return packet.readLengthCodedString(); },
-      buffer: function () { return this.parser.parseLengthCodedBuffer(); },
-      geometry: function () { return this.parser.parseGeometryValue(); }
+      buffer: function () { return packet.readLengthCodedBuffer(); },
+      geometry: function () { return packet.parseGeometryValue(); }
     };
   };
 

test/integration/connection/test-typecast-geometry.js

@@ -0,0 +1,31 @@
+var common = require('../../common');
+var connection = common.createConnection();
+var assert = require('assert');
+
+connection.query({
+  sql: 'select GeomFromText(\'POINT(11 0)\') as foo',
+  typeCast: function (field, next) {
+    if (field.type === 'GEOMETRY') {
+      return field.geometry();
+    }
+    return next();
+  }
+}, function(err, res) {
+  assert.ifError(err);
+  assert.deepEqual(res[0].foo, { x: 11, y: 0 });
+});
+
+connection.query({
+  sql: 'select GeomFromText(\'POINT(11 0)\') as foo',
+  typeCast: function (field, next) {
+    if (field.type === 'GEOMETRY') {
+      return field.buffer();
+    }
+    return next();
+  }
+}, function(err, res) {
+  assert.ifError(err);
+  assert.equal(Buffer.isBuffer(res[0].foo), true);
+});
+
+connection.end();