search

Floppa-equipe-16 / floppa

API for a marketplace or auction website
MIT License
0 stars 0 forks source link

Product affiliated class test #39

Closed JeromeLev141 closed 1 year ago

JEGIR69 commented 1 year ago

Je ne sais pas si c'est mieux de faire un check-up du message de l'erreur, pour s'assurer qu'il retourne le bon message d'erreur par exemple "Invalid parameter Name" et non juste si la function throw une exeption de type InvalidParam.

JeromeLev141 commented 1 year ago

Je ne sais pas si c'est mieux de faire un check-up du message de l'erreur, pour s'assurer qu'il retourne le bon message d'erreur par exemple "Invalid parameter Name" et non juste si la function throw une exeption de type InvalidParam.

Ce serait plus un test de la Classe Exception

JEGIR69 commented 1 year ago

Ce serait plus un test de la Classe Exception

je parle si ton test y check si name est Missing mais throws un MissingParam dont la description est "missing PhoneNumber", ça marche pas.

AnthonySirois commented 1 year ago

Ce serait plus un test de la Classe Exception

je parle si ton test y check si name est Missing mais throws un MissingParam dont la description est "missing PhoneNumber", ça marche pas.

Perso je crois que ce n'est pas nécessaire de tester le message, parce que si le formattage des messages est changé, il va falloir revoir tous les tests

AnthonySirois commented 1 year ago

rip pour le close/reopen de la PR, j'ai été trop vite, je pensais que c'était un bouton pour discard mon comment... sorry