fix: failed requests did not update test runner indicator (#348)

AnWeber · Jan 2, 2025 · 2ee5d46 · 2ee5d46
1 parent 1a6cf6c
commit 2ee5d46
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+## [unreleased]
+
+### Fix
+- errored or skipped referenced httpregion also skips current http region (AnWeber/httpyac#842, #339)
+- failed requests did not update test runner indicator (#348)
+
 ## [6.16.4] (2024-11-03)
 
 ### Features

diff --git a/src/plugin/vscodeHttpyacPlugin.ts b/src/plugin/vscodeHttpyacPlugin.ts
@@ -10,7 +10,12 @@ export function registerVscodePluginsFactory(httpRegionExecutedEmitter: vscode.E
     api.hooks.execute.addInterceptor(bailOnFailedTestInterceptor);
     api.hooks.execute.addInterceptor({
       id: 'httpRegionExecuted',
-      afterTrigger: async function bail(hookContext: { args: [ProcessorContext] }) {
+      afterLoop: async function bail(hookContext: { args: [ProcessorContext] }) {
+        const context = hookContext.args[0];
+        httpRegionExecutedEmitter.fire(context);
+        return true;
+      },
+      onError: async function bail(_err: Error, hookContext: { args: [ProcessorContext] }) {
         const context = hookContext.args[0];
         httpRegionExecutedEmitter.fire(context);
         return true;