On Fri, Apr 29, 2011 at 06:15:10AM +0000, Vincent Delecroix wrote:
> That's nice to have more tests. For the current problem, not all
> cardinalities (of subsets) are actually Integer... I will push a new
> version of the patch in few times (and write that it depends on 10534
> on trac). I also add some TestSuite to the "degenrate guys":
> Subsets([],0), Subsets([1,2],8), ... to see (and it works).
Thanks!
> On the other hand, are there any convention for the place to put the
> TestSuite tests? I remember that test like "loads(dums(O)) == O" are
> better suited to be in the __init__ (because hidden to the user). What
> do you think about "TestSuite(O)" ?
I am myself keeping the same habit, and putting the TestSuite(O).run()
in the __init__ method (when there is one).
Cheers,
Nicolas
--
Nicolas M. Thiéry "Isil" <[email protected]>
http://Nicolas.Thiery.name/
--
You received this message because you are subscribed to the Google Groups
"sage-combinat-devel" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to
[email protected].
For more options, visit this group at
http://groups.google.com/group/sage-combinat-devel?hl=en.
