commit d783cdc82b5e1c1c90cc13ee15e04381935fcbf8 parent e299b133dbb4769355e728e4545accaac2956248 Author: Sebastian <sebasjm@gmail.com> Date: Fri, 20 Oct 2023 10:53:40 -0300 fix typo: reject for promise instead of throw Diffstat:
3 files changed, 15 insertions(+), 8 deletions(-)
diff --git a/packages/taler-wallet-webextension/src/platform/chrome.ts b/packages/taler-wallet-webextension/src/platform/chrome.ts @@ -364,8 +364,8 @@ async function sendMessageToBackground< let timedout = false; const timerId = setTimeout(() => { timedout = true; - throw TalerError.fromDetail(TalerErrorCode.GENERIC_TIMEOUT, {}); - }, 5 * 1000); //five seconds + reject(TalerError.fromDetail(TalerErrorCode.GENERIC_TIMEOUT, {}) ); + }, 20 * 1000); //five seconds chrome.runtime.sendMessage(messageWithId, (backgroundResponse) => { if (timedout) { return false; //already rejected diff --git a/packages/taler-wallet-webextension/src/taler-wallet-interaction-loader.ts b/packages/taler-wallet-webextension/src/taler-wallet-interaction-loader.ts @@ -14,7 +14,7 @@ GNU Taler; see the file COPYING. If not, see <http://www.gnu.org/licenses/> */ -import { CoreApiResponse } from "@gnu-taler/taler-util"; +import { CoreApiResponse, TalerError, TalerErrorCode } from "@gnu-taler/taler-util"; /** * This will modify all the pages that the user load when navigating with Web Extension enabled @@ -183,9 +183,8 @@ async function sendMessageToBackground<Op extends keyof ExtensionOperations>( let timedout = false; const timerId = setTimeout(() => { timedout = true; - throw new Error("timeout"); - // throw TalerError.fromDetail(TalerErrorCode.GENERIC_TIMEOUT, {}); - }, 5 * 1000); //five seconds + reject(TalerError.fromDetail(TalerErrorCode.GENERIC_TIMEOUT, {})) + }, 20 * 1000); //five seconds chrome.runtime.sendMessage(messageWithId, (backgroundResponse) => { if (timedout) { return false; //already rejected diff --git a/packages/taler-wallet-webextension/src/test-utils.ts b/packages/taler-wallet-webextension/src/test-utils.ts @@ -14,14 +14,14 @@ GNU Taler; see the file COPYING. If not, see <http://www.gnu.org/licenses/> */ -import { NotificationType } from "@gnu-taler/taler-util"; +import { NotificationType, TalerBankIntegrationHttpClient, TalerCoreBankHttpClient, TalerRevenueHttpClient, TalerWireGatewayHttpClient } from "@gnu-taler/taler-util"; import { WalletCoreApiClient, WalletCoreOpKeys, WalletCoreRequestType, WalletCoreResponseType, } from "@gnu-taler/taler-wallet-core"; -import { TranslationProvider } from "@gnu-taler/web-util/browser"; +import { ApiContextProvider, TranslationProvider, defaultRequestHandler } from "@gnu-taler/web-util/browser"; import { ComponentChildren, FunctionalComponent, @@ -182,6 +182,14 @@ export function createWalletApiMock(): { }): VNode { let children = _cs; children = create(AlertProvider, { children }, children); + const value = { + request: defaultRequestHandler, + bankCore: new TalerCoreBankHttpClient("/"), + bankIntegration: new TalerBankIntegrationHttpClient("/"), + bankWire: new TalerWireGatewayHttpClient("/",""), + bankRevenue: new TalerRevenueHttpClient("/",""), + } + children = create(ApiContextProvider, { value, children }, children); children = create( TranslationProvider, { children, source: strings, initial: "en", forceLang: "en" },