Projet

Général

Profil

Bug #24191

échec de test_notify_remote_items quand mellon est présent

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
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

Révision 2948c26c (diff)
Ajouté par Serghei Mihai il y a presque 6 ans

tests: improve remote invoices notification code coverage (#24191)

Révision 644423da (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

tox: add missing getlasso.sh (#24191)

Historique

#1

Mis à jour par Serghei Mihai il y a presque 6 ans

  • Assigné à mis à Serghei Mihai

Je prends

#3

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)
#4

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)
#5

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

Formats disponibles : Atom PDF