From cd622cb5da4a21522c4d0fdda00af33a45355f11 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Mon, 10 Feb 2025 11:31:34 +0100 Subject: [PATCH] Bug 36081: Fix some failing tests Content-Type: text/plain; charset=utf-8 Signed-off-by: Magnus Enger Signed-off-by: Marcel de Rooy --- t/db_dependent/ArticleRequests.t | 13 +++++---- t/db_dependent/Koha/ArticleRequest.t | 41 ++++++++++++++++++---------- t/db_dependent/Koha/Biblio.t | 14 ++++++---- t/db_dependent/Koha/Patron.t | 14 ++++++---- 4 files changed, 53 insertions(+), 29 deletions(-) diff --git a/t/db_dependent/ArticleRequests.t b/t/db_dependent/ArticleRequests.t index f4abe2c8af..1753edd4b2 100755 --- a/t/db_dependent/ArticleRequests.t +++ b/t/db_dependent/ArticleRequests.t @@ -71,12 +71,15 @@ my $patron_2 = $builder->build_object( { class => 'Koha::Patrons', value => { fl # store Koha::Notice::Messages->delete; my $article_request_title = 'an article request title'; -my $article_request = Koha::ArticleRequest->new( +my $article_request = $builder->build_object( { - borrowernumber => $patron->id, - biblionumber => $item->biblionumber, - itemnumber => $item->itemnumber, - title => $article_request_title, + class => 'Koha::ArticleRequests', + value => { + borrowernumber => $patron->id, + biblionumber => $item->biblionumber, + itemnumber => $item->itemnumber, + title => $article_request_title, + } } )->request(); diff --git a/t/db_dependent/Koha/ArticleRequest.t b/t/db_dependent/Koha/ArticleRequest.t index 6d158e7505..3de1619b06 100755 --- a/t/db_dependent/Koha/ArticleRequest.t +++ b/t/db_dependent/Koha/ArticleRequest.t @@ -46,10 +46,14 @@ subtest 'request() tests' => sub { my $ar_module = mock_article_request_module(); - my $ar = Koha::ArticleRequest->new( + my $ar = $builder->build_object( { - borrowernumber => $patron->id, - biblionumber => $item->biblionumber, + class => 'Koha::ArticleRequests', + value => { + borrowernumber => $patron->id, + biblionumber => $item->biblionumber, + debit_id => undef, + } } ); @@ -67,11 +71,14 @@ subtest 'request() tests' => sub { # set a fee amount $amount = 10; - $ar = Koha::ArticleRequest->new( + $ar = $builder->build_object( { - borrowernumber => $patron->id, - biblionumber => $item->biblionumber, - itemnumber => $item->id, + class => 'Koha::ArticleRequests', + value => { + borrowernumber => $patron->id, + biblionumber => $item->biblionumber, + itemnumber => $item->id, + } } ); @@ -101,10 +108,13 @@ subtest 'set_pending() tests' => sub { my $ar_module = mock_article_request_module(); - my $ar = Koha::ArticleRequest->new( + my $ar = $builder->build_object( { - borrowernumber => $patron->id, - biblionumber => $biblio->id, + class => 'Koha::ArticleRequests', + value => { + borrowernumber => $patron->id, + biblionumber => $biblio->id, + } } ); @@ -176,11 +186,14 @@ subtest 'cancel() tests' => sub { my $ar_module = mock_article_request_module(); - my $ar = Koha::ArticleRequest->new( + my $ar = $builder->build_object( { - borrowernumber => $patron->id, - biblionumber => $item->biblionumber, - itemnumber => $item->id, + class => 'Koha::ArticleRequests', + value => { + borrowernumber => $patron->id, + biblionumber => $item->biblionumber, + itemnumber => $item->id, + } } ); diff --git a/t/db_dependent/Koha/Biblio.t b/t/db_dependent/Koha/Biblio.t index 9d697b7850..fd02500543 100755 --- a/t/db_dependent/Koha/Biblio.t +++ b/t/db_dependent/Koha/Biblio.t @@ -1524,12 +1524,16 @@ subtest 'article_requests() tests' => sub { my $patron = $builder->build_object( { class => 'Koha::Patrons' } ); - Koha::ArticleRequest->new( + $builder->build_object( { - borrowernumber => $patron->id, - biblionumber => $biblio->id, - itemnumber => $item->id, - title => $biblio->title, + class => 'Koha::ArticleRequests', + value => { + borrowernumber => $patron->id, + biblionumber => $biblio->id, + itemnumber => $item->id, + title => $biblio->title, + + } } )->request; } diff --git a/t/db_dependent/Koha/Patron.t b/t/db_dependent/Koha/Patron.t index f1ea69f046..fdb6132dea 100755 --- a/t/db_dependent/Koha/Patron.t +++ b/t/db_dependent/Koha/Patron.t @@ -1353,12 +1353,16 @@ subtest 'article_requests() tests' => sub { my $item = $builder->build_sample_item; - Koha::ArticleRequest->new( + $builder->build_object( { - borrowernumber => $patron->id, - biblionumber => $item->biblionumber, - itemnumber => $item->id, - title => "Title", + class => 'Koha::ArticleRequests', + value => { + borrowernumber => $patron->id, + biblionumber => $item->biblionumber, + itemnumber => $item->id, + title => "Title", + + } } )->request; } -- 2.39.5