Project

General

Profile

Development #67933

Simplifier le nom des bases en test

Added by Agate Berriot 13 days ago. Updated 8 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
04 August 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Cf https://dev.entrouvert.org/issues/67911#note-13

On avait du code dans les settings de test qui incluait le nom de la branche dans la base de test, or pytest-django a également un mécanisme qui inclut le nom de l'environnement tox quand les tests sont lancés en parallèle.

Par conséquent, on peut simplifier l'existant.


Files


Related issues

Related to Hobo - Development #67911: paralléliser les tests via tox -pSolution déployée03 August 2022

Actions

Associated revisions

Revision 5e5125c9 (diff)
Added by Agate Berriot 9 days ago

tox: fixed inconsistent / obscure db names during tests (#67933)

History

#1

Updated by Agate Berriot 13 days ago

#2

Updated by Agate Berriot 13 days ago

#4

Updated by Benjamin Dauvergne → en congés jusqu'au 29/08 13 days ago

Alors oui, mais je ne vois pas où on tronque après que pytest-django ait rajouté son bordel.

#5

Updated by Benjamin Dauvergne → en congés jusqu'au 29/08 13 days ago

  • Status changed from Solution proposée to Information nécessaire
#6

Updated by Agate Berriot 13 days ago

Benjamin Dauvergne a écrit :

Alors oui, mais je ne vois pas où on tronque après que pytest-django ait rajouté son bordel.

On ne tronque pas, on fait l'inverse : on ajoute pas le tox_env_name dans le nom de la base quand on est en parallèle, et on laisse pytest-django le rajouter après (c'est fait après le chargement des settings, au moment ou la fixture db est utilisée)

#7

Updated by Agate Berriot 13 days ago

  • Assignee changed from Agate Berriot to Benjamin Dauvergne → en congés jusqu'au 29/08
#8

Updated by Benjamin Dauvergne → en congés jusqu'au 29/08 13 days ago

Mais les noms des bases vont être trop long avec le nom de la branche puis le nom de l'env, on est d'accord ?

PS: pour dire que je ne vois pas dans le code de pytest-django d'endroit où il tronquerait donc on fait un pas un avant et un pas en arrière.

#9

Updated by Benjamin Dauvergne → en congés jusqu'au 29/08 12 days ago

  • Status changed from Information nécessaire to Solution validée

J'ai l'impression qu'on se comprend pas :) Le nouveau code tronque le nom de la branche à 15 caractères puis ajoute le TOX_ENV_NAME ou si TOX_PARALLEL_ENV existe s'attend à l'ajout du suffixe TOX_PARALLEL_ENV plus éventuellement celui d'xdist. Tu sembles certaine que dans tous les cas on va rentrer dans le budget des 63 caractères (et pas 68 comme c'est écrit dans le code), et ça semble le cas 15 + 1 caractère de jointure + max(len(tox_env))=33 + 1 + len(gw0) = 53. Mais on perd 10 caractères significatifs au passage, ça aurait été plus simple de tronquer dans db_modify_settings avec un hash sur 2 caractères que de faire ces calculs d'apothicaire.

Bon je valide parce que ça marche, mais ça reste trop sioux pour moi et éloigné des pratiques conseillées dans la doc de pytest-django.

#10

Updated by Benjamin Dauvergne → en congés jusqu'au 29/08 12 days ago

  • Assignee changed from Benjamin Dauvergne → en congés jusqu'au 29/08 to Agate Berriot
#11

Updated by Agate Berriot 9 days ago

  • Status changed from Solution validée to Résolu (à déployer)
commit b151e8babd249526c1a3440cb0d08f08292960a2
Author: Agate Berriot <aberriot@entrouvert.com>
Date:   Thu Aug 4 09:31:48 2022 +0200

    tox: fixed inconsistent / obscure db names during tests (#67933)
#12

Updated by Transition automatique 8 days ago

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

Also available in: Atom PDF