Merge pull request #3385 from manmar92/master

bulkCreate validation with individualHooks returns an array of errors when validation fails. Fix #3356

lib/model.js

@@ -1351,7 +1351,11 @@ module.exports = (function() {
         return Promise.map(instances, function(instance) {
         return Promise.map(instances, function(instance) {
           // hookValidate rejects with errors, validate returns with errors
           // hookValidate rejects with errors, validate returns with errors
           if (options.individualHooks) {
           if (options.individualHooks) {
-            return instance.hookValidate(options);
+            return instance.hookValidate(options).catch(function (err) {
+              if (err) {
+                errors.push({record: instance, errors: err});
+              }
+            });
           } else {
           } else {
             return instance.validate(options).then(function (err) {
             return instance.validate(options).then(function (err) {
               if (err) {
               if (err) {

test/integration/model.test.js

@@ -2561,3 +2561,29 @@ describe(Support.getTestDialectTeaser('Model'), function() {
     return this.sequelize.sync({force: true});
     return this.sequelize.sync({force: true});
   });
   });
 });
 });
+
+  it('should return array of errors if validate and individualHooks are true in bulkCreate', function() {
+    var self = this
+      , data = [{ username: null },
+                { username: null },
+                { username: null }];
+
+    var user = this.sequelize.define('Users', {
+      username: {
+        type: Sequelize.STRING,
+        allowNull: false,
+        validate: {
+          notNull: true,
+          notEmpty: true
+        }
+      }
+    });
+
+    user.bulkCreate(data, {
+      validate: true,
+      individualHooks: true
+    })
+    .catch(function(errors) {
+      expect(errors).to.be.instanceof(Array);
+    });
+  });
\ No newline at end of file