entrouvert
/
passerelle
15
0
Fork 0
Code Issues Pull Requests 15 Releases Activity

toulouse_maelis: check q is a DUI in search-family-dui (#88873) #505

Merged
bdauvergne merged 2 commits from wip/88873-toulouse-parsifal-valider-le-DUI into main 2024-04-03 17:17:38 +02:00
Conversation 2 Commits 2 Files Changed 3 +22 -11
bdauvergne commented 2024-03-29 15:11:45 +01:00
Owner
Copy Link

A DUI is an xs:int.

A DUI is an xs:int.
bdauvergne added 1 commit 2024-03-29 15:11:45 +01:00
gitea/passerelle/pipeline/head This commit looks good Details
2d9d9ded3e toulouse_maelis: check q is a DUI in search-family-dui (#88873) 
A DUI is an xs:int.
nroche approved these changes 2024-04-03 16:17:14 +02:00
nroche left a comment
Owner
Copy Link

Ack.
J'imaginais juste un ligne telle que if q and re.match(r'\d+', q):
mais oui, a DUI is an xs:int.
Aussi j'aurais profité de l'occasion pour ajouter un return self.requests_mock.add( dans utils.py::add_soap_response pour tester s'il y a eu appel ou non, ex:
assert soap_mock.call_count == 1

Ack. J'imaginais juste un ligne telle que `if q and re.match(r'\d+', q):` mais oui, a DUI is an xs:int. Aussi j'aurais profité de l'occasion pour ajouter un `return self.requests_mock.add(` dans utils.py::add_soap_response pour tester s'il y a eu appel ou non, ex: `assert soap_mock.call_count == 1`
bdauvergne commented 2024-04-03 16:53:48 +02:00
Author
Owner
Copy Link

Ack.
J'imaginais juste un ligne telle que if q and re.match(r'\d+', q):
mais oui, a DUI is an xs:int.

Je me suis dit autant pas lever non plus d'erreur quand quelqu'un tapera 9999999999.

Aussi j'aurais profité de l'occasion pour ajouter un return self.requests_mock.add( dans utils.py::add_soap_response pour tester s'il y a eu appel ou non, ex:
assert soap_mock.call_count == 1

Lapin compris, file le patch si tu veux que je l'intègre.

> Ack. > J'imaginais juste un ligne telle que `if q and re.match(r'\d+', q):` > mais oui, a DUI is an xs:int. Je me suis dit autant pas lever non plus d'erreur quand quelqu'un tapera 9999999999. > Aussi j'aurais profité de l'occasion pour ajouter un `return self.requests_mock.add(` dans utils.py::add_soap_response pour tester s'il y a eu appel ou non, ex: > `assert soap_mock.call_count == 1` Lapin compris, file le patch si tu veux que je l'intègre.
nroche added 1 commit 2024-04-03 17:01:14 +02:00
gitea/passerelle/pipeline/head This commit looks good Details
d7500948f0 toulouse-maelis: [tests] count soap requests (#88873)
nroche commented 2024-04-03 17:10:33 +02:00
Owner
Copy Link

J'ai ajouté le patch à la branche.
A prendre ou à laisser, ton commit me va très bien sans.

J'ai ajouté le patch à la branche. A prendre ou à laisser, ton commit me va très bien sans.
bdauvergne merged commit d4bb8059e0 into main 2024-04-03 17:17:38 +02:00
bdauvergne deleted branch wip/88873-toulouse-parsifal-valider-le-DUI 2024-04-03 17:17:38 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
nroche
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/passerelle#505
No description provided.
Delete Branch "wip/88873-toulouse-parsifal-valider-le-DUI"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?