From 7ae75370ecf74c10c94789e49e3f29863e666bda Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Wed, 28 Nov 2018 16:25:12 +0100 Subject: [PATCH] Revert "Bug 21479: Remove the trace if the node does not exist" This reverts commit 5eaf5b69d15a9469206b6cc1da64504be653631c. --- t/db_dependent/selenium/regressions.t | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/t/db_dependent/selenium/regressions.t b/t/db_dependent/selenium/regressions.t index 8b085039e2..bffddc3713 100644 --- a/t/db_dependent/selenium/regressions.t +++ b/t/db_dependent/selenium/regressions.t @@ -36,13 +36,17 @@ subtest 'OPAC - Remove from cart' => sub { $driver->get( $opac_base_url . "opac-search.pl?q=d" ); - # A better way to do that would be to modify the way we display the basket count - # We should show/hide the count instead or recreate the node - my @basket_count_elts = $driver->find_elements('//span[@id="basketcount"]/span'); - is( scalar(@basket_count_elts), 0, 'Basket should be empty'); + my $basket_count_elt; + eval { + # FIXME This will produce a STRACE + # A better way to do that would be to modify the way we display the basket count + # We should show/hide the count instead or recreate the node + $basket_count_elt = $driver->find_element('//span[@id="basketcount"]/span') + }; + like($@, qr{An element could not be located on the page}, 'Basket should be empty'); $driver->find_element('//a[@class="addtocart cart1"]')->click; - my $basket_count_elt = $driver->find_element('//span[@id="basketcount"]/span'); + $basket_count_elt = $driver->find_element('//span[@id="basketcount"]/span'); is( $basket_count_elt->get_text(), 1, 'One element should have been added to the cart' ); -- 2.39.5