Merge pull request #306 from strongloop/decimal-number

Add DECIMAL to Number prop type

lib/discovery.js

@@ -349,6 +349,9 @@ function mixinDiscovery(MySQL, mysql) {
       case 'FLOAT':
       case 'DOUBLE':
       case 'BIGINT':
+      case 'INTEGER':
+      case 'DECIMAL':
+      case 'NUMERIC':
         return 'Number';
       case 'DATE':
       case 'TIMESTAMP':

test/datatypes.test.js

@@ -57,6 +57,22 @@ describe('MySQL specific datatypes', function() {
       Account.destroyAll(done);
     });
 
+    it('discover type of amount', function(done) {
+      db.discoverModelProperties('Account', {})
+        .then(function(defs) {
+          defs.forEach(function(props) {
+            if (props.columnName === 'amount') {
+              assert.deepEqual(props.dataType, 'decimal');
+              assert.deepEqual(props.type, 'Number');
+              done();
+            }
+          });
+        })
+        .catch(function(err) {
+          done(err);
+        });
+    });
+
     it('create an instance', function(done) {
       Account.create(data, function(err, result) {
         if (err) return done(err);