Re: new run-tests

From: Date: Sun, 05 Jul 2009 09:00:37 +0000
Subject: Re: new run-tests
References: 1  Groups: php.qa 
Request: Send a blank email to php-qa+get-65185@lists.php.net to get a copy of this message
Greg Beaver wrote:
Hi, I'm looking at using the new run-tests implementation inside Pyrus, and am a bit concerned at some of the assumptions. It seems that all of the configuration code assumes that it is running from an application where it has absolute control over $argv and friends. A much better design would be to separate these dependencies, write some classes whose sole purpose is parsing $argv to set up the configuration, and let other programs like Pyrus do their own configuration setup and pass that in. As it is designed now, I have to extend rtWinConfiguration and rtUnixConfiguration just to stop it from trying to parse $argv. Hi Greg - I think you are probably right, but I our primary focus was (is) on writing something that met the requirements (http://wiki.php.net/qa/runtests/requirements), so I see your requirement as additional to those. In fact - I don't think it's _that_ hard to do, but it does need a little thought. I'm happy to add it as an additional requirement, but I think we have to address it after we have the basic stuff done.
Stefan - thoughts? Z

« previous php.qa (#65185) next »