[taler-wallet-core] branch master updated: harness: fix exchange base URL in tests with fault injection

[gnunet]

This is an automated email from the git hooks/post-receive script.

dold pushed a commit to branch master
in repository wallet-core.

The following commit(s) were added to refs/heads/master by this push:
     new 7dcfd02da harness: fix exchange base URL in tests with fault injection
7dcfd02da is described below

commit 7dcfd02dae23fe2a6249893ae19ab59b8b5a09f5
Author: Florian Dold <florian@dold.me>
AuthorDate: Wed Jun 21 06:59:34 2023 +0200

    harness: fix exchange base URL in tests with fault injection
---
 packages/taler-harness/src/harness/helpers.ts                  | 10 ++++++++--
 .../src/integrationtests/test-exchange-management.ts           |  6 +++++-
 .../src/integrationtests/test-merchant-exchange-confusion.ts   |  5 +++++
 .../taler-harness/src/integrationtests/test-payment-fault.ts   |  4 ++++
 4 files changed, 22 insertions(+), 3 deletions(-)

diff --git a/packages/taler-harness/src/harness/helpers.ts 
b/packages/taler-harness/src/harness/helpers.ts
index bc2f573e9..fd6e9aa2e 100644
--- a/packages/taler-harness/src/harness/helpers.ts
+++ b/packages/taler-harness/src/harness/helpers.ts
@@ -406,6 +406,11 @@ export async function 
createFaultInjectedMerchantTestkudosEnvironment(
   const faultyMerchant = new FaultInjectedMerchantService(t, merchant, 9083);
   const faultyExchange = new FaultInjectedExchangeService(t, exchange, 9081);
 
+  // Base URL must contain port that the proxy is listening on.
+  await exchange.modifyConfig(async (config) => {
+    config.setString("exchange", "base_url", "http://localhost:9081/";);
+  });
+
   const exchangeBankAccount = await bank.createExchangeAccount(
     "myexchange",
     "x",
@@ -546,8 +551,9 @@ export async function withdrawViaBankV2(
     },
   );
 
-  const withdrawalFinishedCond = wallet.waitForNotificationCond((x) =>
-    x.type === NotificationType.TransactionStateTransition &&
+  const withdrawalFinishedCond = wallet.waitForNotificationCond(
+    (x) =>
+      x.type === NotificationType.TransactionStateTransition &&
       x.newTxState.major === TransactionMajorState.Done &&
       x.transactionId === acceptRes.transactionId,
   );
diff --git 
a/packages/taler-harness/src/integrationtests/test-exchange-management.ts 
b/packages/taler-harness/src/integrationtests/test-exchange-management.ts
index 6b63c3741..c67f861e1 100644
--- a/packages/taler-harness/src/integrationtests/test-exchange-management.ts
+++ b/packages/taler-harness/src/integrationtests/test-exchange-management.ts
@@ -44,7 +44,7 @@ import {
 import { defaultCoinConfig } from "../harness/denomStructures.js";
 
 /**
- * Test if the wallet handles outdated exchange versions correct.y
+ * Test if the wallet handles outdated exchange versions correctly.
  */
 export async function runExchangeManagementTest(
   t: GlobalTestState,
@@ -81,6 +81,10 @@ export async function runExchangeManagementTest(
   exchange.addBankAccount("1", exchangeBankAccount);
 
   const faultyExchange = new FaultInjectedExchangeService(t, exchange, 8091);
+  // Base URL must contain port that the proxy is listening on.
+  await exchange.modifyConfig(async (config) => {
+    config.setString("exchange", "base_url", "http://localhost:8091/";);
+  });
 
   bank.setSuggestedExchange(
     faultyExchange,
diff --git 
a/packages/taler-harness/src/integrationtests/test-merchant-exchange-confusion.ts
 
b/packages/taler-harness/src/integrationtests/test-merchant-exchange-confusion.ts
index 30ab1cd4b..2fccd0b97 100644
--- 
a/packages/taler-harness/src/integrationtests/test-merchant-exchange-confusion.ts
+++ 
b/packages/taler-harness/src/integrationtests/test-merchant-exchange-confusion.ts
@@ -79,6 +79,11 @@ export async function 
createConfusedMerchantTestkudosEnvironment(
   const faultyMerchant = new FaultInjectedMerchantService(t, merchant, 9083);
   const faultyExchange = new FaultInjectedExchangeService(t, exchange, 9081);
 
+  // Base URL must contain port that the proxy is listening on.
+  await exchange.modifyConfig(async (config) => {
+    config.setString("exchange", "base_url", "http://localhost:9081/";);
+  });
+
   const exchangeBankAccount = await bank.createExchangeAccount(
     "myexchange",
     "x",
diff --git a/packages/taler-harness/src/integrationtests/test-payment-fault.ts 
b/packages/taler-harness/src/integrationtests/test-payment-fault.ts
index dea538e35..209830187 100644
--- a/packages/taler-harness/src/integrationtests/test-payment-fault.ts
+++ b/packages/taler-harness/src/integrationtests/test-payment-fault.ts
@@ -84,6 +84,10 @@ export async function runPaymentFaultTest(t: 
GlobalTestState) {
   await exchange.pingUntilAvailable();
 
   const faultyExchange = new FaultInjectedExchangeService(t, exchange, 8091);
+  // Base URL must contain port that the proxy is listening on.
+  await exchange.modifyConfig(async (config) => {
+    config.setString("exchange", "base_url", "http://localhost:8091/";);
+  });
 
   // Print all requests to the exchange
   faultyExchange.faultProxy.addFault({

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.