fix(model): dont remove primary key if using default value in upsert, closes #3247

lib/instance.js

@@ -530,7 +530,7 @@ module.exports = (function() {
         options.fields.push(createdAtAttr);
       }
 
-      if (primaryKeyAttribute && !primaryKeyAttribute.autoIncrement && options.fields.indexOf(primaryKeyName) < 0) {
+      if (primaryKeyAttribute && primaryKeyAttribute.defaultValue && options.fields.indexOf(primaryKeyName) < 0) {
         options.fields.unshift(primaryKeyName);
       }
     }

lib/model.js

@@ -1224,7 +1224,7 @@ module.exports = (function() {
 
     var createdAtAttr = this._timestampAttributes.createdAt
       , updatedAtAttr = this._timestampAttributes.updatedAt
-      , hadPrimary = this.primaryKeyField in values
+      , hadPrimary = this.primaryKeyField in values || this.primaryKeyAttribute in values
       , instance = this.build(values);
 
     return instance.hookValidate(options).bind(this).then(function () {
@@ -1240,7 +1240,7 @@ module.exports = (function() {
 
       // Build adds a null value for the primary key, if none was given by the user.
       // We need to remove that because of some Postgres technicalities.
-      if (!hadPrimary) {
+      if (!hadPrimary && !this.rawAttributes[this.primaryKeyAttribute].defaultValue) {
         delete values[this.primaryKeyField];
       }
 

test/integration/model/upsert.test.js

@@ -92,6 +92,26 @@ describe(Support.getTestDialectTeaser('Model'), function() {
         });
       });
 
+      it('should work with UUIDs wth default values', function () {
+        var User = this.sequelize.define('User', {
+          id: {
+            primaryKey: true,
+            allowNull: false,
+            unique: true,
+            type: Sequelize.UUID,
+            defaultValue: Sequelize.UUIDV4
+          },
+
+          name: {
+            type: Sequelize.STRING,
+          }
+        });
+
+        return User.sync({ force: true }).then(function () {
+          return User.upsert({ name: 'John Doe' });
+        });
+      });
+
       it('works with upsert on a composite primary key', function() {
         var User = this.sequelize.define('user', {
           a: {