Test avec nock sur api ovh à ameliorer

LucasCharrier commented 3 years ago

On ne check pas lors des nocks des apis qu'ils sont appelés avec le bon argument ou on ne teste pas dans les fonctions qui appellent ces apis que les arguments qui lui sont passés sont valides, et certains tests passent alors qu'ils ne devraient pas. Exemple: Dans userController.js, suite à une refacto sur une branche, j'ai fait l'erreur suivante:

module.exports.createEmail = async function (userInfo, creator, toEmail) {
[...]
  await BetaGouv.createEmail(userinfo, password);
[...]

BetaGouv.createEmail attends en premier argument un user.id du type lucas.charrier, je lui ai passé l'objet userInfo en entier et aucun test n'a faillé, alors que la fonction createEmail de userController.js semble assez bien testée.

jdauphant commented 3 years ago

En effet, c'est dérangeant ^^

LucasCharrier commented 3 years ago

Sur createEmail des tests ont été rajouté sur #426 , il reste à faire de même sur les autres fonction du fichier betagouv.

betagouv / secretariat

Test avec nock sur api ovh à ameliorer #424