Bugzilla – Attachment 98877 Details for
Bug 22898
Selenium tests for placing holds from the staff interface
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 22898: Add selenium tests placing holds (staff)
Bug-22898-Add-selenium-tests-placing-holds-staff.patch (text/plain), 3.67 KB, created by
Jonathan Druart
on 2020-02-13 16:29:44 UTC
(
hide
)
Description:
Bug 22898: Add selenium tests placing holds (staff)
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2020-02-13 16:29:44 UTC
Size:
3.67 KB
patch
obsolete
>From b7e207e447cb657421816271ca0c75845cccc15c Mon Sep 17 00:00:00 2001 >From: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Date: Mon, 13 May 2019 11:31:14 -0500 >Subject: [PATCH] Bug 22898: Add selenium tests placing holds (staff) > >Quick selenium tests to prevent regressions like bug 22895. > >Test plan: >Make sure the selenium tests return green > >You can also test the buggy branch: >% git checkout v18.11.05 >% prove t/db_dependent/selenium/basic_workflow.t >=> Failure >% git bz apply 22895. >% prove t/db_dependent/selenium/basic_workflow.t >=> Success > >Signed-off-by: Victor Grousset <victor.grousset@biblibre.com> >--- > t/db_dependent/selenium/basic_workflow.t | 36 +++++++++++++++++++++++++------- > 1 file changed, 28 insertions(+), 8 deletions(-) > >diff --git a/t/db_dependent/selenium/basic_workflow.t b/t/db_dependent/selenium/basic_workflow.t >index 5fbe949e58..4883ef98df 100644 >--- a/t/db_dependent/selenium/basic_workflow.t >+++ b/t/db_dependent/selenium/basic_workflow.t >@@ -40,7 +40,7 @@ use C4::Biblio qw( AddBiblio ); # We shouldn't use it > > use Koha::CirculationRules; > >-use Test::More tests => 20; >+use Test::More tests => 22; > use MARC::Record; > use MARC::Field; > >@@ -77,6 +77,9 @@ our $sample_data = { > issuelength => '5', > lengthunit => 'days', > renewalperiod => '5', >+ reservesallowed => '5', >+ onshelfholds => '1', >+ opacitemholds => 'Y', > }, > }; > our ( $borrowernumber, $start, $prev_time, $cleanup_needed ); >@@ -85,7 +88,7 @@ $dbh->do(q|INSERT INTO itemtypes(itemtype) VALUES (?)|, undef, $sample_data->{it > > SKIP: { > eval { require Selenium::Remote::Driver; }; >- skip "Selenium::Remote::Driver is needed for selenium tests.", 20 if $@; >+ skip "Selenium::Remote::Driver is needed for selenium tests.", 22 if $@; > > $cleanup_needed = 1; > >@@ -155,12 +158,16 @@ SKIP: { > categorycode => $issuing_rules->{categorycode}, > itemtype => $issuing_rules->{itemtype}, > branchcode => $issuing_rules->{branchcode}, >- rules => { >- maxissueqty => $issuing_rules->{maxissueqty}, >- issuelength => $issuing_rules->{issuelength}, >- lengthunit => $issuing_rules->{lengthunit}, >- renewalperiod => $issuing_rules->{renewalperiod}, >- } >+ rules => { >+ maxissueqty => $issuing_rules->{maxissueqty}, >+ issuelength => $issuing_rules->{issuelength}, >+ lengthunit => $issuing_rules->{lengthunit}, >+ renewalperiod => $issuing_rules->{renewalperiod}, >+ reservesallowed => $issuing_rules->{reservesallowed}, >+ onshelfholds => $issuing_rules->{onshelfholds}, >+ opacitemholds => $issuing_rules->{opacitemholds}, >+ >+ } > } > ); > >@@ -227,6 +234,19 @@ SKIP: { > > time_diff("checkin"); > >+ #Place holds >+ $driver->get($base_url."/reserve/request.pl?borrowernumber=$borrowernumber&biblionumber=".$biblionumbers[0]); >+ $driver->find_element('//form[@id="hold-request-form"]//input[@type="submit"]')->click; # Biblio level >+ my $patron = Koha::Patrons->find($borrowernumber); >+ is( $patron->holds->count, 1, ); >+ >+ $driver->get($base_url."/reserve/request.pl?borrowernumber=$borrowernumber&biblionumber=".$biblionumbers[1]); >+ $driver->find_element('//form[@id="hold-request-form"]//input[@type="radio"]')->click; # Item level, there is only 1 item per bib so we are safe >+ $driver->find_element('//form[@id="hold-request-form"]//input[@type="submit"]')->click; >+ is( $patron->holds->count, 2, ); >+ >+ time_diff("holds"); >+ > close $fh; > $driver->quit(); > }; >-- >2.11.0
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 22898
:
89673
|
90208
|
90241
|
97655
|
97656
|
98877
|
98878
|
98879
|
98880
|
98881
|
98882