Projet

Général

Profil

Development #65241

tests admin_pages - ne pas tester en mode pickle de façon systématique

Ajouté par Lauréline Guérin il y a presque 2 ans. Mis à jour il y a presque 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
13 mai 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

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

Révision c65d516a (diff)
Ajouté par Lauréline Guérin il y a presque 2 ans

misc: remove pickle auto tests from admin_pages (#65241)

Historique

#1

Mis à jour par Lauréline Guérin il y a presque 2 ans

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/)

#2

Mis à jour par Frédéric Péters il y a presque 2 ans

  • Statut changé de Solution proposée à Solution validée
#3

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)
#4

Mis à jour par Transition automatique il y a presque 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#5

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF