[weboob] Fwd: Re: [M2 Lille1][browser2] tests de Suzanne

Flo weboob at flo.fourcot.fr
Tue Oct 21 08:57:28 CEST 2014




-------- Message transféré --------
Sujet : Re: [weboob] [M2 Lille1][browser2] tests de Suzanne
Date : Mon, 20 Oct 2014 17:47:50 +0200
De : Pierre-Philippe Berenguer <pp.berenguer at etudiant.univ-lille1.fr>
Pour : Flo <weboob at flo.fourcot.fr>
Copie à : Omar Chahbouni <omar.chahbouni1 at gmail.com>, Maxime Chaste
<maxime.chaste at gmail.com>

Bonjour,

ci-joint les tests de browser2.

*weboob-0.j/weboob/tools/test.py*
On les lance depuis *weboob-0/tools/run_tests.sh*

Cordialement,

Pierre-Philippe Berenguer



Le 18 octobre 2014 20:25, Flo <weboob at flo.fourcot.fr> a écrit :

> Bonjour,
>
> sur le fond, il y a pas mal de choses bien. Cependant ce patch est pour
> le moment absolument pas intégrable.
>
> Quelques remarques donc :
>
> >       def test_raise_for_status(self):
>
> Les tabulations sont utilisées presque partout. Il faudrait remplacer
> chaque tabulation par quatre espaces.
>
> >               domB.RESTRICT_URL =True
>
> Mettre une espace avant le True.
>
> >               #si on met une mauvaise BASEURL ca renvoie false
>
> Les commentaires doivent être en anglais, partout.
>
> >               domB.RESTRICT_URL=["https::/faulsURL","
> https://api.github.com/"]
>
> Mettre une espace avant et après le "=".
> Je conseille de faire le tour du fichier, il y en a plein comme ça (avec
> une seule espace après, etc).
>
> >       # WARNING : RUN WITH INTERNET !
>
> Non nécessaire, la plupart de nos tests (tous les modules...)
> nécessitent Internet.
>
> >               retour_voulu = """%s%s%s%s""" % (str1, str2, str3, str4)
>
> Nom de variable en français. Je pense pas que les triples quotes soient
> nécessaires.
>
> > class TestCase(TestCase):
>
> Un nom un peu plus expressif ne ferait pas de mal :-)
>



-------------- next part --------------
A non-text attachment was scrubbed...
Name: test.py
Type: text/x-python
Size: 12805 bytes
Desc: not available
URL: <https://lists.symlink.me/pipermail/weboob/attachments/20141021/98aeff78/attachment.py>


More information about the weboob mailing list