Re: run-tests vs phpruntests

On 05/15/2012 08:23 AM, zoe slattery wrote:

Does anyone have any strong views on how 'forgiving' run-tests.php should be? For example, this test: ext/libxml/tests/bug61367-read.phpt has this --INI-- section: --INI-- open_basedir=. ; Suppress spurious "Trying to get property of non-object" notices error_reporting=E_ALL & ~E_NOTICE run-tests.php just ignores any line that is not of the form 'something=somethingelse', in the parallel run-tests.php code we currently throw an exception in an INI setting is not of the form 'something=somethingelse', so this test won't run because of the 'comment' line. The behaviour should be consistent which can be achieved either by modifying tests or making the new code do what run-tests.php currently does. I have a slight preference for not allowing anything in an INI section except an ini setting, this would mean changing these two tests. Anyone have strong views either way?

I'd prefer allowing comments, but if it affects only 2 out 12,000+ tests I wouldn't complain if it were changed. Silently ignoring non well-formed assignments doesn't really count as allowing comments. Chris -- christopher.jones@oracle.com http://twitter.com/#!/ghrd