Don't fail on foreign key references to inexistant DAOs, let it run and the DB s…

…hould complain about referencing inexistant tables

lib/dao-factory-manager.js

@@ -60,7 +60,7 @@ module.exports = (function() {
     })
 
     sorter.sort().reverse().forEach(function(name) {
-      iterator(daos[name])
+      iterator(daos[name], name)
     })
   }
 

lib/sequelize.js

@@ -366,8 +366,12 @@ module.exports = (function() {
     // Topologically sort by foreign key constraints to give us an appropriate
     // creation order
 
-    this.daoFactoryManager.forEachDAO(function(dao) {
-      chainer.add(dao, 'sync', [options])
+    this.daoFactoryManager.forEachDAO(function(dao, daoName) {
+      if (dao) {
+        chainer.add(dao, 'sync', [options])
+      } else {
+        // DB should throw an SQL error if referencing inexistant table
+      }
     })
 
     return chainer.runSerially()

test/sequelize.test.js

@@ -477,6 +477,17 @@ describe(Support.getTestDialectTeaser("Sequelize"), function () {
           done()
         })
       })
+
+      it('returns an error correctly if unable to sync a foreign key referenced model', function (done) {
+        var Application = this.sequelize.define('Application', {
+          authorID: { type: Sequelize.BIGINT, allowNull: false, references: 'User', referencesKey: 'id' },
+        })
+
+        this.sequelize.sync().error(function (error) {
+          assert.ok(error);
+          done()
+        })
+      })
     }
 
     describe("doesn't emit logging when explicitly saying not to", function() {