https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342020-01-17T18:37:14ZRedmine Entr’ouvertCombo - Development #39083: tests, retirer le bout wtm._patch_settings devenu inutilehttps://dev.entrouvert.org/issues/39083?journal_id=2104972020-01-17T18:37:14ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/40527">0001-tests-remove-unnecessary-request-finalizer-for-setti.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/40527/0001-tests-remove-unnecessary-request-finalizer-for-setti.patch">0001-tests-remove-unnecessary-request-finalizer-for-setti.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> Combo - Development #39083: tests, retirer le bout wtm._patch_settings devenu inutilehttps://dev.entrouvert.org/issues/39083?journal_id=2105002020-01-17T23:35:24ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> Combo - Development #39083: tests, retirer le bout wtm._patch_settings devenu inutilehttps://dev.entrouvert.org/issues/39083?journal_id=2105152020-01-18T14:47:09ZBenjamin Dauvergne
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>En cours</i></li></ul><p>Dans la mesure ou le plugin pytest présent dans django-webtest fait toujours le patch/unpatch je ne vois pas pourquoi le virer (par contre on peut très bien utiliser directement django_app dans app).</p>
<p><a class="external" href="https://github.com/django-webtest/django-webtest/blob/master/django_webtest/pytest_plugin.py#L25">https://github.com/django-webtest/django-webtest/blob/master/django_webtest/pytest_plugin.py#L25</a></p> Combo - Development #39083: tests, retirer le bout wtm._patch_settings devenu inutilehttps://dev.entrouvert.org/issues/39083?journal_id=2105162020-01-18T14:53:13ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Je ne comprends pas ton commentaire; si le boulot de patch/unpatch est déjà assuré, pourquoi continuer à le refaire ici ?</p> Combo - Development #39083: tests, retirer le bout wtm._patch_settings devenu inutilehttps://dev.entrouvert.org/issues/39083?journal_id=2105182020-01-18T15:01:30ZBenjamin Dauvergne
<ul></ul><p>Frédéric Péters a écrit :</p>
<blockquote>
<p>Je ne comprends pas ton commentaire; si le boulot de patch/unpatch est déjà assuré, pourquoi continuer à le refaire ici ?</p>
</blockquote>
<p>Ben il n'est pas assuré par django_webtest.DjangoTestApp() qui est la seule chose que la fixture app appelle donc non. Voir <a class="external" href="https://github.com/django-webtest/django-webtest/blob/master/django_webtest/__init__.py">https://github.com/django-webtest/django-webtest/blob/master/django_webtest/__init__.py</a> c'est WebTestMixin qui fait ça dans <code>__call__()</code> pas DjangoTestApp (WebTestMixin c'est une mixin pour TestCase, voir plus loin l'instanciation dans la classe WebTest).</p>
<p>Je disais donc qu'il faudrait plutôt le code suivant :</p>
<pre>
@pytest.fixture
def app(request, clean_media, django_app):
return django_app
</pre> Combo - Development #39083: tests, retirer le bout wtm._patch_settings devenu inutilehttps://dev.entrouvert.org/issues/39083?journal_id=3577012022-02-14T19:36:47ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Rejeté</i></li></ul>