Development #65241
tests admin_pages - ne pas tester en mode pickle de façon systématique
0%
Description
Dans ces écrans on ne manipule que des données stockées en xml.
Supprimer les:
def pytest_generate_tests(metafunc): if 'pub' in metafunc.fixturenames: metafunc.parametrize('pub', ['pickle', 'sql'], indirect=True)
de toutes les tests/admin_pages
Ajouter des tests explicites en nosql pour les quelques tests qui testent des bout de code ayant un comportement différent en fonction de sql/pickle mode
Vérifier la couverture de code
Fichiers
Révisions associées
Historique
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Fichier 0001-misc-remove-pickle-auto-tests-from-admin_pages-65241.patch 0001-misc-remove-pickle-auto-tests-from-admin_pages-65241.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Différence de -2 lignes pour la couverture de code, dans wcs/qommon/ident/password.py dans PasswordAuthMethod:submit
Un des tests admin_pages/test_user.py passaient par accident dans la condition if PasswordAccount.has_key(username):
, car joué 2 fois (une fois en pickle l'autre en sql), sans wipe() des users.
(comparer https://jenkins.entrouvert.org/job/wcs/3701/cobertura/qommon_ident/password_py/ et https://jenkins.entrouvert.org/job/wcs-wip/job/wip%252Fcheck-test-admin-pages-sql-mode/31/cobertura/qommon_ident/password_py/)
Mis à jour par Frédéric Péters il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Lauréline Guérin il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c65d516a028bcb089ed5f4b9aa82f78079ea5377 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Apr 8 17:43:54 2022 +0200 misc: remove pickle auto tests from admin_pages (#65241)
Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: remove pickle auto tests from admin_pages (#65241)