From d4970fc7e4970aabd4993d4b17c5ef41c2c86e7c Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 29 Apr 2025 15:16:01 +0200 Subject: [PATCH] Bug 39323: Fix selenium/update_child_to_adult.t This test is not really needed. It the new window opens correctly the next test will pass. And should fail if it does not. --- t/db_dependent/selenium/update_child_to_adult.t | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/t/db_dependent/selenium/update_child_to_adult.t b/t/db_dependent/selenium/update_child_to_adult.t index 8e56ecf8b3c..929ff9d9367 100755 --- a/t/db_dependent/selenium/update_child_to_adult.t +++ b/t/db_dependent/selenium/update_child_to_adult.t @@ -112,7 +112,7 @@ subtest 'Update child to patron' => sub { my $adult_borrowernumber = $adult_1->borrowernumber; subtest 'Update child to adult' => sub { - plan tests => 3; + plan tests => 2; $driver->get( $base_url . "/members/moremember.pl?borrowernumber=" . $child_borrowernumber ); $driver->find_element('//div[@id="toolbar"]/div[@class="btn-group"][last()]')->click; # More button group my $update_link = $driver->find_element('//a[@id="updatechild"]'); @@ -122,10 +122,6 @@ subtest 'Update child to patron' => sub { 'The update link should not have a data-bs-toggle attribute => not a tooltip and can be clickable' ); $update_link->click; - like( - $driver->get_current_url, qr{/members/moremember\.pl\?borrowernumber=$child_borrowernumber\#$}, - 'Current window has a "#" in the URL, event has been triggered' - ); # Switch to the popup window # Note that if there is only 1 adult in the DB the popup does not appears, but an alert instead. Not tested so far. -- 2.34.1