Produits Entr'ouvert » w.c.s.

On ne l'aurait pas vu parce que le FakeResponse dans tests/utilities.py n'a pas les attributs d'une réponse quixote (status à la place de status_code, reason à la place de reason_phrase, data à la place de body, etc) ?

POC (je continuerai ce soir éventuellement)

diff --git a/tests/utilities.py b/tests/utilities.py
index db3e859c..d88451f8 100644
--- a/tests/utilities.py
+++ b/tests/utilities.py
@@ -290,8 +290,8 @@ class HttpRequestsMocking(object):

         class FakeResponse(object):
             def __init__(self, status, data, headers):
-                self.status = status
-                self.reason = 'whatever'
+                self.status_code = status
+                self.reason_phrase = 'whatever'
                 self.data = data
                 self.headers = headers or {}
                 self.length = len(data or '')

Crash du test :

wcs/wf/wscall.py:317: in perform
    self.action_on_error(self.action_on_app_error, formdata, response, data=data)
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

self = <wcs.wf.wscall.WebserviceCallStatusItem object at 0x7f366491d250>, action = ':stop', formdata = <Baz 'baz #1-1' id:1>
response = <utilities.FakeResponse object at 0x7f366491d110>, data = '{"data": "", "err": 1}', exc_info = None

    def action_on_error(self, action, formdata, response=None, data=None, exc_info=None):
        if action in (':pass', ':stop') and (self.notify_on_errors or self.record_errors):
            if exc_info:
                summary = traceback.format_exception_only(exc_info[0], exc_info[1])[-1]
            else:
                summary = ''
                if response:
>                   summary = '%s %s' % (response.status, response.reason)
E                   AttributeError: 'FakeResponse' object has no attribute 'status'

wcs/wf/wscall.py:367: AttributeError

Ce serait vraiment difficile de refaire quelque chose à base de httmock ? J'avoue je n'ai pas bien regardé les différences.

De requests, j'obtiens les attributs status_code (bien) mais aussi reason (et pas reason_phrase). (tu écris "réponse quixote" mais à cet endroit c'est une réponse de python-requests).

_~~

Sans doute qu'on pourrait revoir le principe des tests

Tout à fait.

Ok avec ça.

Ma remarque c'était surtout pour ne pas se retrouver avec une autre différence qui laisse passer des bugs à travers la requête mais ok aussi pour la correction actuelle.