From 2ee5d4619eb28e903060bbbd2549c34127562799 Mon Sep 17 00:00:00 2001 From: Andreas Weber Date: Thu, 2 Jan 2025 22:39:35 +0100 Subject: [PATCH] fix: failed requests did not update test runner indicator (#348) --- CHANGELOG.md | 6 ++++++ src/plugin/vscodeHttpyacPlugin.ts | 7 ++++++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index a916a16..181273f 100644 --- 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 index 5f1f7aa..1134a0d 100644 --- 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;