Hello Javier, Although your comment is valuable, my opinion is that we should be focusing on adding coverage to mago, rather than keeping adding stuff to the framework itself. Do you see a real benefit that would worth the effort? Cheers, Ara. On 22/09/09 11:19, Javier Collado wrote: Hello, While looking at a merge proposal, I saw a pattern that was repeated multiple times: if <condition>: raise AssertionError, <comment> I understand that the use of "assert" statement in a testing library is discouraged since it isn't executed when optimizations are activated so the if/raise lines here are completely fine. However, I was thinking that as many testing libraries provide their own assertion methods, perhaps mago TestSuite base class should do it also. Do you agree on this? Do you think is there any other behaviour that would be worth to add? Best regards, Javier_______________________________________________ http://live.gnome.org/DesktopTesting desktop-testing-list mailing list desktop-testing-list gnome org http://mail.gnome.org/mailman/listinfo/desktop-testing-list |