În data de Mi, 24-02-2010 la 18:27 +0000, Henning Eggers a scris:
> Review: Approve code
> > Conflict solved.
>
> Cool, looks good. ;)
>
>
> > Are you saying that we should not write unit tests using doctest format?
>
> Yes, at least that's what I do. They execute slower, don't they? But
> as you pointed out, the test was already there, so you just extended
> it. That's what I do, too ... ;) So you can keep it as it is if you
> want to.
OK. This is not the only unit test written in doctest format.
În data de Mi, 24-02-2010 la 18:27 +0000, Henning Eggers a scris:
> Review: Approve code
> > Conflict solved.
>
> Cool, looks good. ;)
>
>
> > Are you saying that we should not write unit tests using doctest format?
>
> Yes, at least that's what I do. They execute slower, don't they? But
> as you pointed out, the test was already there, so you just extended
> it. That's what I do, too ... ;) So you can keep it as it is if you
> want to.
OK. This is not the only unit test written in doctest format.
$ ls lib/lp/ translations/ browser/ tests/* .txt translations/ browser/ tests/distroser ies-views. txt translations/ browser/ tests/language- views.txt translations/ browser/ tests/poexport- request- views.txt translations/ browser/ tests/pofile- base-views. txt translations/ browser/ tests/pofile- views.txt translations/ browser/ tests/potemplat e-views. txt translations/ browser/ tests/productse ries-views. txt translations/ browser/ tests/translati onimportqueue- views.txt translations/ browser/ tests/translati onmessage- views.txt translations/ browser/ tests/translato r-views. txt
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
Maybe we should open a bug to convert all those test from doctest to
"pure" python code.
> Thanks for the extra comments in the code, btw. And all the clean-up
> work!
> You have a double "allowed allowed" in there somewhere, btw.
Typo fixed and pushed.
When you have time, can you please send this branch to ec2 test?
Kindest regards,
--
Adi Roiban