don't call $sel->start, it breaks things
authorJonathan Rockway <jon@jrock.us>
Thu, 6 Mar 2008 12:29:55 +0000 (06:29 -0600)
committerJonathan Rockway <jon@jrock.us>
Thu, 6 Mar 2008 12:30:22 +0000 (06:30 -0600)
commit38cd3a5c41a54640e7e1541fefb30e6b57c7f735
tree61de227673e854ea66798d1ad76dfa012a3802ea
parent6efea5d1c40bdea3e817a3929fe6ede7a07ddcbd
don't call $sel->start, it breaks things

t/01live.t fails with an error message telling that it couldn't start the Selenium
server for an unknown reason, despite the server running and having started a
browser. This server is also never stopped so the test never finishes because
it's waiting for child processes to exit.

It turns out that the $sel->start; call is not needed at all, and everything works
just as well without it. It's also not mentioned in Test::WWW::Selenium
documentation and examples.

The attached patch fixes this issue and gives a shot at stopping the Selenium
server by issuing a shutDown command before killing the Alien::SeleniumRC
child.

Tests run successfully for me and the module installs with this patch.
lib/Test/WWW/Selenium/Catalyst.pm