[taler-wallet-core] branch master updated: specify bank access API separately in tests

[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 499e003f specify bank access API separately in tests
499e003f is described below

commit 499e003ff8bfcb75a20619c65c4a03a73cb7a850
Author: Florian Dold <florian@dold.me>
AuthorDate: Thu Aug 25 18:34:25 2022 +0200

    specify bank access API separately in tests
---
 packages/taler-util/src/walletTypes.ts                            | 7 +++++++
 packages/taler-wallet-cli/src/integrationtests/test-bank-api.ts   | 2 +-
 .../src/integrationtests/test-forced-selection.ts                 | 1 +
 .../taler-wallet-cli/src/integrationtests/test-wallet-dbless.ts   | 1 +
 .../taler-wallet-cli/src/integrationtests/test-wallettesting.ts   | 3 +++
 packages/taler-wallet-core/src/bank-api-client.ts                 | 8 +++-----
 packages/taler-wallet-core/src/dbless.ts                          | 3 ++-
 packages/taler-wallet-core/src/operations/testing.ts              | 2 ++
 packages/taler-wallet-core/src/operations/withdraw.ts             | 2 ++
 packages/taler-wallet-core/src/wallet.ts                          | 1 +
 10 files changed, 23 insertions(+), 7 deletions(-)

diff --git a/packages/taler-util/src/walletTypes.ts 
b/packages/taler-util/src/walletTypes.ts
index 6d3837a6..b34eb0d0 100644
--- a/packages/taler-util/src/walletTypes.ts
+++ b/packages/taler-util/src/walletTypes.ts
@@ -837,6 +837,7 @@ export const codecForTestPayArgs = (): Codec<TestPayArgs> =>
 export interface IntegrationTestArgs {
   exchangeBaseUrl: string;
   bankBaseUrl: string;
+  bankAccessApiBaseUrl: string;
   merchantBaseUrl: string;
   merchantAuthToken?: string;
   amountToWithdraw: string;
@@ -851,6 +852,7 @@ export const codecForIntegrationTestArgs = (): 
Codec<IntegrationTestArgs> =>
     .property("merchantAuthToken", codecOptional(codecForString()))
     .property("amountToSpend", codecForAmountString())
     .property("amountToWithdraw", codecForAmountString())
+    .property("bankAccessApiBaseUrl", codecForAmountString())
     .build("IntegrationTestArgs");
 
 export interface AddExchangeRequest {
@@ -1065,6 +1067,10 @@ export interface CoreApiResponseError {
 export interface WithdrawTestBalanceRequest {
   amount: string;
   bankBaseUrl: string;
+  /**
+   * Bank access API base URL.  Defaults to the bankBaseUrl.
+   */
+  bankAccessApiBaseUrl: string;
   exchangeBaseUrl: string;
   forcedDenomSel?: ForcedDenomSel;
 }
@@ -1141,6 +1147,7 @@ export const codecForWithdrawTestBalance =
       .property("bankBaseUrl", codecForString())
       .property("exchangeBaseUrl", codecForString())
       .property("forcedDenomSel", codecForAny())
+      .property("bankAccessApiBaseUrl", codecForString())
       .build("WithdrawTestBalanceRequest");
 
 export interface ApplyRefundResponse {
diff --git a/packages/taler-wallet-cli/src/integrationtests/test-bank-api.ts 
b/packages/taler-wallet-cli/src/integrationtests/test-bank-api.ts
index 97dbf369..31113aa1 100644
--- a/packages/taler-wallet-cli/src/integrationtests/test-bank-api.ts
+++ b/packages/taler-wallet-cli/src/integrationtests/test-bank-api.ts
@@ -104,7 +104,7 @@ export async function runBankApiTest(t: GlobalTestState) {
   // Make sure that registering twice results in a 409 Conflict
   {
     const e = await t.assertThrowsTalerErrorAsync(async () => {
-      await BankApi.registerAccount(bank, "user1", "pw1");
+      await BankApi.registerAccount(bank, "user1", "pw2");
     });
     t.assertTrue(e.errorDetail.httpStatusCode === 409);
   }
diff --git 
a/packages/taler-wallet-cli/src/integrationtests/test-forced-selection.ts 
b/packages/taler-wallet-cli/src/integrationtests/test-forced-selection.ts
index 0fe5f639..302e3364 100644
--- a/packages/taler-wallet-cli/src/integrationtests/test-forced-selection.ts
+++ b/packages/taler-wallet-cli/src/integrationtests/test-forced-selection.ts
@@ -47,6 +47,7 @@ export async function runForcedSelectionTest(t: 
GlobalTestState) {
     exchangeBaseUrl: exchange.baseUrl,
     amount: "TESTKUDOS:10",
     bankBaseUrl: bank.baseUrl,
+    bankAccessApiBaseUrl: bank.bankAccessApiBaseUrl,
     forcedDenomSel: {
       denoms: [
         {
diff --git 
a/packages/taler-wallet-cli/src/integrationtests/test-wallet-dbless.ts 
b/packages/taler-wallet-cli/src/integrationtests/test-wallet-dbless.ts
index 3837e567..75d6c208 100644
--- a/packages/taler-wallet-cli/src/integrationtests/test-wallet-dbless.ts
+++ b/packages/taler-wallet-cli/src/integrationtests/test-wallet-dbless.ts
@@ -57,6 +57,7 @@ export async function runWalletDblessTest(t: GlobalTestState) 
{
       http,
       reserveKeyPair.pub,
       bank.baseUrl,
+      bank.bankAccessApiBaseUrl,
       exchangeInfo,
       "TESTKUDOS:10",
     );
diff --git 
a/packages/taler-wallet-cli/src/integrationtests/test-wallettesting.ts 
b/packages/taler-wallet-cli/src/integrationtests/test-wallettesting.ts
index c42ae5ad..c16a85f1 100644
--- a/packages/taler-wallet-cli/src/integrationtests/test-wallettesting.ts
+++ b/packages/taler-wallet-cli/src/integrationtests/test-wallettesting.ts
@@ -121,6 +121,7 @@ export async function runWallettestingTest(t: 
GlobalTestState) {
     amountToSpend: "TESTKUDOS:5",
     amountToWithdraw: "TESTKUDOS:10",
     bankBaseUrl: bank.baseUrl,
+    bankAccessApiBaseUrl: bank.bankAccessApiBaseUrl,
     exchangeBaseUrl: exchange.baseUrl,
     merchantAuthToken: merchantAuthToken,
     merchantBaseUrl: merchant.makeInstanceBaseUrl(),
@@ -144,6 +145,7 @@ export async function runWallettestingTest(t: 
GlobalTestState) {
   await wallet.client.call(WalletApiOperation.WithdrawTestBalance, {
     amount: "TESTKUDOS:10",
     bankBaseUrl: bank.baseUrl,
+    bankAccessApiBaseUrl: bank.bankAccessApiBaseUrl,
     exchangeBaseUrl: exchange.baseUrl,
   });
 
@@ -169,6 +171,7 @@ export async function runWallettestingTest(t: 
GlobalTestState) {
   await wallet.client.call(WalletApiOperation.WithdrawTestBalance, {
     amount: "TESTKUDOS:10",
     bankBaseUrl: bank.baseUrl,
+    bankAccessApiBaseUrl: bank.bankAccessApiBaseUrl,
     exchangeBaseUrl: exchange.baseUrl,
   });
 
diff --git a/packages/taler-wallet-core/src/bank-api-client.ts 
b/packages/taler-wallet-core/src/bank-api-client.ts
index fa93409c..c2de6598 100644
--- a/packages/taler-wallet-core/src/bank-api-client.ts
+++ b/packages/taler-wallet-core/src/bank-api-client.ts
@@ -197,10 +197,7 @@ export namespace BankApi {
     );
 
     logger.info(`response status ${resp.status}`);
-    const respJson = await readSuccessResponseJsonOrThrow(
-      resp,
-      codecForAny(),
-    );
+    const respJson = await readSuccessResponseJsonOrThrow(resp, codecForAny());
 
     // FIXME: We don't check the status here!
   }
@@ -214,7 +211,7 @@ export namespace BankApi {
       `accounts/${bankUser.username}/withdrawals/${wopi.withdrawal_id}/abort`,
       bank.baseUrl,
     );
-    await bank.http.postJson(
+    const resp = await bank.http.postJson(
       url.href,
       {},
       {
@@ -226,6 +223,7 @@ export namespace BankApi {
         },
       },
     );
+    await readSuccessResponseJsonOrThrow(resp, codecForAny());
   }
 }
 
diff --git a/packages/taler-wallet-core/src/dbless.ts 
b/packages/taler-wallet-core/src/dbless.ts
index 4669b0be..3a775c3f 100644
--- a/packages/taler-wallet-core/src/dbless.ts
+++ b/packages/taler-wallet-core/src/dbless.ts
@@ -111,12 +111,13 @@ export async function topupReserveWithDemobank(
   http: HttpRequestLibrary,
   reservePub: string,
   bankBaseUrl: string,
+  bankAccessApiBaseUrl: string,
   exchangeInfo: ExchangeInfo,
   amount: AmountString,
 ) {
   const bankHandle: BankServiceHandle = {
     baseUrl: bankBaseUrl,
-    bankAccessApiBaseUrl: "??", // FIXME!
+    bankAccessApiBaseUrl: bankAccessApiBaseUrl,
     http,
   };
   const bankUser = await BankApi.createRandomBankUser(bankHandle);
diff --git a/packages/taler-wallet-core/src/operations/testing.ts 
b/packages/taler-wallet-core/src/operations/testing.ts
index bec8ec8f..8d28c62f 100644
--- a/packages/taler-wallet-core/src/operations/testing.ts
+++ b/packages/taler-wallet-core/src/operations/testing.ts
@@ -348,6 +348,7 @@ export async function runIntegrationTest(
   await withdrawTestBalance(ws, {
     amount: args.amountToWithdraw,
     bankBaseUrl: args.bankBaseUrl,
+    bankAccessApiBaseUrl: args.bankAccessApiBaseUrl,
     exchangeBaseUrl: args.exchangeBaseUrl,
   });
   await ws.runUntilDone();
@@ -376,6 +377,7 @@ export async function runIntegrationTest(
   await withdrawTestBalance(ws, {
     amount: Amounts.stringify(withdrawAmountTwo),
     bankBaseUrl: args.bankBaseUrl,
+    bankAccessApiBaseUrl: args.bankBaseUrl,
     exchangeBaseUrl: args.exchangeBaseUrl,
   });
 
diff --git a/packages/taler-wallet-core/src/operations/withdraw.ts 
b/packages/taler-wallet-core/src/operations/withdraw.ts
index f92ce506..03ec9b61 100644
--- a/packages/taler-wallet-core/src/operations/withdraw.ts
+++ b/packages/taler-wallet-core/src/operations/withdraw.ts
@@ -382,6 +382,8 @@ export async function getBankWithdrawalInfo(
     codecForWithdrawOperationStatusResponse(),
   );
 
+  logger.info(`bank withdrawal operation status: ${j2s(status)}`);
+
   return {
     amount: Amounts.parseOrThrow(status.amount),
     confirmTransferUrl: status.confirm_transfer_url,
diff --git a/packages/taler-wallet-core/src/wallet.ts 
b/packages/taler-wallet-core/src/wallet.ts
index 3c83cea6..8ae4b2b7 100644
--- a/packages/taler-wallet-core/src/wallet.ts
+++ b/packages/taler-wallet-core/src/wallet.ts
@@ -740,6 +740,7 @@ async function dispatchRequestInternal(
       await withdrawTestBalance(ws, {
         amount: "TESTKUDOS:10",
         bankBaseUrl: "https://bank.test.taler.net/";,
+        bankAccessApiBaseUrl: "https://bank.test.taler.net/";,
         exchangeBaseUrl: "https://exchange.test.taler.net/";,
       });
       return {};

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