Produits Entr'ouvert » w.c.s.

→ test.

Mais plutôt virer la partie apikey et assurer la signature des appels (en utilisant call_webservice de wcs/wscalls.py).

print payload

Avertissement aux réels relecteurs : ne pas laisser passer ça.

Je pense que ce sera toujours vrai dans la mesure ou SMSMocking hérite de PasserelleSMS.

        if isinstance(self, qommon.sms.PasserelleSMS):

À mon avis tu devrais plutôt faire deux classes SMSMocking et renvoyer celle pour passerelle quand mode == 'passerelle' (idéalement pas de mocking du tout puisque tu veux tester l'appel HTTP lui même, mais pour cela il faudrait pouvoir débrayer SMSMocking en en faisant une vrai fixture avec un setUp() et un clean()).

Benjamin Dauvergne a écrit :

Je pense que ce sera toujours vrai dans la mesure ou SMSMocking hérite de PasserelleSMS.

Yep, le test sera toujours vrai. Je le vire.

À mon avis tu devrais plutôt faire deux classes SMSMocking et renvoyer celle pour passerelle quand mode == 'passerelle' (idéalement pas de mocking du tout puisque tu veux tester l'appel HTTP lui même, mais pour cela il faudrait pouvoir débrayer SMSMocking en en faisant une vrai fixture avec un setUp() et un clean()).

Là, ça veut dire que je dois modifier l'autre test et cela sort du contexte de ce ticket.

Ack (si tu peux vérifier que tous les tests passent avec les deux patchs avant, merci, et ajouter le une référence à ce ticket sur le patch à SMSMocking).

(et ne pas pousser ça avant lundi)

pytest n'est pas content:

___________________________________________________________________________________________________________________________ ERROR at setup of test_sms ___________________________________________________________________________________________________________________________
pytest.fixture functions cannot use ``yield``. Instead write and return an inner function/generator and let the consumer call and iterate over it.:

    @pytest.fixture
    def sms_mocking():
        with SMSMocking() as sms:
            yield sms
/home/serghei/dev/wcs/tests/conftest.py:55

Tu es en quelle version?

La 3 comme tout le monde.

Yep, avec la 3.0.7 ça passe.
Du coup on retire la version dans le projet jenkins ?

w.c.s. devrait utiliser un jenkins.sh comme tout le monde et le jenkins.sh devrait appeler tox, comme tout le monde.

w.c.s. devrait utiliser un jenkins.sh comme tout le monde et le jenkins.sh devrait appeler tox, comme tout le monde.

Avec hobo et fargo qui éclatent à nouveau aujourd'hui sans autre raison que cette configuration, je ne suis pas encore convaincu.

w.c.s. devrait utiliser un jenkins.sh comme tout le monde et le jenkins.sh devrait appeler tox, comme tout le monde.

Créé #15562 pour suivre ce sujet.

Avec hobo et fargo qui éclatent à nouveau aujourd'hui sans autre raison que cette configuration, je ne suis pas encore convaincu.

Et django-mellon, et tous les autres projets qui vont être rebuildés dans les heures à venir.

Tu penses vraiment qu'il y a un rapport avec tox ou bien c'est juste du fud ? Parce que la plupart des projets build avec tox et on a pas vu les mêmes soucis (passerelle par exemple, qui s'est buildé ente hobo et django-mellon).

[hors sujet pour le ticket, je réponds dans #15562]