Project

General

Profile

Development #65241

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

Added by Lauréline Guerin 16 days ago. Updated 12 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
13 May 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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


Files

Associated revisions

Revision c65d516a (diff)
Added by Lauréline Guérin 12 days ago

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

History

#1

Updated by Lauréline Guerin 16 days ago

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

Updated by Frédéric Péters 12 days ago

  • Status changed from Solution proposée to Solution validée
#3

Updated by Lauréline Guerin 12 days ago

  • Status changed from Solution validée to 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

Updated by Transition automatique 12 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF