View Issue Details

IDProjectCategoryView StatusLast Update
0009719Talermerchant backendpublic2025-04-11 22:07
ReporterChristian Grothoff Assigned ToChristian Grothoff  
PrioritynormalSeverityfeatureReproducibilityN/A
Status assignedResolutionopen 
Platformi7OSDebian GNU/LinuxOS Versionsqueeze
Product Versiongit (master) 
Target Version1.0 
Summary0009719: test 0-limit for deposit instant-triggers KYC requirement in SPA
DescriptionThis is the common use-case for GLS, and must work.
TagsNo tags attached.

Activities

sebasjm

2025-04-10 22:24

developer   ~0024504

test-kyc-merchant-deposit already validate this, but AFAIK to get an up to date information from the merchant backend the merchant needs to try to create an order, otherwise is just cached in the database

sebasjm

2025-04-11 17:11

developer   ~0024513

kyc-merchant-deposit-rewrite has been updated to fail

Ends with
FATAL: test failed with exception Error: Too many retries

here https://git.taler.net/taler-typescript-core.git/tree/packages/taler-harness/src/integrationtests/test-kyc-merchant-deposit-rewrite.ts#n241

After retrying 10 times asking kyc status

Christian Grothoff

2025-04-11 21:07

manager   ~0024517

Well we see the following in the logs:

2025-04-11T20:57:15.490909+0200 taler-merchant-kyccheck-46502 INFO Will repeat inquiry in 6 days

That's after we got the KYC-still-required from the exchange. Naturally, your test doesn't then wait for 6 days.

There are several points to consider here thus:
1) 6 days is probably a bit high;
2) the test probably should still not rely on the background job to do this in a timely fashion, and instead should
start taler-merchant-kyccheck explicitly with timetravel (--timetravel) to force it to fetch the latest status.

I'll go and reduce it from 6 days to 6 hours, but please add the taler-merchant-kyccheck with timetravel (after the KYC was done!). Let's see if we then pass.

Christian Grothoff

2025-04-11 21:15

manager   ~0024518

ee77fb01..ce5fe921 increases kyc-check frequency to 6h as long as we have relevant soft zero-limits in the limits.

sebasjm

2025-04-11 22:07

developer   ~0024520

the test now does a timetravel +7 days before asking for kyc.

still the same behavior

Issue History

Date Modified Username Field Change
2025-04-09 17:21 Christian Grothoff New Issue
2025-04-09 17:21 Christian Grothoff Status new => assigned
2025-04-09 17:21 Christian Grothoff Assigned To => sebasjm
2025-04-10 22:24 sebasjm Note Added: 0024504
2025-04-11 17:11 sebasjm Note Added: 0024513
2025-04-11 17:12 sebasjm Assigned To sebasjm => Christian Grothoff
2025-04-11 17:12 sebasjm Category merchant backoffice SPA => merchant backend
2025-04-11 21:07 Christian Grothoff Note Added: 0024517
2025-04-11 21:15 Christian Grothoff Note Added: 0024518
2025-04-11 21:15 Christian Grothoff Assigned To Christian Grothoff => sebasjm
2025-04-11 22:07 sebasjm Note Added: 0024520
2025-04-11 22:07 sebasjm Assigned To sebasjm => Christian Grothoff