From 5b61c030894e0f23caf3d856f531f0ce5723bf22 Mon Sep 17 00:00:00 2001 From: Nick Clemens Date: Wed, 7 Aug 2024 17:31:46 +0000 Subject: [PATCH] Bug 37373: (follow-up) Add id and adjust tests --- .../prog/en/includes/patron-search.inc | 4 +-- t/db_dependent/selenium/patrons_search.t | 26 +++++++++---------- 2 files changed, 15 insertions(+), 15 deletions(-) diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc index 5b2e3f23a3c..747a106f65a 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/patron-search.inc @@ -13,7 +13,7 @@ [% BLOCK patron_search_filters_simple %]
Enter patron card number or partial name:
- +
[% END %] @@ -36,7 +36,7 @@
  1. - +
  2. [% FOR f IN filters %] diff --git a/t/db_dependent/selenium/patrons_search.t b/t/db_dependent/selenium/patrons_search.t index e410c4833c5..a6554cb121c 100755 --- a/t/db_dependent/selenium/patrons_search.t +++ b/t/db_dependent/selenium/patrons_search.t @@ -239,7 +239,7 @@ subtest 'Search patrons' => sub { # NOTE: We should probably ensure the bad field is removed from 'standard' search here, else searches are broken C4::Context->set_preference('DefaultPatronSearchFields',""); $driver->get( $base_url . "/members/members-home.pl" ); - $s->fill_form( { 'class=search_patron_filter' => 'test_patron' } ); + $s->fill_form( { 'search_patron_filter' => 'test_patron' } ); $s->submit_form; my $first_patron = Koha::Patrons->search( { surname => { like => 'test_patron_%' } } )->next; @@ -276,7 +276,7 @@ subtest 'Search patrons' => sub { ); $driver->get( $base_url . "/members/members-home.pl" ); - $s->fill_form( { 'class=search_patron_filter' => 'test_patron' } ); + $s->fill_form( { 'search_patron_filter' => 'test_patron' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -297,7 +297,7 @@ subtest 'Search patrons' => sub { is( $driver->find_element('//div[@id="'.$table_id.'_info"]')->get_text, sprintf('Showing 1 to %s of %s entries', $PatronsPerPage, $total_number_of_patrons), 'Resetting filters works as expected' ); # Pattern terms must be split - $s->fill_form( { 'class=search_patron_filter' => 'test patron' } ); + $s->fill_form( { 'search_patron_filter' => 'test patron' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -307,7 +307,7 @@ subtest 'Search patrons' => sub { sleep $DT_delay && $s->wait_for_ajax; # Search on non-searchable attribute, we expect no result! - $s->fill_form( { 'class=search_patron_filter' => 'test_attr_1' } ); + $s->fill_form( { 'search_patron_filter' => 'test_attr_1' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -317,7 +317,7 @@ subtest 'Search patrons' => sub { $driver->find_element('//form[@class="patron_search_form"]//*[@class="btn btn-default clear_search"]')->click(); # Search on searchable attribute, we expect 2 patrons - $s->fill_form( { 'class=search_patron_filter' => 'test_attr_2' } ); + $s->fill_form( { 'search_patron_filter' => 'test_attr_2' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -326,7 +326,7 @@ subtest 'Search patrons' => sub { # clear form $driver->find_element('//form[@class="patron_search_form"]//*[@class="btn btn-default clear_search"]')->click(); - $s->fill_form( { 'class=search_patron_filter' => 'test_attr_3' } ); # Terms must be split + $s->fill_form( { 'search_patron_filter' => 'test_attr_3' } ); # Terms must be split $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -336,7 +336,7 @@ subtest 'Search patrons' => sub { $driver->find_element('//form[@class="patron_search_form"]//*[@class="btn btn-default clear_search"]')->click(); # Search on searchable attribute as specific field, we expect 2 patrons - $s->fill_form( { 'class=search_patron_filter' => 'test_attr_4' } ); + $s->fill_form( { 'search_patron_filter' => 'test_attr_4' } ); $driver->find_element('//form[@class="patron_search_form"]//*[@class="searchfieldstype_filter"]//option[@value="_ATTR_'.$attribute_type_searchable_not_default->code.'"]')->click(); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -361,7 +361,7 @@ subtest 'Search patrons' => sub { C4::Context->set_preference( 'PatronsPerPage', 5 ); $driver->get( $base_url . "/members/members-home.pl" ); - $s->fill_form( { 'class=search_patron_filter' => 'test_patron' } ); + $s->fill_form( { 'search_patron_filter' => 'test_patron' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; my $patron_selected_text = $driver->find_element('//div[@id="table_search_selections"]/span')->get_text; @@ -387,7 +387,7 @@ subtest 'Search patrons' => sub { # Perform another search $driver->get( $base_url . "/members/members-home.pl" ); - $s->fill_form( { 'class=search_patron_filter' => 'test_patron' } ); + $s->fill_form( { 'search_patron_filter' => 'test_patron' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; $patron_selected_text = $driver->find_element('//div[@id="table_search_selections"]/span')->get_text; @@ -414,7 +414,7 @@ subtest 'Search patrons' => sub { # We have a patron with date of birth=1980-06-17 => formatted as 17/06/1980 $driver->get( $base_url . "/members/members-home.pl" ); - $s->fill_form( { 'class=search_patron_filter' => 'test_patron' } ); + $s->fill_form( { 'search_patron_filter' => 'test_patron' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -491,7 +491,7 @@ subtest 'Search patrons in modal' => sub { $s->wait_for_element_visible('//div[@id="patron_search_modal"]//div[@class="modal-header"]'); # Search for our test patrons - $s->fill_form( { 'class=search_patron_filter' => 'test_patron' } ); + $s->fill_form( { 'search_patron_filter' => 'test_patron' } ); $s->submit_form; sleep $DT_delay && $s->wait_for_ajax; @@ -582,7 +582,7 @@ subtest 'Search patrons in modal' => sub { $s->wait_for_element_visible('//div[@id="patron_search_modal_owner"]//div[@class="modal-header"]'); # Search for our test patrons - $driver->find_element('//div[@id="patron_search_modal_owner"]//input[@class="search_patron_filter"]') + $driver->find_element('//div[@id="patron_search_modal_owner"]//input[@id="search_patron_filter"]') ->send_keys('test_patron'); $driver->find_element('//div[@id="patron_search_modal_owner"]//input[@type="submit"]')->click; @@ -644,7 +644,7 @@ subtest 'Search patrons in modal' => sub { $s->wait_for_element_visible('//div[@id="patron_search_modal_users"]//div[@class="modal-header"]'); # Search for our test patrons - $driver->find_element('//div[@id="patron_search_modal_users"]//input[@class="search_patron_filter"]') + $driver->find_element('//div[@id="patron_search_modal_users"]//input[@id="search_patron_filter"]') ->send_keys('test_patron'); $driver->find_element('//div[@id="patron_search_modal_users"]//input[@type="submit"]')->click; -- 2.39.2