fix: support of --bail does also work in referenced http files (#540)

CHANGELOG.md

@@ -1,3 +1,9 @@
+## [6.6.5]  (2023-09-06)
+
+### Fixes
+
+- support of `--bail` does also work in referenced http files (#540)
+
 ## [6.6.4]  (2023-09-02)
 
 ### Fixes

package.json

@@ -4,7 +4,7 @@
   "license": "MIT",
   "publisher": "AnWeber",
   "description": "HTTP/REST CLI Client for *.http files",
-  "version": "6.6.4",
+  "version": "6.6.5",
   "homepage": "https://github.com/AnWeber/httpyac",
   "repository": {
     "type": "git",

src/cli/send/bailOnFailedTestInterceptor.ts → src/cli/send/plugin/bailOnFailedTestInterceptor.ts

@@ -1,4 +1,4 @@
-import * as models from '../../models';
+import * as models from '../../../models';
 import { HookTriggerContext } from 'hookpoint';
 
 export const bailOnFailedTestInterceptor = {

src/cli/send/plugin/cliHttpyacPlugin.ts

@@ -0,0 +1,14 @@
+import { HttpyacHooksApi } from '../../../models';
+import { bailOnFailedTestInterceptor } from './bailOnFailedTestInterceptor';
+import { loggerFlushInterceptor } from './loggerFlushInterceptor';
+import { testExitCodeInterceptor } from './testExitCodeInterceptor';
+
+export function createCliPluginRegister(bail: boolean) {
+  return function registerCliPlugin(api: HttpyacHooksApi) {
+    api.hooks.execute.addInterceptor(loggerFlushInterceptor);
+    api.hooks.execute.addInterceptor(testExitCodeInterceptor);
+    if (bail) {
+      api.hooks.execute.addInterceptor(bailOnFailedTestInterceptor);
+    }
+  };
+}

src/cli/send/plugin/index.ts

@@ -0,0 +1,4 @@
+export * from './bailOnFailedTestInterceptor';
+export * from './cliHttpyacPlugin';
+export * from './loggerFlushInterceptor';
+export * from './testExitCodeInterceptor';

src/cli/send/loggerFlushInterceptor.ts → src/cli/send/plugin/loggerFlushInterceptor.ts

@@ -1,4 +1,4 @@
-import * as models from '../../models';
+import * as models from '../../../models';
 import { HookTriggerContext } from 'hookpoint';
 
 export const loggerFlushInterceptor = {

src/cli/send/testExitCodeInterceptor.ts → src/cli/send/plugin/testExitCodeInterceptor.ts

@@ -1,8 +1,8 @@
-import * as models from '../../models';
+import * as models from '../../../models';
 import { HookTriggerContext } from 'hookpoint';
 
 export const testExitCodeInterceptor = {
-  id: 'bailOnFailed',
+  id: 'testExitCode',
   afterTrigger: async function bail(hookContext: HookTriggerContext<[models.ProcessorContext], boolean>) {
     const context = hookContext.args[0];
     const failedTest = context.httpRegion.testResults?.find?.(obj => !obj.result);

src/cli/send/send.ts

@@ -3,11 +3,9 @@ import { fileProvider, Logger } from '../../io';
 import * as models from '../../models';
 import { HttpFileStore } from '../../store';
 import * as utils from '../../utils';
-import { bailOnFailedTestInterceptor } from './bailOnFailedTestInterceptor';
+import { createCliPluginRegister } from './plugin';
 import { toSendJsonOutput } from './jsonOutput';
-import { loggerFlushInterceptor } from './loggerFlushInterceptor';
 import { SendOptions, getLogLevel, SendFilterOptions, OutputType } from './options';
-import { testExitCodeInterceptor } from './testExitCodeInterceptor';
 import { default as chalk } from 'chalk';
 import { Command } from 'commander';
 import { promises as fs } from 'fs';
@@ -54,7 +52,6 @@ async function execute(fileNames: Array<string>, options: SendOptions): Promise<
   initRequestLogger(options, context);
   try {
     if (httpFiles.length > 0) {
-      initCliHooks(httpFiles, options);
       let isFirstRequest = true;
       const jsonOutput: Record<string, Array<models.ProcessedHttpRegion>> = {};
       while (options.interactive || isFirstRequest) {
@@ -160,19 +157,11 @@ export function initRequestLogger(cliOptions: SendOptions, context: Omit<models.
   }
 }
 
-function initCliHooks(httpFiles: Array<models.HttpFile>, cliOptions: SendOptions) {
-  for (const httpFile of utils.distinct(httpFiles)) {
-    httpFile.hooks.execute.addInterceptor(loggerFlushInterceptor);
-    httpFile.hooks.execute.addInterceptor(testExitCodeInterceptor);
-    if (cliOptions.bail) {
-      httpFile.hooks.execute.addInterceptor(bailOnFailedTestInterceptor);
-    }
-  }
-}
-
 async function getHttpFiles(fileNames: Array<string>, options: SendOptions, config: models.EnvironmentConfig) {
   const httpFiles: models.HttpFile[] = [];
-  const httpFileStore = new HttpFileStore();
+  const httpFileStore = new HttpFileStore({
+    cli: createCliPluginRegister(!!options.bail),
+  });
 
   const parseOptions: models.HttpFileStoreOptions = {
     workingDir: process.cwd(),