NewBill.js : post vs patch

[OgdenRichard]

En l'état actuel du code c'est la méthode NewBill.handleChangeFile qui lance le call api POST, tandis que la méthode NewBill.handleSubmit lance un call api PATCH.
Cela pose quelques problèmes :

• lancer un POST à l'upload d'un fichier a-t-il la moindre utilité ?
• de plus si on upload un autre fichier avant de submit le form, on aura deux tickets créés dont un comportant des null sur les champs required
• par rapport au projet, on nous demande de tester la méthode POST sur NewBill, et instinctivement on pourrait penser qu'il s'agit de tester le call lancé au submit, donc au final... le call PATCH. Le cahier des charges n'est donc pas très clair.

Enfin, la gestion des catch sur NewBill oblige l'étudiant à donner dans des solutions compliquées pour tester les erreurs API.
Par exemple :
expect(console.error).toHaveBeenCalled()
ne laisse pas vraiment la possibilité de tester le message d'erreur, et donc de vérifier s'il s'agit d'une erreur 404 ou 500.