Développement #51624
EmailValidator et test mal écrit
Start date:
03 March 2021
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
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 ?
Files
Associated revisions
History
Updated by Benjamin Dauvergne almost 4 years ago
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'^(?:[./|]|.*[@%!`#&?]|.*/\.\./)')
Updated by Valentin Deniaud over 3 years ago
- File 0001-tests-fix-faulty-email-validator-test-51624.patch 0001-tests-fix-faulty-email-validator-test-51624.patch added
- Tracker changed from Bug to Développement
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Paul Marillonnet over 3 years ago
- Status changed from Solution proposée to Solution validée
Ok.
Updated by Valentin Deniaud over 3 years ago
- Status changed from Solution validée to 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)Updated by Frédéric Péters over 3 years ago
- Status changed from Résolu (à déployer) to Solution déployée
tests: fix faulty email validator test (#51624)