@@ -, +, @@ --------- prove -v t/db_dependent/selenium until it doesn't skip because of missing dependencies, but rather because of not running a selenium server. -- they will skip gracefully and informatively. --- t/db_dependent/selenium/administration_tasks.t | 9 ++++++++- t/db_dependent/selenium/basic_workflow.t | 10 ++++++++-- 2 files changed, 16 insertions(+), 3 deletions(-) --- a/t/db_dependent/selenium/administration_tasks.t +++ a/t/db_dependent/selenium/administration_tasks.t @@ -24,6 +24,7 @@ use Modern::Perl; use C4::Context; use Test::More tests => 1; +use Try::Tiny; use t::lib::Selenium; @@ -42,7 +43,13 @@ SKIP: { $cleanup_needed = 1; - my $s = t::lib::Selenium->new; + my $s; + try { + $s = t::lib::Selenium->new; + } catch { + skip "Unable to create selenium driver link. Please review https://wiki.koha-community.org/wiki/Using_Selenium_with_Koha", 1; + }; + my $driver = $s->driver; my $mainpage = $s->base_url . q|mainpage.pl|; $driver->get($mainpage); --- a/t/db_dependent/selenium/basic_workflow.t +++ a/t/db_dependent/selenium/basic_workflow.t @@ -41,6 +41,7 @@ use C4::Biblio qw( AddBiblio ); # We shouldn't use it use Test::More tests => 20; use MARC::Record; use MARC::Field; +use Try::Tiny; use t::lib::Selenium; @@ -85,9 +86,14 @@ SKIP: { $cleanup_needed = 1; - open my $fh, '>>', '/tmp/output.txt'; + my $s; + try { + $s = t::lib::Selenium->new; + } catch { + skip "Unable to create selenium driver link. Please review https://wiki.koha-community.org/wiki/Using_Selenium_with_Koha", 20; + }; - my $s = t::lib::Selenium->new; + open my $fh, '>>', '/tmp/output.txt'; my $driver = $s->driver; my $base_url = $s->base_url; --