Projet

Général

Profil

Development #67020

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

Ajouté par A. Berriot 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:
Oui
Planning:
Non

Description

Même principe que https://dev.entrouvert.org/issues/66996

En local, avec Avec PASSWORD_HASHERS = ["django.contrib.auth.hashers.MD5PasswordHasher"] dans tests/settings.py, on passe de 82 à 55s de temps d'exécution des tests.


Fichiers


Demandes liées

Copié depuis Chrono - Development #66996: Améliorer la rapidité d'exécution des tests en changeant PASSWORD_HASHERSFermé05 juillet 2022

Actions

Révisions associées

Révision 0d516c0b (diff)
Ajouté par A. Berriot il y a presque 2 ans

ci: reduce test execution time by changing password hasher (#67020)

Historique

#1

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

  • Copié depuis Development #66996: Améliorer la rapidité d'exécution des tests en changeant PASSWORD_HASHERS ajouté
#2

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

  • Fichier 0001-ci-reduce-test-execution-time-by-changing-password-h.patch supprimé
#4

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

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

Go. (si jamais il y avait quelque chose à dire ce serait qu'on a une petite préférence pour les single quotes)

#5

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

  • Assigné à changé de A. Berriot à Benjamin Dauvergne

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

Go. (si jamais il y avait quelque chose à dire ce serait qu'on a une petite préférence pour les single quotes)

Merci pour la relecture ! Pour les quotes, il me semblait que black remplaçait de toutes façons les single par des doubles ? (https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#strings)

#6

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

  • Assigné à changé de Benjamin Dauvergne à A. Berriot
#7

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit 0d516c0bd6745c03c77496e64cb49a7fe9af668e
Author: Agate <aberriot@entrouvert.com>
Date:   Tue Jul 5 14:02:43 2022 +0200

    ci: reduce test execution time by changing password hasher (#67020)
#8

Mis à jour par Benjamin Dauvergne il y a presque 2 ans

Agate Berriot a écrit :

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

Go. (si jamais il y avait quelque chose à dire ce serait qu'on a une petite préférence pour les single quotes)

Merci pour la relecture ! Pour les quotes, il me semblait que black remplaçait de toutes façons les single par des doubles ? (https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#strings)

On a désactivé ça je crois (je ne sais plus pourquoi).

#9

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

Benjamin Dauvergne a écrit :

Agate Berriot a écrit :

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

Go. (si jamais il y avait quelque chose à dire ce serait qu'on a une petite préférence pour les single quotes)

Merci pour la relecture ! Pour les quotes, il me semblait que black remplaçait de toutes façons les single par des doubles ? (https://black.readthedocs.io/en/stable/the_black_code_style/current_style.html#strings)

On a désactivé ça je crois (je ne sais plus pourquoi).

okay, j'imagine que c'est dans la conf des hooks de pre commit, mais ces hooks la ne fonctionnent pas encore chez moi (pas eu le temps de débugguer), je vais m'en occuper.

#10

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

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

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

Automatic expiration

Formats disponibles : Atom PDF