refactor: align before/afterEach function design

pact-foundation · Oct 5, 2024 · d368c80 · d368c80
1 parent 8c7cef6
commit d368c80
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/src/dsl/verifier/proxy/hooks.ts b/src/dsl/verifier/proxy/hooks.ts
@@ -8,21 +8,21 @@ export const registerBeforeHook = (
   config: ProxyOptions,
   stateSetupPath: string
 ): void => {
+  if (config.beforeEach) logger.trace("registered 'beforeEach' hook");
   app.use(async (req, res, next) => {
-    if (config.beforeEach !== undefined) {
-      logger.trace("registered 'beforeEach' hook");
-      if (req.path === stateSetupPath) {
-        logger.debug("executing 'beforeEach' hook");
-        try {
-          await config.beforeEach();
-        } catch (e) {
-          logger.error(`error executing 'beforeEach' hook: ${e.message}`);
-          logger.debug(`Stack trace was: ${e.stack}`);
-          next(new Error(`error executing 'beforeEach' hook: ${e.message}`));
-        }
+    if (req.path === stateSetupPath && config.beforeEach) {
+      logger.debug("executing 'beforeEach' hook");
+      try {
+        await config.beforeEach();
+        next();
+      } catch (e) {
+        logger.error(`error executing 'beforeEach' hook: ${e.message}`);
+        logger.debug(`Stack trace was: ${e.stack}`);
+        next(new Error(`error executing 'beforeEach' hook: ${e.message}`));
       }
+    } else {
+      next();
     }
-    next();
   });
 };
 
@@ -31,7 +31,7 @@ export const registerAfterHook = (
   config: ProxyOptions,
   stateSetupPath: string
 ): void => {
-  logger.trace("registered 'afterEach' hook");
+  if (config.afterEach) logger.trace("registered 'afterEach' hook");
   app.use(async (req, res, next) => {
     if (req.path !== stateSetupPath && config.afterEach) {
       logger.debug("executing 'afterEach' hook");