more tests for hooks added to make coverage reporter happy

Author: RobinBuschmann

test/models/Hook.ts

@@ -8,6 +8,14 @@ import { AfterSave, AfterUpdate, BeforeSave, BeforeUpdate } from "../../index";
 import { BeforeCreate, BeforeValidate, Column, Model, Table } from "../../index";
 import { BeforeFind, BeforeFindAfterExpandIncludeAll } from "../../index";
 import { BeforeBulkDelete, AfterBulkDelete, AfterDelete, BeforeDelete } from "../../index";
+import {AfterBulkSync} from '../../lib/annotations/hooks/AfterBulkSync';
+import {AfterConnect} from '../../lib/annotations/hooks/AfterConnect';
+import {AfterDefine} from '../../lib/annotations/hooks/AfterDefine';
+import {AfterInit} from '../../lib/annotations/hooks/AfterInit';
+import {BeforeBulkSync} from '../../lib/annotations/hooks/BeforeBulkSync';
+import {BeforeConnect} from '../../lib/annotations/hooks/BeforeConnect';
+import {BeforeDefine} from '../../lib/annotations/hooks/BeforeDefine';
+import {BeforeInit} from '../../lib/annotations/hooks/BeforeInit';
 
 /**
  * Model used to test hook decorators. Defined hooks are mocked out for testing.
@@ -79,6 +87,30 @@ export class Hook extends Model<Hook> {
   @AfterBulkCreate
   static afterBulkCreateHook(instances: Hook[], options: any): void {}
 
+  @BeforeBulkSync
+  static beforeBulkSyncHook(instances: Hook[], options: any): void {}
+
+  @AfterBulkSync
+  static afterBulkSyncHook(instances: Hook[], options: any): void {}
+
+  @BeforeConnect
+  static beforeConnectHook(instances: Hook[], options: any): void {}
+
+  @AfterConnect
+  static afterConnectHook(instances: Hook[], options: any): void {}
+
+  @BeforeDefine
+  static beforeDefineHook(instances: Hook[], options: any): void {}
+
+  @AfterDefine
+  static afterDefineHook(instances: Hook[], options: any): void {}
+
+  @BeforeInit
+  static beforeInitHook(instances: Hook[], options: any): void {}
+
+  @AfterInit
+  static afterInitHook(instances: Hook[], options: any): void {}
+
   @BeforeBulkDestroy
   static beforeBulkDestroyHook(options: any): void {}
 

test/specs/hooks/hooks.spec.ts

@@ -81,6 +81,15 @@ describe('hook', () => {
     const afterFindHookStub = sinon.stub(Hook, 'afterFindHook');
     const beforeCountHookStub = sinon.stub(Hook, 'beforeCountHook');
 
+    const beforeBulkSyncHookStub = sinon.stub(Hook, 'beforeBulkSyncHook');
+    const afterBulkSyncHookStub = sinon.stub(Hook, 'afterBulkSyncHook');
+    const beforeConnectHookStub = sinon.stub(Hook, 'beforeConnectHook');
+    const afterConnectHookStub = sinon.stub(Hook, 'afterConnectHook');
+    const beforeDefineHookStub = sinon.stub(Hook, 'beforeDefineHook');
+    const afterDefineHookStub = sinon.stub(Hook, 'afterDefineHook');
+    const beforeInitHookStub = sinon.stub(Hook, 'beforeInitHook');
+    const afterInitHookStub = sinon.stub(Hook, 'afterInitHook');
+
     // these hooks are aliases for the equivalent “destroy” hooks
     const beforeDeleteHookStub = sinon.stub(Hook, 'beforeDeleteHook');
     const afterDeleteHookStub = sinon.stub(Hook, 'afterDeleteHook');
@@ -176,6 +185,15 @@ describe('hook', () => {
     expect(Hook['options'].hooks['afterFind']).to.include(afterFindHookStub);
     expect(Hook['options'].hooks['beforeCount']).to.include(beforeCountHookStub);
 
+    expect(Hook['options'].hooks['beforeBulkSync']).to.include(beforeBulkSyncHookStub);
+    expect(Hook['options'].hooks['afterBulkSync']).to.include(afterBulkSyncHookStub);
+    expect(Hook['options'].hooks['beforeConnect']).to.include(beforeConnectHookStub);
+    expect(Hook['options'].hooks['afterConnect']).to.include(afterConnectHookStub);
+    expect(Hook['options'].hooks['beforeDefine']).to.include(beforeDefineHookStub);
+    expect(Hook['options'].hooks['afterDefine']).to.include(afterDefineHookStub);
+    expect(Hook['options'].hooks['beforeInit']).to.include(beforeInitHookStub);
+    expect(Hook['options'].hooks['afterInit']).to.include(afterInitHookStub);
+
     expect(Hook['options'].hooks['beforeDestroy']).to.include(beforeDeleteHookStub);
     expect(Hook['options'].hooks['afterDestroy']).to.include(afterDeleteHookStub);
     expect(Hook['options'].hooks['beforeBulkDestroy']).to.include(beforeBulkDeleteHookStub);