Skip to content

correction tests fonctionnels liés à Paybox#2318

Open
agallou wants to merge 1 commit into
masterfrom
fix_tests_paybox
Open

correction tests fonctionnels liés à Paybox#2318
agallou wants to merge 1 commit into
masterfrom
fix_tests_paybox

Conversation

@agallou

@agallou agallou commented Jul 14, 2026

Copy link
Copy Markdown
Member

Les tests fonctionnels qui appellait paybox ne passaient plus, par exemple https://github.com/afup/web/actions/runs/29336322441/job/87099244889?pr=2317

Cela pour deux raisons.

La première, il y a maintenant une seconde étape, un autre formulaire à valider avant de saisir le numéro de carte. Pour cela en ajoutant une étape dans les tests behats, cela évite le souci.

Et ensuite la seconde, qui ne concernait que le scénario "Scenario: Accès à l'adhésion entreprise"

Paybox nous renvoyait une erreur : "An error occurred while creating or updating row in fappel".

A priori car le cmd envoyé dans ce cas n'était pas unique.

On ajoute donc un élement unique à la fin et le supprime dans le callback pour ne pas inteférer avec son parsing.

@agallou agallou requested review from Mopolo and stakovicz as code owners July 14, 2026 15:41
Les tests fonctionnels qui appellait paybox ne passaient plus, par exemple https://github.com/afup/web/actions/runs/29336322441/job/87099244889?pr=2317

Cela pour deux raisons.

La première, il y a maintenant une seconde étape, un autre formulaire à valider avant de saisir le numéro de carte.

Et ensuite la seconde, qui ne concernait que le scénario "Scenario: Accès à l'adhésion entreprise"

Paybox nous renvoyait une erreur : "An error occurred while creating or updating row in fappel".

A priori car le cmd envoyé dans ce cas n'était pas unique.

On ajoute donc un élement unique à la fin et le supprime dans le callback pour ne pas inteférer avec son parsing.
@agallou agallou force-pushed the fix_tests_paybox branch from 8e1117e to 845808b Compare July 14, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants