Revert "Bug 21479: Remove the trace if the node does not exist"
authorFridolin Somers <fridolin.somers@biblibre.com>
Wed, 28 Nov 2018 15:25:12 +0000 (16:25 +0100)
committerFridolin Somers <fridolin.somers@biblibre.com>
Wed, 28 Nov 2018 15:25:12 +0000 (16:25 +0100)
This reverts commit 5eaf5b69d15a9469206b6cc1da64504be653631c.

t/db_dependent/selenium/regressions.t

index 8b08503..bffddc3 100644 (file)
@@ -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' );