View Issue Details

IDProjectCategoryView StatusLast Update
0009922Talerlibeufin-nexuspublic2025-07-11 00:40
ReporterChristian Grothoff Assigned ToFlorian Dold  
PrioritynormalSeveritytweakReproducibilityN/A
Status resolvedResolutionfixed 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version1.0 stretch goals 
Summary0009922: get wallet tests to work with libeufin [3h]
Descriptionmerchant.git :: ea8b6682..bd6574c1 deactivated the use of libeufin for the walletharness tests, as it failed with some permissions issue. We should look into it and make sure the tests pass with both fakebank and libeufin.
TagsNo tags attached.

Activities

Florian Dold

2025-05-22 02:25

manager   ~0024964

Problem seems to be in the harness: The add-kycauth endpoint isn't called with admin bank account credentials, but with exchange bank account credentials.

Florian Dold

2025-06-05 13:45

manager   ~0025148

Actually there are various test failures, they need to be looked at in more detail.

Florian Dold

2025-07-11 00:40

manager   ~0025489

commit beb6e25bf87b4e159497a13e3feae5c69ab59871 (HEAD -> master)
Author: Florian Dold <florian@dold.me>
Date: Fri Jul 11 00:37:46 2025 +0200

    harness,wallet: make tests pass under libeufin
    
    In most cases, we simply didn't create accounts required for the test.
    The taler-fakebank doesn't require this step and makes up accounts on
    the spot.
    
    But in withdrawal-bank-integrated, the bug was an idempotency issue in
    wallet-core that only showed up when run under libeufin (likely timing
    sensitive).

Issue History

Date Modified Username Field Change
2025-05-09 19:16 Christian Grothoff New Issue
2025-05-09 19:17 Christian Grothoff Status new => confirmed
2025-05-09 19:17 Christian Grothoff Target Version => 1.0 stretch goals
2025-05-11 10:55 Florian Dold Assigned To => Florian Dold
2025-05-11 10:55 Florian Dold Status confirmed => assigned
2025-05-22 02:25 Florian Dold Note Added: 0024964
2025-06-05 13:45 Florian Dold Summary get wallet tests to work with libeufin => get wallet tests to work with libeufin [3h]
2025-06-05 13:45 Florian Dold Note Added: 0025148
2025-07-11 00:40 Florian Dold Status assigned => resolved
2025-07-11 00:40 Florian Dold Resolution open => fixed
2025-07-11 00:40 Florian Dold Note Added: 0025489