[Jifty-commit] jifty branch, master, updated. 1.10518-50-g9da5c55

[Jifty commits]

The branch, master has been updated
       via  9da5c55053519b09d806957434b467e93953267f (commit)
      from  7d91ce29da36ad59ed16d9121e70f9435f27faa8 (commit)

Summary of changes:
 lib/Jifty/Test/WWW/Selenium.pm |   11 ++++++++---
 1 files changed, 8 insertions(+), 3 deletions(-)

- Log -----------------------------------------------------------------
commit 9da5c55053519b09d806957434b467e93953267f
Author: Thomas Sibley <trs at bestpractical.com>
Date:   Tue Oct 11 22:34:02 2011 -0400

    Test for Alien::SeleniumRC failure and yell about it

diff --git a/lib/Jifty/Test/WWW/Selenium.pm b/lib/Jifty/Test/WWW/Selenium.pm
index 3f3d11a..f2b944b 100644
--- a/lib/Jifty/Test/WWW/Selenium.pm
+++ b/lib/Jifty/Test/WWW/Selenium.pm
@@ -122,9 +122,14 @@ sub _start_src {
         Test::More::diag "start selenium rc [$$]";
         local $SIG{CHLD} = \&_REAPER;
         local $SIG{TERM} = sub { exit 0 };
-        Alien::SeleniumRC::start(@{ $args{args} || [] });
-        Test::More::diag "selenium rc [$$] finished.";
-        exit;
+        eval { Alien::SeleniumRC::start(@{ $args{args} || [] }) };
+        if ($@) {
+            Test::More::diag "selenium rc [$$] failed to start: $@";
+            exit 1;
+        } else {
+            Test::More::diag "selenium rc [$$] finished.";
+            exit;
+        }
     }
 }
 

-----------------------------------------------------------------------