Model.prototype.constructor and Sequelize.prototype.constructor referenced base models instead of Model/Sequelize

Author: Robin Buschmann

lib/models/BaseModel.ts

@@ -34,7 +34,7 @@ export abstract class BaseModel {
 
     // copies all prototype members of this to target.prototype
     Object
-      .getOwnPropertyNames(this.prototype)
+      .keys(this.prototype)
       .forEach(name => target.prototype[name] = this.prototype[name])
     ;
 

lib/models/BaseSequelize.ts

@@ -24,7 +24,7 @@ export abstract class BaseSequelize {
 
     // copies all prototype members of this to target.prototype
     Object
-      .getOwnPropertyNames(this.prototype)
+      .keys(this.prototype)
       .forEach(name => target.prototype[name] = this.prototype[name])
     ;
 

test/specs/models/model.spec.ts

@@ -0,0 +1,14 @@
+import {expect} from 'chai';
+import {Model} from "../../../lib/models/Model";
+
+describe('model', () => {
+
+  describe('constructor', () => {
+
+    it('should equal Model class', () => {
+      expect(Model.prototype.constructor).to.equal(Model);
+    });
+
+  });
+
+});

test/specs/sequelize.spec.ts renamed to test/specs/models/sequelize.spec.ts

@@ -1,29 +1,38 @@
 import {expect} from 'chai';
-import {createSequelize} from "../utils/sequelize";
-import {Game} from "../models/exports/Game";
-import Gamer from "../models/exports/gamer.model";
+import {createSequelize} from "../../utils/sequelize";
+import {Game} from "../../models/exports/Game";
+import Gamer from "../../models/exports/gamer.model";
+import {Sequelize} from "../../../lib/models/Sequelize";
 
 describe('sequelize', () => {
 
   const sequelize = createSequelize(false);
 
+  describe('constructor', () => {
+
+    it('should equal Sequelize class', () => {
+      expect(sequelize.constructor).to.equal(Sequelize);
+    });
+
+  });
+
   describe('addModels', () => {
 
     it('should not throw', () => {
 
-      expect(() => sequelize.addModels([__dirname + '/../models/exports/'])).not.to.throw();
+      expect(() => sequelize.addModels([__dirname + '/../../models/exports/'])).not.to.throw();
     });
 
     it('should throw', () => {
 
-      expect(() => sequelize.addModels([__dirname + '/../models/exports/throws'])).to.throw();
+      expect(() => sequelize.addModels([__dirname + '/../../models/exports/throws'])).to.throw();
     });
 
     describe('default exported models', () => {
 
       it('should work as expected', () => {
 
-        sequelize.addModels([__dirname + '/../models/exports/']);
+        sequelize.addModels([__dirname + '/../../models/exports/']);
 
         expect(() => Gamer.build({})).not.to.throw;
 
@@ -38,7 +47,7 @@ describe('sequelize', () => {
 
       it('should work as expected', () => {
 
-        sequelize.addModels([__dirname + '/../models/exports/']);
+        sequelize.addModels([__dirname + '/../../models/exports/']);
 
         expect(() => Game.build({})).not.to.throw;