Bug #24191
échec de test_notify_remote_items quand mellon est présent
Début:
31 mai 2018
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
Dans test_notify_remote_items il y a ce bout de code :
regie.notify_new_remote_invoices() assert mock_get.call_count == 0 regie.webservice_url = 'http://example.org/regie' # is_remote regie.save() regie.notify_new_remote_invoices() mock_get.assert_not_called()
Et les deux vérifications sur le non-appel à mock_get vont échouer si jamais mellon est présent, parce que :
- pour la première, qui je suppose teste qu'il ne se passe rien quand regie.webservice_url n'est pas positionné. (mais comme l'objet créé par une fixture est réutilisé, il peut avoir gagné dans un des tests précédents un webservice_url
- pour la seconde, je l'imagine passer par :
def notify_new_remote_invoices(self): if UserSAMLIdentifier is None: # remote invoices retrieval requires SAML return
et si jamais mellon est présent, UserSAMLIdentifier sera défini, et ça ne passera donc pas.
À faire, sans doute ajouter django-mellon au tox.ini, et ajuster le test.
Fichiers
Révisions associées
tox: add missing getlasso.sh (#24191)
Historique
Mis à jour par Serghei Mihai il y a presque 6 ans
- Fichier 0001-tests-improve-remote-invoices-notification-code-cove.patch 0001-tests-improve-remote-invoices-notification-code-cove.patch ajouté
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a presque 6 ans
- Statut changé de Nouveau à Résolu (à déployer)
Rebasé et poussé.
commit 2948c26c2fc11d8455133aaaad5a4ccd12e8f02e Author: Serghei Mihai <smihai@entrouvert.com> Date: Thu May 31 17:24:18 2018 +0200 tests: improve remote invoices notification code coverage (#24191)
Mis à jour par Frédéric Péters il y a presque 6 ans
Et manquait le script dans le patch :
commit 644423da929d904785d534b02c5d3619b923f078 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Jun 8 15:49:01 2018 +0200 tox: add missing getlasso.sh (#24191)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
tests: improve remote invoices notification code coverage (#24191)