On Nov 10, 11:45 am, pang <pablo.ang...@uam.es> wrote: > I searched for "wikiinteract" on trac, and came up with patches with > no testing at all: 9737, 9729, 9728 and 9623. I couldn't find an > example that is nicely tested.
On second thought, these are not trivial to split and test. Roughly every line consists of one call to a function that is tested otherwise, and just prints or shows some output. All the lines are pretty much independent and serve for different purposes, so maybe testing is not necessary/useful/.... Also, many people will use these interacts for inspiration, using sage.interact.xxx??, and splitting a simple interact into unrecognizable format would make them useless in that sense. It would be good to keep simple interacts simple. So if those interacts are included into the library as they are, that means they get once-in-a-lifetime peer review, and some testing for compatibility in future versions (if they throw an exception, it will be caught). Ok, that is fine. I'll review them, and proceed to include others, unless anyone has any other thought. -- To post to this group, send an email to sage-devel@googlegroups.com To unsubscribe from this group, send an email to sage-devel+unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org
