From 7c9680749fa3a56ee65e823b318b22876a95edce Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Tue, 21 Jan 2020 12:57:01 +0100 Subject: [PATCH] Bug 22898: Fix failure since bug 19618 See comment in the code, this is not the correct fix, but cannot find something better for now. Signed-off-by: Martin Renvoize --- t/db_dependent/selenium/basic_workflow.t | 2 ++ 1 file changed, 2 insertions(+) diff --git a/t/db_dependent/selenium/basic_workflow.t b/t/db_dependent/selenium/basic_workflow.t index 7a87827910..62dcf0e470 100644 --- a/t/db_dependent/selenium/basic_workflow.t +++ b/t/db_dependent/selenium/basic_workflow.t @@ -237,12 +237,14 @@ SKIP: { #Place holds $driver->get($base_url."/reserve/request.pl?borrowernumber=$borrowernumber&biblionumber=".$biblionumbers[0]); $driver->find_element('//form[@id="hold-request-form"]//button[@type="submit"]')->click; # Biblio level + $driver->pause(1000); # This seems wrong, since bug 19618 the hold is created async with an AJAX call. Not sure what is happening here but the next statements are exectuted before the hold is created and the count is wrong (still 0) 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"]//button[@type="submit"]')->click; + $driver->pause(1000); is( $patron->holds->count, 2, ); time_diff("holds"); -- 2.39.5