Limiter les actions pour les comptes expirés

[raphodn]

voir l'issue #94

Modifications apportées :

• nouvelle méthode utils.checkUserIsExpired(user)
• Login : amélioration et ajout de tests (un compte expiré ne devrait pas pouvoir se connecter)
• Users : pour un compte expiré, impossible de créer un email (ni créer une redirection ou changer le password, mais ces actions sont reservés à l'utilisateur(trice) connecté, et comme un compte expiré ne peut pas se connecter...)
• users.json : ajout de utilisateur.expire
• quelques modifs de style (sorry ca rend la PR compliqué à lire...)

[raphodn]

Bizarre, les tests passent pour moi en local. Ca semble venir de users.json > utilisateur.actif qui n'a pas de champs end, et semble donc être considéré comme expiré.

La méthode que j'ai implémenté est la suivante :

module.exports.checkUserIsExpired = function(user) {
  // L'utilisateur(trice) est considéré comme expiré si:
  // - il/elle existe
  // - il/elle a une date de fin
  // - son/sa date de fin est passée
  return user &&
    user.end != undefined &&
    new Date(user.end).getTime() < new Date().getTime();
}

[jdauphant]

@raphodn Il y a des comptes qui n'ont pas de date de fin, il faut les ignorer (j'ai le même soucis avec mon autre PR)

[jdauphant]

ça ne serait pas mieux juste isExpired ?

[raphodn]

t'as raison je vais mettre isExpired

[jdauphant]

Pour moi le code est bon.
A part un détail, j'aurais plutôt vu "user.isExpired" au lieu de "user.userIsExpired"

[jdauphant]

Les tests ne doit pas pas prendre le fichier "users.json" mais l'api du site beta.gouv.fr à mon avis

[raphodn]

Il y a des comptes qui n'ont pas de date de fin, il faut les ignorer (j'ai le même soucis avec mon autre PR)

Tu veux dire que c'est normal que les tests failent, ou que je dois modifier mon implémentation ?

A part un détail, j'aurais plutôt vu "user.isExpired" au lieu de "user.userIsExpired"

will do 👍

Les tests ne doit pas pas prendre le fichier "users.json" mais l'api du site beta.gouv.fr à mon avis

t'as fait la modif il semblerait du coup ?