Bug 36081: Fix some failing tests
Signed-off-by: Magnus Enger <magnus@libriotech.no> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Alex Buckley <alexbuckley@catalyst.net.nz>
This commit is contained in:
parent
05353f0cf0
commit
5a8aef083d
|
|
@ -69,12 +69,15 @@ my $patron_2 = $builder->build_object({ class => 'Koha::Patrons', value => { fla
|
|||
# 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();
|
||||
|
||||
|
|
|
|||
|
|
@ -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,
|
||||
}
|
||||
}
|
||||
);
|
||||
|
||||
|
|
@ -174,11 +184,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,
|
||||
}
|
||||
}
|
||||
);
|
||||
|
||||
|
|
|
|||
|
|
@ -1331,12 +1331,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;
|
||||
}
|
||||
|
|
|
|||
|
|
@ -1348,12 +1348,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;
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue