Merge pull request #375 from adamsch1/master

Clearer error if you have a typo on the column data type

lib/dao-factory.js

@@ -49,6 +49,8 @@ module.exports = (function() {
 
 
     this.primaryKeys = {};
     this.primaryKeys = {};
     Utils._.each(this.attributes, function(dataTypeString, attributeName) {
     Utils._.each(this.attributes, function(dataTypeString, attributeName) {
+      // If you don't specify a valid data type lets help you debug it
+      if( dataTypeString === undefined ) throw new Error("Unrecognized data type for field " + attributeName );  
       if((attributeName != 'id') && (dataTypeString.indexOf('PRIMARY KEY') !== -1)) {
       if((attributeName != 'id') && (dataTypeString.indexOf('PRIMARY KEY') !== -1)) {
         self.primaryKeys[attributeName] = dataTypeString
         self.primaryKeys[attributeName] = dataTypeString
       }
       }

spec/dao-factory.spec.js

@@ -205,6 +205,20 @@ describe(Helpers.getTestDialectTeaser("DAOFactory"), function() {
       })
       })
     })
     })
 
 
+    it('raises an error if you mess up the datatype', function(done)  {
+
+      try {
+        var User = this.sequelize.define('UserBadDataType', {
+          activity_date: Sequelize.DATe
+        });
+        done()
+      }
+      catch( e ) { 
+        expect(e.message).toEqual('Unrecognized data type for field activity_date')
+        done()
+      }
+    })
+
     it('sets a 64 bit int in bigint', function(done) {
     it('sets a 64 bit int in bigint', function(done) {
       var User = this.sequelize.define('UserWithBigIntFields', {
       var User = this.sequelize.define('UserWithBigIntFields', {
         big: Sequelize.BIGINT
         big: Sequelize.BIGINT