Outsource PHPUnit_Selenium XML definitions #528

Closed
giorgiosironi opened this Issue Mar 23, 2012 · 4 comments

Projects

None yet

3 participants

@giorgiosironi
Contributor

How can we bring this:
https://github.com/sebastianbergmann/phpunit/blob/3.6/PHPUnit/TextUI/Command.php#L663
into PHPUnit_Selenium? It would be more easy to extend as also Selenium2TestCase will need configuration.

@edorian
Collaborator
edorian commented Mar 27, 2012

As it is part of the phpunit xml configuration file that could prove a little difficult as we'd need a way for extensions to define their own config values or allow access to the (preferably already parsed and merged) xml configuration.

I need to look at the code a little more but it should be doable and the time seems right :)

@edorian
Collaborator
edorian commented May 15, 2012

The browser configuration is only needed when phpunit-selenium is there anyways. Would you be fine with having a PHPUnit_Extensions_SeleniumTestCase::getSeleniumBrowserConfiguration($configuration) ? Or have the method on some oether selenium class.

Having a generic way for plugins to do config stuff would be nice but since this needs to be B/C anyways I could go for a hack here?

@giorgiosironi
Contributor

Yes, getSeleniumBrowserConfiguration() could be moved, but the scope of the methods it uses is not accessible from PHPUnit_Selenium classes (e.g. getInteger() is protected).

@sebastianbergmann

Closing due to #1686.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment