View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009922 | Taler | libeufin-nexus | public | 2025-05-09 17:16 | 2025-08-31 17:10 |
| Reporter | Christian Grothoff | Assigned To | Florian Dold | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | tweak | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
| Product Version | git (master) | ||||
| Target Version | 1.0 stretch goals | Fixed in Version | 1.0 stretch goals | ||
| Summary | 0009922: get wallet tests to work with libeufin [3h] | ||||
| Description | merchant.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. | ||||
| Tags | No tags attached. | ||||
| 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. | |
| Actually there are various test failures, they need to be looked at in more detail. | |
| 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). | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2025-05-09 17:16 | Christian Grothoff | New Issue | |
| 2025-05-09 17:17 | Christian Grothoff | Status | new => implementing |
| 2025-05-09 17:17 | Christian Grothoff | Target Version | => 1.0 stretch goals |
| 2025-05-11 08:55 | Florian Dold | Assigned To | => Florian Dold |
| 2025-05-11 08:55 | Florian Dold | Status | implementing => assigned |
| 2025-05-22 00:25 | Florian Dold | Note Added: 0024964 | |
| 2025-06-05 11:45 | Florian Dold | Summary | get wallet tests to work with libeufin => get wallet tests to work with libeufin [3h] |
| 2025-06-05 11:45 | Florian Dold | Note Added: 0025148 | |
| 2025-07-10 22:40 | Florian Dold | Status | assigned => informing |
| 2025-07-10 22:40 | Florian Dold | Resolution | open => fixed |
| 2025-07-10 22:40 | Florian Dold | Note Added: 0025489 | |
| 2025-08-31 17:09 | Christian Grothoff | Status | informing => closed |
| 2025-08-31 17:10 | Christian Grothoff | Fixed in Version | => 1.0 stretch goals |