Jonathan Druart
e7d40b7a99
Agreement.vendor_id has a default value defined in TestBuilder, but here we were assuming there was one. We should then generate it and pass the value when building the agreement objects. The failure was: koha_1 | # Failed test '200 OK' koha_1 | # at t/db_dependent/api/v1/erm_agreements.t line 95. koha_1 | # got: '400' koha_1 | # expected: '200' koha_1 | koha_1 | # Failed test 'exact match for JSON Pointer ""' koha_1 | # at t/db_dependent/api/v1/erm_agreements.t line 95. koha_1 | # Structures begin differing at: koha_1 | # $got = HASH(0x55cde781a480) koha_1 | # $expected = ARRAY(0x55cde75af9c8) koha_1 | # Looks like you failed 2 tests of 32. 400 because we were calling http://localhost:8081/api/v1/erm/agreements?vendor_id=' which returns {"errors":[{"message":"Expected integer - got string.","path":"\/vendor_id"}],"status":400}% Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
493 lines
16 KiB
Perl
Executable file
493 lines
16 KiB
Perl
Executable file
#!/usr/bin/env perl
|
|
|
|
# This file is part of Koha.
|
|
#
|
|
# Koha is free software; you can redistribute it and/or modify it
|
|
# under the terms of the GNU General Public License as published by
|
|
# the Free Software Foundation; either version 3 of the License, or
|
|
# (at your option) any later version.
|
|
#
|
|
# Koha is distributed in the hope that it will be useful, but
|
|
# WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
# GNU General Public License for more details.
|
|
#
|
|
# You should have received a copy of the GNU General Public License
|
|
# along with Koha; if not, see <http://www.gnu.org/licenses>.
|
|
|
|
use Modern::Perl;
|
|
|
|
use Test::More tests => 5;
|
|
use Test::Mojo;
|
|
|
|
use t::lib::TestBuilder;
|
|
use t::lib::Mocks;
|
|
|
|
use Koha::ERM::Agreements;
|
|
use Koha::Database;
|
|
|
|
my $schema = Koha::Database->new->schema;
|
|
my $builder = t::lib::TestBuilder->new;
|
|
|
|
my $t = Test::Mojo->new('Koha::REST::V1');
|
|
t::lib::Mocks::mock_preference( 'RESTBasicAuth', 1 );
|
|
|
|
subtest 'list() tests' => sub {
|
|
|
|
plan tests => 32;
|
|
|
|
$schema->storage->txn_begin;
|
|
|
|
Koha::ERM::Agreements->search->delete;
|
|
|
|
my $librarian = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 2**28 }
|
|
}
|
|
);
|
|
my $password = 'thePassword123';
|
|
$librarian->set_password( { password => $password, skip_validation => 1 } );
|
|
my $userid = $librarian->userid;
|
|
|
|
my $patron = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 0 }
|
|
}
|
|
);
|
|
|
|
$patron->set_password( { password => $password, skip_validation => 1 } );
|
|
my $unauth_userid = $patron->userid;
|
|
|
|
## Authorized user tests
|
|
# No agreements, so empty array should be returned
|
|
$t->get_ok("//$userid:$password@/api/v1/erm/agreements")->status_is(200)
|
|
->json_is( [] );
|
|
|
|
my $vendor_1 = $builder->build_object({ class => 'Koha::Acquisition::Booksellers' });
|
|
my $agreement = $builder->build_object(
|
|
{
|
|
class => 'Koha::ERM::Agreements',
|
|
value => { vendor_id => $vendor_1->id },
|
|
}
|
|
);
|
|
|
|
# One agreement created, should get returned
|
|
$t->get_ok("//$userid:$password@/api/v1/erm/agreements")->status_is(200)
|
|
->json_is( [ $agreement->to_api ] );
|
|
|
|
my $another_agreement = $builder->build_object(
|
|
{
|
|
class => 'Koha::ERM::Agreements',
|
|
value => { vendor_id => $vendor_1->id }
|
|
}
|
|
);
|
|
|
|
my $vendor_2 = $builder->build_object({ class => 'Koha::Acquisition::Booksellers' });
|
|
my $agreement_with_another_vendor_id = $builder->build_object(
|
|
{
|
|
class => 'Koha::ERM::Agreements',
|
|
value => { vendor_id => $vendor_2->id },
|
|
}
|
|
);
|
|
|
|
# Two agreements created, they should both be returned
|
|
$t->get_ok("//$userid:$password@/api/v1/erm/agreements")->status_is(200)
|
|
->json_is(
|
|
[
|
|
$agreement->to_api,
|
|
$another_agreement->to_api,
|
|
$agreement_with_another_vendor_id->to_api
|
|
]
|
|
);
|
|
|
|
# Filtering works, two agreements sharing vendor_id
|
|
$t->get_ok( "//$userid:$password@/api/v1/erm/agreements?vendor_id="
|
|
. $vendor_1->id )->status_is(200)
|
|
->json_is( [ $agreement->to_api, $another_agreement->to_api ] );
|
|
|
|
# Attempt to search by name like 'ko'
|
|
$agreement->delete;
|
|
$another_agreement->delete;
|
|
$agreement_with_another_vendor_id->delete;
|
|
$t->get_ok( qq~//$userid:$password@/api/v1/erm/agreements?q=[{"me.name":{"like":"%ko%"}}]~)
|
|
->status_is(200)
|
|
->json_is( [] );
|
|
|
|
my $agreement_to_search = $builder->build_object(
|
|
{
|
|
class => 'Koha::ERM::Agreements',
|
|
value => {
|
|
name => 'koha',
|
|
}
|
|
}
|
|
);
|
|
|
|
# Search works, searching for name like 'ko'
|
|
$t->get_ok( qq~//$userid:$password@/api/v1/erm/agreements?q=[{"me.name":{"like":"%ko%"}}]~)
|
|
->status_is(200)
|
|
->json_is( [ $agreement_to_search->to_api ] );
|
|
|
|
# Warn on incorrect filter date format
|
|
$t->get_ok( "//$userid:$password@/api/v1/erm/agreements?max_expiration_date=19-03-2021")
|
|
->status_is(400)
|
|
->json_is(
|
|
"/errors" => [
|
|
{
|
|
message => "Does not match date format.",
|
|
path => "/max_expiration_date"
|
|
}
|
|
]
|
|
);
|
|
|
|
# Attempt to filter by expired on 2021-03-19
|
|
$t->get_ok( "//$userid:$password@/api/v1/erm/agreements?max_expiration_date=2021-03-19")
|
|
->status_is(200)
|
|
->json_is( [] );
|
|
|
|
my $agreement_to_filter = $builder->build_object(
|
|
{
|
|
class => 'Koha::ERM::Agreements'
|
|
}
|
|
);
|
|
$agreement_to_filter->periods(
|
|
[
|
|
{
|
|
'started_on'=>'2021-03-17',
|
|
'ended_on'=>'2021-03-19'
|
|
}
|
|
]
|
|
);
|
|
# Filter by expired on 2021-03-19
|
|
$t->get_ok( "//$userid:$password@/api/v1/erm/agreements?max_expiration_date=2021-03-19")
|
|
->status_is(200)
|
|
->json_is( [ $agreement_to_filter->to_api ] );
|
|
|
|
# Warn on unsupported query parameter
|
|
$t->get_ok("//$userid:$password@/api/v1/erm/agreements?blah=blah")
|
|
->status_is(400)
|
|
->json_is(
|
|
[ { path => '/query/blah', message => 'Malformed query string' } ] );
|
|
|
|
# Unauthorized access
|
|
$t->get_ok("//$unauth_userid:$password@/api/v1/erm/agreements")
|
|
->status_is(403);
|
|
|
|
$schema->storage->txn_rollback;
|
|
};
|
|
|
|
subtest 'get() tests' => sub {
|
|
|
|
plan tests => 14;
|
|
|
|
$schema->storage->txn_begin;
|
|
|
|
my $agreement =
|
|
$builder->build_object( { class => 'Koha::ERM::Agreements' } );
|
|
my $librarian = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 2**28 }
|
|
}
|
|
);
|
|
my $password = 'thePassword123';
|
|
$librarian->set_password( { password => $password, skip_validation => 1 } );
|
|
my $userid = $librarian->userid;
|
|
|
|
my $patron = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 0 }
|
|
}
|
|
);
|
|
|
|
$patron->set_password( { password => $password, skip_validation => 1 } );
|
|
my $unauth_userid = $patron->userid;
|
|
|
|
# This agreement exists, should get returned
|
|
$t->get_ok( "//$userid:$password@/api/v1/erm/agreements/"
|
|
. $agreement->agreement_id )->status_is(200)
|
|
->json_is( $agreement->to_api );
|
|
|
|
# Return one agreement with some embeds
|
|
$t->get_ok( "//$userid:$password@/api/v1/erm/agreements/"
|
|
. $agreement->agreement_id => {'x-koha-embed' => 'periods,user_roles,agreement_licenses'} )->status_is(200)
|
|
->json_is( { %{ $agreement->to_api }, periods => [], user_roles => [], agreement_licenses => [] });
|
|
|
|
# Return one agreement with all embeds
|
|
$t->get_ok( "//$userid:$password@/api/v1/erm/agreements/"
|
|
. $agreement->agreement_id => {'x-koha-embed' => 'periods,user_roles,user_roles.patron,agreement_licenses,agreement_licenses.license'} )->status_is(200)
|
|
->json_is( { %{ $agreement->to_api }, periods => [], user_roles => [], agreement_licenses => [] });
|
|
|
|
# Unauthorized access
|
|
$t->get_ok( "//$unauth_userid:$password@/api/v1/erm/agreements/"
|
|
. $agreement->agreement_id )->status_is(403);
|
|
|
|
# Attempt to get non-existent agreement
|
|
my $agreement_to_delete =
|
|
$builder->build_object( { class => 'Koha::ERM::Agreements' } );
|
|
my $non_existent_id = $agreement_to_delete->agreement_id;
|
|
$agreement_to_delete->delete;
|
|
|
|
$t->get_ok("//$userid:$password@/api/v1/erm/agreements/$non_existent_id")
|
|
->status_is(404)->json_is( '/error' => 'Agreement not found' );
|
|
|
|
$schema->storage->txn_rollback;
|
|
};
|
|
|
|
subtest 'add() tests' => sub {
|
|
|
|
plan tests => 22;
|
|
|
|
$schema->storage->txn_begin;
|
|
|
|
my $librarian = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 2**28 }
|
|
}
|
|
);
|
|
my $password = 'thePassword123';
|
|
$librarian->set_password( { password => $password, skip_validation => 1 } );
|
|
my $userid = $librarian->userid;
|
|
|
|
my $patron = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 0 }
|
|
}
|
|
);
|
|
|
|
$patron->set_password( { password => $password, skip_validation => 1 } );
|
|
my $unauth_userid = $patron->userid;
|
|
|
|
my $agreement = {
|
|
vendor_id => undef,
|
|
name => "Agreement name",
|
|
description => "Agreement description",
|
|
status => "active",
|
|
closure_reason => "",
|
|
is_perpetual => 1,
|
|
renewal_priority => "",
|
|
license_info => "Agreement license_info",
|
|
};
|
|
|
|
# Unauthorized attempt to write
|
|
$t->post_ok( "//$unauth_userid:$password@/api/v1/erm/agreements" => json =>
|
|
$agreement )->status_is(403);
|
|
|
|
# Authorized attempt to write invalid data
|
|
my $agreement_with_invalid_field = {
|
|
blah => "Agreement Blah",
|
|
name => "Agreement name",
|
|
description => "Agreement description",
|
|
status => "active",
|
|
closure_reason => "",
|
|
is_perpetual => 1,
|
|
renewal_priority => "",
|
|
license_info => "Agreement license_info",
|
|
};
|
|
|
|
$t->post_ok( "//$userid:$password@/api/v1/erm/agreements" => json =>
|
|
$agreement_with_invalid_field )->status_is(400)->json_is(
|
|
"/errors" => [
|
|
{
|
|
message => "Properties not allowed: blah.",
|
|
path => "/body"
|
|
}
|
|
]
|
|
);
|
|
|
|
# Authorized attempt to write
|
|
my $agreement_id =
|
|
$t->post_ok(
|
|
"//$userid:$password@/api/v1/erm/agreements" => json => $agreement )
|
|
->status_is( 201, 'SWAGGER3.2.1' )->header_like(
|
|
Location => qr|^/api/v1/erm/agreements/\d*|,
|
|
'SWAGGER3.4.1'
|
|
)->json_is( '/vendor_id' => $agreement->{vendor_id} )
|
|
->json_is( '/name' => $agreement->{name} )
|
|
->json_is( '/description' => $agreement->{description} )
|
|
->json_is( '/status' => $agreement->{status} )
|
|
->json_is( '/closure_reason' => $agreement->{closure_reason} )
|
|
->json_is( '/is_perpetual' => $agreement->{is_perpetual} )
|
|
->json_is( '/renewal_priority' => $agreement->{renewal_priority} )
|
|
->json_is( '/license_info' => $agreement->{license_info} )
|
|
->tx->res->json->{agreement_id};
|
|
|
|
# Authorized attempt to create with null id
|
|
$agreement->{agreement_id} = undef;
|
|
$t->post_ok(
|
|
"//$userid:$password@/api/v1/erm/agreements" => json => $agreement )
|
|
->status_is(400)->json_has('/errors');
|
|
|
|
# Authorized attempt to create with existing id
|
|
$agreement->{agreement_id} = $agreement_id;
|
|
$t->post_ok(
|
|
"//$userid:$password@/api/v1/erm/agreements" => json => $agreement )
|
|
->status_is(400)->json_is(
|
|
"/errors" => [
|
|
{
|
|
message => "Read-only.",
|
|
path => "/body/agreement_id"
|
|
}
|
|
]
|
|
);
|
|
|
|
$schema->storage->txn_rollback;
|
|
};
|
|
|
|
subtest 'update() tests' => sub {
|
|
|
|
plan tests => 15;
|
|
|
|
$schema->storage->txn_begin;
|
|
|
|
my $librarian = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 2**28 }
|
|
}
|
|
);
|
|
my $password = 'thePassword123';
|
|
$librarian->set_password( { password => $password, skip_validation => 1 } );
|
|
my $userid = $librarian->userid;
|
|
|
|
my $patron = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 0 }
|
|
}
|
|
);
|
|
|
|
$patron->set_password( { password => $password, skip_validation => 1 } );
|
|
my $unauth_userid = $patron->userid;
|
|
|
|
my $agreement_id =
|
|
$builder->build_object( { class => 'Koha::ERM::Agreements' } )->agreement_id;
|
|
|
|
# Unauthorized attempt to update
|
|
$t->put_ok(
|
|
"//$unauth_userid:$password@/api/v1/erm/agreements/$agreement_id" =>
|
|
json => { name => 'New unauthorized name change' } )->status_is(403);
|
|
|
|
# Attempt partial update on a PUT
|
|
my $agreement_with_missing_field = {
|
|
description => 'New description',
|
|
status => 'active',
|
|
closure_reason => undef,
|
|
is_perpetual => 1,
|
|
renewal_priority => undef,
|
|
license_info => 'New license_info',
|
|
};
|
|
|
|
$t->put_ok(
|
|
"//$userid:$password@/api/v1/erm/agreements/$agreement_id" => json =>
|
|
$agreement_with_missing_field )->status_is(400)
|
|
->json_is( "/errors" =>
|
|
[ { message => "Missing property.", path => "/body/name" } ] );
|
|
|
|
# Full object update on PUT
|
|
my $agreement_with_updated_field = {
|
|
vendor_id => undef,
|
|
name => 'New name',
|
|
description => 'New description',
|
|
status => 'closed',
|
|
closure_reason => undef,
|
|
is_perpetual => 1,
|
|
renewal_priority => undef,
|
|
license_info => 'New license_info',
|
|
};
|
|
|
|
$t->put_ok(
|
|
"//$userid:$password@/api/v1/erm/agreements/$agreement_id" => json =>
|
|
$agreement_with_updated_field )->status_is(200)
|
|
->json_is( '/name' => 'New name' );
|
|
|
|
# Authorized attempt to write invalid data
|
|
my $agreement_with_invalid_field = {
|
|
blah => "Agreement Blah",
|
|
name => "Agreement name",
|
|
description => "Agreement description",
|
|
status => "closed",
|
|
closure_reason => undef,
|
|
is_perpetual => 1,
|
|
renewal_priority => undef,
|
|
license_info => "Agreement license_info",
|
|
};
|
|
|
|
$t->put_ok(
|
|
"//$userid:$password@/api/v1/erm/agreements/$agreement_id" => json =>
|
|
$agreement_with_invalid_field )->status_is(400)->json_is(
|
|
"/errors" => [
|
|
{
|
|
message => "Properties not allowed: blah.",
|
|
path => "/body"
|
|
}
|
|
]
|
|
);
|
|
|
|
# Attempt to update non-existent agreement
|
|
my $agreement_to_delete =
|
|
$builder->build_object( { class => 'Koha::ERM::Agreements' } );
|
|
my $non_existent_id = $agreement_to_delete->agreement_id;
|
|
$agreement_to_delete->delete;
|
|
|
|
$t->put_ok( "//$userid:$password@/api/v1/erm/agreements/$non_existent_id" =>
|
|
json => $agreement_with_updated_field )->status_is(404);
|
|
|
|
# Wrong method (POST)
|
|
$agreement_with_updated_field->{agreement_id} = 2;
|
|
|
|
$t->post_ok(
|
|
"//$userid:$password@/api/v1/erm/agreements/$agreement_id" => json =>
|
|
$agreement_with_updated_field )->status_is(404);
|
|
|
|
$schema->storage->txn_rollback;
|
|
};
|
|
|
|
subtest 'delete() tests' => sub {
|
|
|
|
plan tests => 7;
|
|
|
|
$schema->storage->txn_begin;
|
|
|
|
my $librarian = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 2**28 }
|
|
}
|
|
);
|
|
my $password = 'thePassword123';
|
|
$librarian->set_password( { password => $password, skip_validation => 1 } );
|
|
my $userid = $librarian->userid;
|
|
|
|
my $patron = $builder->build_object(
|
|
{
|
|
class => 'Koha::Patrons',
|
|
value => { flags => 0 }
|
|
}
|
|
);
|
|
|
|
$patron->set_password( { password => $password, skip_validation => 1 } );
|
|
my $unauth_userid = $patron->userid;
|
|
|
|
my $agreement_id =
|
|
$builder->build_object( { class => 'Koha::ERM::Agreements' } )->agreement_id;
|
|
|
|
# Unauthorized attempt to delete
|
|
$t->delete_ok(
|
|
"//$unauth_userid:$password@/api/v1/erm/agreements/$agreement_id")
|
|
->status_is(403);
|
|
|
|
# Delete existing agreement
|
|
$t->delete_ok("//$userid:$password@/api/v1/erm/agreements/$agreement_id")
|
|
->status_is( 204, 'SWAGGER3.2.4' )->content_is( '', 'SWAGGER3.3.4' );
|
|
|
|
# Attempt to delete non-existent agreement
|
|
$t->delete_ok("//$userid:$password@/api/v1/erm/agreements/$agreement_id")
|
|
->status_is(404);
|
|
|
|
$schema->storage->txn_rollback;
|
|
};
|