Looks fine. Not sure why the nightly builds have been passing, though, given that they're not set up to use Selenium.
Fix the selenium test skip exception when the Selenium RC is not installed.
Review Request #1598 — Created May 18, 2010 and submitted
Fix the selenium test skip exception when the Selenium RC is not installed. When the Selenium RC isn't installed, the first time SeleniumUnitTest.setUp is called it will raise SkipTest, but subsequent calls wouldn't go through this code path because cls.selenium was still set to non-None. Any other calls to cls.selenium would raise an HTTP connection error, but that got reported as an error instead of properly skipping the test.
Ran unit tests and saw SKIP instead of ERROR.