Browse Source
This patch overloads the to_api methods on the Fund class, so conflicting (on mapping) attribute names are not a problem. To test: 1. Apply this patch 2. Run: $ kshell k$ prove t/db_dependent/Koha/Acquisition/Fund.t \ t/db_dependent/Koha/REST/Plugin/Objects.t => SUCCESS: Tests pass! 3. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>20.05.x
2 changed files with 67 additions and 0 deletions
@ -0,0 +1,44 @@ |
|||
#!/usr/bin/perl |
|||
|
|||
# Copyright 2019 Koha Development team |
|||
# |
|||
# 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 => 1; |
|||
|
|||
use t::lib::TestBuilder; |
|||
|
|||
use Koha::Database; |
|||
|
|||
my $schema = Koha::Database->schema; |
|||
my $builder = t::lib::TestBuilder->new; |
|||
|
|||
subtest 'to_api() tests' => sub { |
|||
|
|||
plan tests => 2; |
|||
|
|||
$schema->storage->txn_begin; |
|||
|
|||
my $fund = $builder->build_object({ class => 'Koha::Acquisition::Funds' }); |
|||
my $fund_api = $fund->to_api(); |
|||
|
|||
is( $fund->budget_id, $fund_api->{fund_id}, 'Mapping is correct for budget_id' ); |
|||
is( $fund->budget_period_id, $fund_api->{budget_id}, 'Mapping is correct for budget_period_id' ); |
|||
|
|||
$schema->storage->txn_rollback; |
|||
}; |
Loading…
Reference in new issue