[taler-wallet-core] branch master updated: -dce and type fixes

[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 ae818df3b -dce and type fixes
ae818df3b is described below

commit ae818df3bff18bc1517107f3728df65daafd3b20
Author: Florian Dold <florian@dold.me>
AuthorDate: Wed Nov 2 18:27:54 2022 +0100

    -dce and type fixes
---
 .../src/wallet/DepositPage/stories.tsx             |  15 --
 .../src/wallet/DepositPage/test.ts                 | 240 ++++++++++++---------
 2 files changed, 137 insertions(+), 118 deletions(-)

diff --git 
a/packages/taler-wallet-webextension/src/wallet/DepositPage/stories.tsx 
b/packages/taler-wallet-webextension/src/wallet/DepositPage/stories.tsx
index af9c620cb..f03788d4e 100644
--- a/packages/taler-wallet-webextension/src/wallet/DepositPage/stories.tsx
+++ b/packages/taler-wallet-webextension/src/wallet/DepositPage/stories.tsx
@@ -28,21 +28,6 @@ export default {
   title: "wallet/deposit",
 };
 
-const accountsType = {
-  "": labelForAccountType(""),
-  iban: labelForAccountType("iban"),
-  bitcoin: labelForAccountType("bitcoin"),
-  "x-taler-bank": labelForAccountType("x-taler-bank"),
-};
-async function alwaysReturnFeeToOne(): Promise<DepositGroupFees> {
-  const fee = {
-    currency: "EUR",
-    value: 1,
-    fraction: 0,
-  };
-  return { coin: fee, refresh: fee, wire: fee };
-}
-
 // const ac = parsePaytoUri("payto://iban/ES8877998399652238")!;
 // const accountMap = createLabelsForBankAccount([ac]);
 
diff --git a/packages/taler-wallet-webextension/src/wallet/DepositPage/test.ts 
b/packages/taler-wallet-webextension/src/wallet/DepositPage/test.ts
index 4a648312e..9f336ac1a 100644
--- a/packages/taler-wallet-webextension/src/wallet/DepositPage/test.ts
+++ b/packages/taler-wallet-webextension/src/wallet/DepositPage/test.ts
@@ -20,53 +20,60 @@
  */
 
 import {
-  Amounts, DepositGroupFees,
+  Amounts,
+  DepositGroupFees,
   parsePaytoUri,
-  stringifyPaytoUri
+  stringifyPaytoUri,
 } from "@gnu-taler/taler-util";
 import { WalletApiOperation } from "@gnu-taler/taler-wallet-core";
 import { expect } from "chai";
-import { createWalletApiMock, mountHook, nullFunction } from 
"../../test-utils.js";
+import {
+  createWalletApiMock,
+  mountHook,
+  nullFunction,
+} from "../../test-utils.js";
 
 import { useComponentState } from "./state.js";
 
 const currency = "EUR";
 const withoutFee = (): DepositGroupFees => ({
-  coin: Amounts.parseOrThrow(`${currency}:0`),
-  wire: Amounts.parseOrThrow(`${currency}:0`),
-  refresh: Amounts.parseOrThrow(`${currency}:0`),
+  coin: Amounts.stringify(`${currency}:0`),
+  wire: Amounts.stringify(`${currency}:0`),
+  refresh: Amounts.stringify(`${currency}:0`),
 });
 
 const withSomeFee = (): DepositGroupFees => ({
-  coin: Amounts.parseOrThrow(`${currency}:1`),
-  wire: Amounts.parseOrThrow(`${currency}:1`),
-  refresh: Amounts.parseOrThrow(`${currency}:1`),
+  coin: Amounts.stringify(`${currency}:1`),
+  wire: Amounts.stringify(`${currency}:1`),
+  refresh: Amounts.stringify(`${currency}:1`),
 });
 
 describe("DepositPage states", () => {
   it("should have status 'no-enough-balance' when balance is empty", async () 
=> {
     const { handler, mock } = createWalletApiMock();
-    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction }
+    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction 
};
 
     handler.addWalletCallResponse(WalletApiOperation.GetBalances, undefined, {
-      balances: [{
-        available: `${currency}:0`,
-        hasPendingTransactions: false,
-        pendingIncoming: `${currency}:0`,
-        pendingOutgoing: `${currency}:0`,
-        requiresUserInput: false,
-      }],
-    })
-    handler.addWalletCallResponse(WalletApiOperation.ListKnownBankAccounts, 
undefined, {
-      accounts: []
+      balances: [
+        {
+          available: `${currency}:0`,
+          hasPendingTransactions: false,
+          pendingIncoming: `${currency}:0`,
+          pendingOutgoing: `${currency}:0`,
+          requiresUserInput: false,
+        },
+      ],
     });
+    handler.addWalletCallResponse(
+      WalletApiOperation.ListKnownBankAccounts,
+      undefined,
+      {
+        accounts: [],
+      },
+    );
 
     const { pullLastResultOrThrow, waitForStateUpdate, assertNoPendingUpdate } 
=
-      mountHook(() =>
-        useComponentState(
-          props, mock
-        ),
-      );
+      mountHook(() => useComponentState(props, mock));
 
     {
       const { status } = pullLastResultOrThrow();
@@ -81,31 +88,33 @@ describe("DepositPage states", () => {
     }
 
     await assertNoPendingUpdate();
-    expect(handler.getCallingQueueState()).eq("empty")
+    expect(handler.getCallingQueueState()).eq("empty");
   });
 
   it("should have status 'no-accounts' when balance is not empty and accounts 
is empty", async () => {
     const { handler, mock } = createWalletApiMock();
-    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction }
+    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction 
};
 
     handler.addWalletCallResponse(WalletApiOperation.GetBalances, undefined, {
-      balances: [{
-        available: `${currency}:1`,
-        hasPendingTransactions: false,
-        pendingIncoming: `${currency}:0`,
-        pendingOutgoing: `${currency}:0`,
-        requiresUserInput: false,
-      }],
-    })
-    handler.addWalletCallResponse(WalletApiOperation.ListKnownBankAccounts, 
undefined, {
-      accounts: []
+      balances: [
+        {
+          available: `${currency}:1`,
+          hasPendingTransactions: false,
+          pendingIncoming: `${currency}:0`,
+          pendingOutgoing: `${currency}:0`,
+          requiresUserInput: false,
+        },
+      ],
     });
+    handler.addWalletCallResponse(
+      WalletApiOperation.ListKnownBankAccounts,
+      undefined,
+      {
+        accounts: [],
+      },
+    );
     const { pullLastResultOrThrow, waitForStateUpdate, assertNoPendingUpdate } 
=
-      mountHook(() =>
-        useComponentState(
-          props, mock
-        )
-      );
+      mountHook(() => useComponentState(props, mock));
 
     {
       const { status } = pullLastResultOrThrow();
@@ -120,7 +129,7 @@ describe("DepositPage states", () => {
     }
 
     await assertNoPendingUpdate();
-    expect(handler.getCallingQueueState()).eq("empty")
+    expect(handler.getCallingQueueState()).eq("empty");
   });
 
   const ibanPayto = {
@@ -138,27 +147,29 @@ describe("DepositPage states", () => {
 
   it("should have status 'ready' but unable to deposit ", async () => {
     const { handler, mock } = createWalletApiMock();
-    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction }
+    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction 
};
 
     handler.addWalletCallResponse(WalletApiOperation.GetBalances, undefined, {
-      balances: [{
-        available: `${currency}:1`,
-        hasPendingTransactions: false,
-        pendingIncoming: `${currency}:0`,
-        pendingOutgoing: `${currency}:0`,
-        requiresUserInput: false,
-      }],
-    })
-    handler.addWalletCallResponse(WalletApiOperation.ListKnownBankAccounts, 
undefined, {
-      accounts: [ibanPayto]
+      balances: [
+        {
+          available: `${currency}:1`,
+          hasPendingTransactions: false,
+          pendingIncoming: `${currency}:0`,
+          pendingOutgoing: `${currency}:0`,
+          requiresUserInput: false,
+        },
+      ],
     });
+    handler.addWalletCallResponse(
+      WalletApiOperation.ListKnownBankAccounts,
+      undefined,
+      {
+        accounts: [ibanPayto],
+      },
+    );
 
     const { pullLastResultOrThrow, waitForStateUpdate, assertNoPendingUpdate } 
=
-      mountHook(() =>
-        useComponentState(
-          props, mock
-        ),
-      );
+      mountHook(() => useComponentState(props, mock));
 
     {
       const { status } = pullLastResultOrThrow();
@@ -178,35 +189,49 @@ describe("DepositPage states", () => {
     }
 
     await assertNoPendingUpdate();
-    expect(handler.getCallingQueueState()).eq("empty")
+    expect(handler.getCallingQueueState()).eq("empty");
   });
 
   it("should not be able to deposit more than the balance ", async () => {
     const { handler, mock } = createWalletApiMock();
-    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction }
+    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction 
};
 
     handler.addWalletCallResponse(WalletApiOperation.GetBalances, undefined, {
-      balances: [{
-        available: `${currency}:5`,
-        hasPendingTransactions: false,
-        pendingIncoming: `${currency}:0`,
-        pendingOutgoing: `${currency}:0`,
-        requiresUserInput: false,
-      }],
-    })
-    handler.addWalletCallResponse(WalletApiOperation.ListKnownBankAccounts, 
undefined, {
-      accounts: [talerBankPayto, ibanPayto]
+      balances: [
+        {
+          available: `${currency}:5`,
+          hasPendingTransactions: false,
+          pendingIncoming: `${currency}:0`,
+          pendingOutgoing: `${currency}:0`,
+          requiresUserInput: false,
+        },
+      ],
     });
-    handler.addWalletCallResponse(WalletApiOperation.GetFeeForDeposit, 
undefined, withoutFee())
-    handler.addWalletCallResponse(WalletApiOperation.GetFeeForDeposit, 
undefined, withoutFee())
-    handler.addWalletCallResponse(WalletApiOperation.GetFeeForDeposit, 
undefined, withoutFee())
+    handler.addWalletCallResponse(
+      WalletApiOperation.ListKnownBankAccounts,
+      undefined,
+      {
+        accounts: [talerBankPayto, ibanPayto],
+      },
+    );
+    handler.addWalletCallResponse(
+      WalletApiOperation.GetFeeForDeposit,
+      undefined,
+      withoutFee(),
+    );
+    handler.addWalletCallResponse(
+      WalletApiOperation.GetFeeForDeposit,
+      undefined,
+      withoutFee(),
+    );
+    handler.addWalletCallResponse(
+      WalletApiOperation.GetFeeForDeposit,
+      undefined,
+      withoutFee(),
+    );
 
     const { pullLastResultOrThrow, waitForStateUpdate, assertNoPendingUpdate } 
=
-      mountHook(() =>
-        useComponentState(
-          props, mock
-        ),
-      );
+      mountHook(() => useComponentState(props, mock));
 
     {
       const { status } = pullLastResultOrThrow();
@@ -214,7 +239,7 @@ describe("DepositPage states", () => {
     }
 
     expect(await waitForStateUpdate()).true;
-    const accountSelected = stringifyPaytoUri(ibanPayto.uri)
+    const accountSelected = stringifyPaytoUri(ibanPayto.uri);
 
     {
       const r = pullLastResultOrThrow();
@@ -227,7 +252,7 @@ describe("DepositPage states", () => {
       expect(r.totalFee).deep.eq(Amounts.parseOrThrow(`${currency}:0`));
       expect(r.account.onChange).not.undefined;
 
-      r.account.onChange!(accountSelected)
+      r.account.onChange!(accountSelected);
     }
 
     expect(await waitForStateUpdate()).true;
@@ -316,29 +341,39 @@ describe("DepositPage states", () => {
 
   it("should calculate the fee upon entering amount ", async () => {
     const { handler, mock } = createWalletApiMock();
-    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction }
+    const props = { currency, onCancel: nullFunction, onSuccess: nullFunction 
};
 
     handler.addWalletCallResponse(WalletApiOperation.GetBalances, undefined, {
-      balances: [{
-        available: `${currency}:10`,
-        hasPendingTransactions: false,
-        pendingIncoming: `${currency}:0`,
-        pendingOutgoing: `${currency}:0`,
-        requiresUserInput: false,
-      }],
-    })
-    handler.addWalletCallResponse(WalletApiOperation.ListKnownBankAccounts, 
undefined, {
-      accounts: [talerBankPayto, ibanPayto]
+      balances: [
+        {
+          available: `${currency}:10`,
+          hasPendingTransactions: false,
+          pendingIncoming: `${currency}:0`,
+          pendingOutgoing: `${currency}:0`,
+          requiresUserInput: false,
+        },
+      ],
     });
-    handler.addWalletCallResponse(WalletApiOperation.GetFeeForDeposit, 
undefined, withSomeFee())
-    handler.addWalletCallResponse(WalletApiOperation.GetFeeForDeposit, 
undefined, withSomeFee())
+    handler.addWalletCallResponse(
+      WalletApiOperation.ListKnownBankAccounts,
+      undefined,
+      {
+        accounts: [talerBankPayto, ibanPayto],
+      },
+    );
+    handler.addWalletCallResponse(
+      WalletApiOperation.GetFeeForDeposit,
+      undefined,
+      withSomeFee(),
+    );
+    handler.addWalletCallResponse(
+      WalletApiOperation.GetFeeForDeposit,
+      undefined,
+      withSomeFee(),
+    );
 
     const { pullLastResultOrThrow, waitForStateUpdate, assertNoPendingUpdate } 
=
-      mountHook(() =>
-        useComponentState(
-          props, mock
-        ),
-      );
+      mountHook(() => useComponentState(props, mock));
 
     {
       const { status } = pullLastResultOrThrow();
@@ -346,7 +381,7 @@ describe("DepositPage states", () => {
     }
 
     expect(await waitForStateUpdate()).true;
-    const accountSelected = stringifyPaytoUri(ibanPayto.uri)
+    const accountSelected = stringifyPaytoUri(ibanPayto.uri);
 
     {
       const r = pullLastResultOrThrow();
@@ -359,7 +394,7 @@ describe("DepositPage states", () => {
       expect(r.totalFee).deep.eq(Amounts.parseOrThrow(`${currency}:0`));
       expect(r.account.onChange).not.undefined;
 
-      r.account.onChange!(accountSelected)
+      r.account.onChange!(accountSelected);
     }
 
     expect(await waitForStateUpdate()).true;
@@ -392,7 +427,6 @@ describe("DepositPage states", () => {
     }
 
     await assertNoPendingUpdate();
-    expect(handler.getCallingQueueState()).eq("empty")
+    expect(handler.getCallingQueueState()).eq("empty");
   });
-
 });

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