fix: fix applyAspect correctly being apply to whole class methods inc…

…luding constructor

src/apply-aspect.ts

@@ -1,14 +1,12 @@
-export function applyAspect (advicePool: any) {
+export function applyAspect ({ constructor = [], ...methodAdvices }: any) {
   return function (target) {
-    for (let key in advicePool) {
-      if (key === "constructor") {
-        advicePool[key].forEach(advice => target = advice(target))
-      } else {
-        advicePool[key].forEach(advice => {
-          Object.defineProperty(target.prototype, key, advice(target, key, Object.getOwnPropertyDescriptor(target.prototype, key)))
-        })
-      }
+    for (let key in methodAdvices) {
+      methodAdvices[key].forEach(advice =>
+        Object.defineProperty(target.prototype, key,
+          advice(target, key, Object.getOwnPropertyDescriptor(target.prototype, key))))
     }
+
+    constructor.forEach(advice => target = advice(target))
     return target
   }
 }

src/decorators.ts

@@ -27,7 +27,7 @@ function applyReflect (target, advices, methodName, keyJoinPoint, original) {
     const keyBeforeInstance = generateKey(KEY_BEFORE_INSTANCE, methodName)
     const keyAfterInstance = generateKey(KEY_AFTER_INSTANCE, methodName)
     // this is only needed to avoid babel's TypeError https://github.com/babel/babel/issues/682
-    // and should be removed in the future when babel isn't no longer needed to transpile ES6 classes
+    // and should be removed when ES6 classes will be fully supported
     const result = wrapMethod(target, methodName, keyBeforeInstance, keyAfterInstance)
     result.prototype = target.prototype
     return result

test/aspect.spec.ts

@@ -1,4 +1,4 @@
-import { applyAspect, beforeMethod, afterMethod } from "../src"
+import { applyAspect, afterInstance, beforeInstance, beforeMethod, afterMethod } from "../src"
 
 const methodSpy = jest.fn()