Projet

Général

Profil

Development #67029

Améliorer la rapidité d'exécution des tests en changeant PASSWORD_HASHERS

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

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

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

Même principe que #66996 et #67020, merci Agate.

En local, avec PASSWORD_HASHERS = ['django.contrib.auth.hashers.MD5PasswordHasher'] dans tests/settings.py, je passe pour tests/form_pages/test_all.py de 465.74s à 291.97s.

Révisions associées

Révision 16529e20 (diff)
Ajouté par Frédéric Péters il y a presque 2 ans

tests: run with simplest password hasher to speed up execution (#67029)

Historique

#1

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

  • Statut changé de Nouveau à Solution validée

Je venais de demander à Agate si elle voulait bien le faire :)

#2

Mis à jour par A. Berriot il y a presque 2 ans

  • Assigné à changé de Frédéric Péters à Benjamin Dauvergne

Lauréline Guerin a écrit :

Je venais de demander à Agate si elle voulait bien le faire :)

Du coup c'est bon je laisse ma place :D

#3

Mis à jour par A. Berriot il y a presque 2 ans

  • Assigné à changé de Benjamin Dauvergne à Frédéric Péters
#4

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

Sur jenkins le job de la branche a terminé en 41min 39s, vs 43min 54s pour le dernier job sur la branche principale, en fait tout le bénéfice était concentré sur le fichier testé en local, sur les autres on ne recrée pas de compte à chaque test :/

#5

Mis à jour par A. Berriot il y a presque 2 ans

Frédéric Péters a écrit :

Sur jenkins le job de la branche a terminé en 41min 39s, vs 43min 54s pour le dernier job sur la branche principale, en fait tout le bénéfice était concentré sur le fichier testé en local, sur les autres on ne recrée pas de compte à chaque test :/

Le gain a l'air moindre effectivement. Après de ce que je vois, pour la partie test pure, on passe de

========== 3345 passed, 74 skipped, 15 warnings in 2350.14s (0:39:10) ==========

Sur la branche main, à

========== 3345 passed, 74 skipped, 15 warnings in 2125.19s (0:35:25) ==========

Sur la branche que tu as créée, ce ui ifait quand même presque 10% de réduction, c'est pas mal

(je me base pas sur la durée de pipeline globale, il y 'a trop d'autres opérations effectuées)

#6

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

  • Statut changé de Solution validée à Résolu (à déployer)

(je m'étais mis à rêver d'un gain supérieur)

commit 16529e200030468f561237825fb3a2531bb601b6
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jul 5 14:31:26 2022 +0200

    tests: run with simplest password hasher to speed up execution (#67029)
#7

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

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

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF