From 2bef7fdbfbf30c539c2c01865cdc7899b65a1373 Mon Sep 17 00:00:00 2001 From: Robin Braumann Date: Thu, 7 May 2026 18:36:22 +0200 Subject: [PATCH] Fix pipeline --- .../request-helper-functions.test.ts | 22 +++++++++++-------- 1 file changed, 13 insertions(+), 9 deletions(-) diff --git a/packages/core/src/execution-engine/node-execution-context/utils/__tests__/request-helper-functions.test.ts b/packages/core/src/execution-engine/node-execution-context/utils/__tests__/request-helper-functions.test.ts index ff706c29e69..0cfdd1f9440 100644 --- a/packages/core/src/execution-engine/node-execution-context/utils/__tests__/request-helper-functions.test.ts +++ b/packages/core/src/execution-engine/node-execution-context/utils/__tests__/request-helper-functions.test.ts @@ -33,6 +33,10 @@ import { } from '../request-helper-functions'; const TEST_CA_CERT = '-----BEGIN CERTIFICATE-----\nTEST\n-----END CERTIFICATE-----'; +type RedirectResponseDetails = Parameters< + NonNullable>['beforeRedirect']> +>[1]; +const redirectResponseDetails: RedirectResponseDetails = { headers: {}, statusCode: 302 }; describe('Request Helper Functions', () => { describe('proxyRequestToAxios', () => { @@ -501,7 +505,7 @@ describe('Request Helper Functions', () => { hostname: 'example.de', href: requestObject.uri, }; - axiosOptions.beforeRedirect!(redirectOptions, mock(), mock()); + axiosOptions.beforeRedirect!(redirectOptions, redirectResponseDetails); expect(redirectOptions.agent).toEqual(redirectOptions.agents.https); expect((redirectOptions.agent as HttpsAgent).options).toMatchObject({ servername: 'example.de', @@ -2240,9 +2244,9 @@ describe('Request Helper Functions', () => { }; expect(axiosOptions.beforeRedirect).toBeDefined(); - expect(() => axiosOptions.beforeRedirect!(redirectOptions, mock(), mock())).toThrow( - 'Domain not allowed', - ); + expect(() => + axiosOptions.beforeRedirect!(redirectOptions, redirectResponseDetails), + ).toThrow('Domain not allowed'); }); test.each([['example.com'], [undefined]])( @@ -2260,7 +2264,7 @@ describe('Request Helper Functions', () => { expect(axiosOptions.beforeRedirect).toBeDefined(); expect(() => - axiosOptions.beforeRedirect!(redirectOptions, mock(), mock()), + axiosOptions.beforeRedirect!(redirectOptions, redirectResponseDetails), ).not.toThrow(); }, ); @@ -2280,9 +2284,9 @@ describe('Request Helper Functions', () => { }; expect(axiosConfig.beforeRedirect).toBeDefined(); - expect(() => axiosConfig.beforeRedirect!(redirectOptions, mock(), mock())).toThrow( - 'Domain not allowed', - ); + expect(() => + axiosConfig.beforeRedirect!(redirectOptions, redirectResponseDetails), + ).toThrow('Domain not allowed'); }); test.each([['example.com'], [undefined]])( @@ -2301,7 +2305,7 @@ describe('Request Helper Functions', () => { expect(axiosConfig.beforeRedirect).toBeDefined(); expect(() => - axiosConfig.beforeRedirect!(redirectOptions, mock(), mock()), + axiosConfig.beforeRedirect!(redirectOptions, redirectResponseDetails), ).not.toThrow(); }, );