entrouvert/wcs
entrouvert
/
wcs
14
1
Fork 0
Code Issues Pull Requests 34 Releases Activity

testdefs: fix conversion from WebserviceResponseError to request error (#87641) #1209

Merged
fpeters merged 1 commits from wip/87641-requests-invalid-exception into main 2024-02-29 18:54:18 +01:00
Conversation 3 Commits 1 Files Changed 2 +5 -4
fpeters commented 2024-02-29 18:21:29 +01:00
Owner
Copy Link

@vdeniaud j'essaie ça mais je te laisse la main si tu veux gérer différemment.

@vdeniaud j'essaie ça mais je te laisse la main si tu veux gérer différemment.
fpeters added 1 commit 2024-02-29 18:21:30 +01:00
vdeniaud changed title from WIP: testdefs: fix conversion from WebserviceResponseError to request error (#87641) to testdefs: fix conversion from WebserviceResponseError to request error (#87641) 2024-02-29 18:53:05 +01:00
vdeniaud approved these changes 2024-02-29 18:53:37 +01:00
vdeniaud left a comment
Owner
Copy Link

Idéalement il ne faudrait rien afficher en plus mais ça n'a pas l'air évident, faisons comme ça

Idéalement il ne faudrait rien afficher en plus mais ça n'a pas l'air évident, faisons comme ça
fpeters merged commit 8f5adc758f into main 2024-02-29 18:54:18 +01:00
fpeters deleted branch wip/87641-requests-invalid-exception 2024-02-29 18:54:18 +01:00
vdeniaud commented 2024-02-29 18:56:32 +01:00
Owner
Copy Link

Il est tard donc c'est peut-être évident : quel est le truc qui fait que dans les tests l'exception est rattrapée mais qu'en conditions réelles ça remonte en trace ?

Il est tard donc c'est peut-être évident : quel est le truc qui fait que dans les tests l'exception est rattrapée mais qu'en conditions réelles ça remonte en trace ?
vdeniaud commented 2024-02-29 18:58:52 +01:00
Owner
Copy Link

OK le contexte est différent ce test c'est un ws dans un formulaire et la trace c'est un ws dans un workflow, pas de mystère.

OK le contexte est différent ce test c'est un ws dans un formulaire et la trace c'est un ws dans un workflow, pas de mystère.
fpeters commented 2024-03-01 08:24:42 +01:00
Author
Owner
Copy Link

OK le contexte est différent ce test c'est un ws dans un formulaire et la trace c'est un ws dans un workflow, pas de mystère.

Et pas de mystère sur ma flemme à écrire un test :/

> OK le contexte est différent ce test c'est un ws dans un formulaire et la trace c'est un ws dans un workflow, pas de mystère. Et pas de mystère sur ma flemme à écrire un test :/
Sign in to join this conversation.
Reviewers
No reviewers
vdeniaud
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/wcs#1209
No description provided.
Delete Branch "wip/87641-requests-invalid-exception"

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?