Test for issue #4 (deliberately breaks build)

test/common.js

@@ -66,6 +66,7 @@ var clear = () => Bluebird.all(lo.values(models).map(m => m.destroy({ truncate:
 
 /* --------- Export --------- */
 module.exports = {
+  Sequelize,
   sequelize,
   models: sequelize.models,
   sync,

test/issues/4-spec.js

@@ -0,0 +1,43 @@
+
+var Bluebird = require('bluebird');
+
+var { Sequelize, sequelize, sync } = require('../common');
+
+var embed = require('../../src/index')(sequelize);
+var helpers = embed.util.helpers;
+var mkIncludes = helpers.includes, mkInclude = helpers.include;
+
+
+/* --- setup --- */
+
+var ModelA = sequelize.define('ModelA', {});
+
+var ModelB = sequelize.define('ModelB', {
+  valA: Sequelize.STRING,
+  valB: Sequelize.STRING
+});
+
+ModelA.ModelB = ModelA.hasOne(ModelB, { as: 'b', foreignKey: 'aId' });
+
+var include = mkIncludes(mkInclude(ModelA.ModelB));
+var opts = { reload: { include } };
+
+
+/* --- tests --- */
+
+describe('issue-4', () => {
+
+  beforeEach(sync);
+
+  it('should match pk despite different type', done => {
+    ModelA.create({}).then(a =>
+      ModelB.create({ id: 3, aId: a.id, valA: 'ayy', valB: 'bae' }).then(b =>
+        embed.update(ModelA, { id: a.id, b: { id: '3', valA: 'ayyy' } }, include, opts)
+          .then(result => {
+            expect(result.b.id).toBe(b.id);
+            expect(result.b.valA).toBe('ayyy');
+            expect(result.b.valB).toBe('bae');
+            done();
+          })));
+  });
+});