From: Robin B. <ro...@kn...> - 2001-10-23 13:48:32
|
On Tuesday 23 October 2001 00:02, Bjoern Hoehrmann wrote: > * Robin Berjon wrote: > >On Monday 22 October 2001 05:18, Bjoern Hoehrmann wrote: > >> What if Text::Balanced 1.99 re-introduces this bug? :-) > >> Having a test case, we can notice it. > > > >Well... it would be a bit over the border to test other modules in our own > >distribution. > > We test only CSS::SAC which has to parse correctly, no matter what it > uses to do parsing. If other modules fail, CSS::SAC also fails. I just > don't want to get a 100% success for the tests if Text::Balanced is > outdated. Then don't worry too much, it was one of the test cases that triggered the T::B bug. Normally, CSS::SAC should refuse to install (or at least warn violently) for outdated T::Bs. Unless of course you copy it by hand to its install dir, in which case you're obviously asking for trouble. That's not to say that we don't want more test cases. Ideally, we'd test everything, and every fixed bug would get a test case to check for regression problems. It's just that tests are painful to write in normal situations, and even harder for event driven code. Thankfully, Barrie Slaymaker recently sent me a diff module that should help lots with that. -- _______________________________________________________________________ Robin Berjon <ro...@kn...> -- CTO k n o w s c a p e : // venture knowledge agency www.knowscape.com ----------------------------------------------------------------------- Make it idiot proof and someone will make a better idiot. |