From f2492cac6d0e97aa0c90b0659cae47af834af699 Mon Sep 17 00:00:00 2001 From: ms Date: Thu, 14 Oct 2021 23:23:35 +0200 Subject: Fix key-rotation test case. --- .../integrationtests/test-libeufin-keyrotation.ts | 24 ++++++++++++++++------ 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/packages/taler-wallet-cli/src/integrationtests/test-libeufin-keyrotation.ts b/packages/taler-wallet-cli/src/integrationtests/test-libeufin-keyrotation.ts index 6a5e5c204..8e527804c 100644 --- a/packages/taler-wallet-cli/src/integrationtests/test-libeufin-keyrotation.ts +++ b/packages/taler-wallet-cli/src/integrationtests/test-libeufin-keyrotation.ts @@ -57,11 +57,23 @@ export async function runLibeufinKeyrotationTest(t: GlobalTestState) { user01sandbox.ebicsBankAccount.subscriber.hostID, ); - const resp = await LibeufinNexusApi.fetchTransactions( - libeufinServices.libeufinNexus, - user01nexus.localAccountName, - ); - console.log("Attempted fetch after key rotation", resp.data); - t.assertTrue(resp.status == 500); + try { + await LibeufinNexusApi.fetchTransactions( + libeufinServices.libeufinNexus, + user01nexus.localAccountName, + ); + } catch (e: any) { + /** + * Asserting that Nexus responded with a 500 Internal server + * error, because the bank signed the last response with a new + * key pair that was never downloaded by Nexus. + * + * NOTE: the bank accepted the request addressed to the old + * public key. Should it in this case reject the request even + * before trying to verify it? + */ + t.assertTrue(e.response.status == 500); + t.assertTrue(e.response.data.code == 9000); + } } runLibeufinKeyrotationTest.suites = ["libeufin"]; -- cgit v1.2.3