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

testdef, ne pas enregistrer d'erreurs sur le formulaire (#76502) #235

Merged
fpeters merged 2 commits from wip/76502-testdef-ne-pas-enregistrer-d-err into main 2023-05-02 16:37:27 +02:00
Conversation 0 Commits 2 Files Changed 4 +46 -7
vdeniaud commented 2023-04-11 15:52:29 +02:00
Owner
Copy Link

Ça masquait un bug, corrigé dans 0001.

Il y aurait pu y avoir tout un travail pour rattacher les erreurs à un test au lieu du formulaire, mais je ne vois pas d'erreurs intéressantes/légitimes, donc juste ne pas enregistrer et laisser l'exception être levée, ça donne l'assurance qu'il n'y ait pas d'autres bugs masqués dans les tests.

Dans la réalité, on verra bien si ça fait des traces et on gérera.

Ça masquait un bug, corrigé dans 0001. Il y aurait pu y avoir tout un travail pour rattacher les erreurs à un test au lieu du formulaire, mais je ne vois pas d'erreurs intéressantes/légitimes, donc juste ne pas enregistrer et laisser l'exception être levée, ça donne l'assurance qu'il n'y ait pas d'autres bugs masqués dans les tests. Dans la réalité, on verra bien si ça fait des traces et on gérera.
vdeniaud added 2 commits 2023-04-11 15:52:30 +02:00
gitea/wcs/pipeline/head This commit looks good Details
e6463db7c4 testdef: never record errors on form (#76480)
vdeniaud force-pushed wip/76502-testdef-ne-pas-enregistrer-d-err from e6463db7c4 to 9330e06c8d 2023-04-27 13:51:24 +02:00 Compare
fpeters approved these changes 2023-04-30 13:27:05 +02:00
fpeters merged commit 1e0c2f8fbb into main 2023-05-02 16:37:27 +02:00
fpeters deleted branch wip/76502-testdef-ne-pas-enregistrer-d-err 2023-05-02 16:37:27 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
fpeters
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#235
No description provided.
Delete Branch "wip/76502-testdef-ne-pas-enregistrer-d-err"

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?