Development #51624
EmailValidator et test mal écrit
Début:
03 mars 2021
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Dans test_validators.py,
60 @pytest.mark.parametrize('email', ['ok@ok.com', 'a|b@ok.com', 'a/..b@ok.com']) 61 def test_email_validator_ok(email): 62 EmailValidator()('ok@ok.com')
Et forcément quand on corrige on s'aperçoit que le test ne passe pas pour 'a/..b@ok.com'... C'est grave docteur ?
Fichiers
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
Non a priori c'est le test qui est faut (c'est surtout pour documenter ce qu'accepte le validateur de Django ou pas). On a rajouté cette contrainte, mais j'ai oublié pourquoi :
LOCALPART_FORBIDDEN_RE = re.compile(r'^(?:[./|]|.*[@%!`#&?]|.*/\.\./)')
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Fichier 0001-tests-fix-faulty-email-validator-test-51624.patch 0001-tests-fix-faulty-email-validator-test-51624.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Paul Marillonnet il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4484b7cea85de85b693eb09c242205b979fdba2b Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu Mar 4 14:58:39 2021 +0100 tests: fix faulty email validator test (#51624)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: fix faulty email validator test (#51624)