Produits Entr'ouvert » Mandaye

Sur 0002: tu ne veux pas retourner un JSON car c'est attendu par la cellule Combo derrière ?

Je me dis que vu c'est une API à nous il faut uniformiser le format des réponses: {"data": ..., "err": ..., "err_class": ... }

Note générale : si tout ça modifie l'API, il ne faut bien sûr pas oublier de modifier le template blurp associé.

response = session.post(login_url, data=login_info, verify=False), ainsi qu'un autre plus loin; je viens de vérifier et https://mediatheques-test-entrouvert.montpellier3m.fr/ et https://mediatheques.montpellier3m.fr/ répondent sans problème de certificat; évitons le verify=False.

En appliquant le 0001 j'ai:

 git apply 0001-don-t-use-PhantomJS-when-using-archimed-ws-14967.patch
0001-don-t-use-PhantomJS-when-using-archimed-ws-14967.patch:659: trailing whitespace.
    "d": "Anonymous", 
0001-don-t-use-PhantomJS-when-using-archimed-ws-14967.patch:663: trailing whitespace.
                "__type": "ObjectCollection:#Archimed.Serialization", 
0001-don-t-use-PhantomJS-when-using-archimed-ws-14967.patch:664: trailing whitespace.
                "badPasswordCount": 0, 
0001-don-t-use-PhantomJS-when-using-archimed-ws-14967.patch:665: trailing whitespace.
                "checkAdditionalsResult": true, 
0001-don-t-use-PhantomJS-when-using-archimed-ws-14967.patch:667: trailing whitespace.
            }, 
warning: squelched 7 whitespace errors
warning: 12 lines add whitespace errors.

Certes, mais ça vient des fichiers de test .json, pas du code.

Ça n'empêche pas que, de manière générale, c'est utile/nécessaire d'avoir [color] diff = auto dans son ~/.gitconfig, pour repérer les espaces pourris à l'avance.

Ok pour 0001 et 0002, mais pour 0003 en suivant les tests je dirais qu'il faut chercher les attributs dans response.json() et non response.data.

Vraiment, verify=False, faut pas.

Dans les tests du 0002 je mettrais aussi des json.loads(response.content) pour plus de clarté.

Ack