https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342022-08-04T07:20:47ZRedmine Entr’ouvertHobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3987362022-08-04T07:20:47ZA. Berriot
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/67911">Development #67911</a>: paralléliser les tests via tox -p</i> ajouté</li></ul> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3987372022-08-04T07:31:59ZA. Berriot
<ul><li><strong>Fichier</strong> <a href="/attachments/68325">0001-tox-fixed-inconsistent-obscure-db-names-during-tests.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/68325/0001-tox-fixed-inconsistent-obscure-db-names-during-tests.patch">0001-tox-fixed-inconsistent-obscure-db-names-during-tests.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3987382022-08-04T07:38:39ZA. Berriot
<ul><li><strong>Fichier</strong> <a href="/attachments/68326">0001-tox-fixed-inconsistent-obscure-db-names-during-tests.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/68326/0001-tox-fixed-inconsistent-obscure-db-names-during-tests.patch">0001-tox-fixed-inconsistent-obscure-db-names-during-tests.patch</a> ajouté</li></ul> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3987632022-08-04T08:46:05ZBenjamin Dauvergne
<ul></ul><p>Alors oui, mais je ne vois pas où on tronque après que pytest-django ait rajouté son bordel.</p> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3987642022-08-04T08:46:12ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Information nécessaire</i></li></ul> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3987842022-08-04T09:54:18ZA. Berriot
<ul></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>Alors oui, mais je ne vois pas où on tronque après que pytest-django ait rajouté son bordel.</p>
</blockquote>
<p>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)</p> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3988132022-08-04T11:18:34ZA. Berriot
<ul><li><strong>Assigné à</strong> changé de <i>A. Berriot</i> à <i>Benjamin Dauvergne</i></li></ul> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3988972022-08-04T15:51:00ZBenjamin Dauvergne
<ul></ul><p>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 ?</p>
<p>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.</p> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3989482022-08-05T07:08:58ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Information nécessaire</i> à <i>Solution validée</i></li></ul><p>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.</p>
<p>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.</p> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3989532022-08-05T07:22:01ZBenjamin Dauvergne
<ul><li><strong>Assigné à</strong> changé de <i>Benjamin Dauvergne</i> à <i>A. Berriot</i></li></ul> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3993032022-08-08T07:43:09ZA. Berriot
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>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)</pre> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=3995442022-08-09T08:14:30ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> Hobo - Development #67933: Simplifier le nom des bases en testhttps://dev.entrouvert.org/issues/67933?journal_id=4130202022-10-09T02:42:01ZTransition automatique
<ul></ul><p>Automatic expiration</p>