From ae82953138aad2b07c6191ab5014b1abdb04aeda Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 25 Jun 2015 14:56:18 -0400 Subject: [PATCH] Bug 13853: (QA followup) Add unit tests for branch getting methods Signed-off-by: Jonathan Druart Signed-off-by: Tomas Cohen Arazi --- t/db_dependent/Items.t | 30 +++++++++++++++++++++++++++++- 1 file changed, 29 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/Items.t b/t/db_dependent/Items.t index b78e131426..c6444926ef 100755 --- a/t/db_dependent/Items.t +++ b/t/db_dependent/Items.t @@ -23,10 +23,11 @@ use C4::Biblio; use C4::Branch; use Koha::Database; -use Test::More tests => 6; +use Test::More tests => 8; BEGIN { use_ok('C4::Items'); + use_ok('Koha::Items'); } my $dbh = C4::Context->dbh; @@ -389,6 +390,33 @@ subtest 'SearchItems test' => sub { $dbh->rollback; }; +subtest 'Koha::Item(s) tests' => sub { + + plan tests => 5; + + # Start transaction + my $schema = Koha::Database->new()->schema(); + $schema->storage->txn_begin(); + $dbh->{RaiseError} = 1; + + # Create a biblio and item for testing + C4::Context->set_preference('marcflavour', 'MARC21'); + my ($bibnum, $bibitemnum) = get_biblio(); + my ($item_bibnum, $item_bibitemnum, $itemnumber) = AddItem({ homebranch => $branch1, holdingbranch => $branch2 } , $bibnum); + + # Get item. + my $item = Koha::Items->find( $itemnumber ); + ok( $item, "Got Koha::Item" ); + + my $homebranch = $item->home_branch(); + ok( $homebranch, "Got Koha::Branch from home_branch method" ); + is( $homebranch->branchcode(), $branch1, "Home branch code matches homebranch" ); + + my $holdingbranch = $item->holding_branch(); + ok( $holdingbranch, "Got Koha::Branch from holding_branch method" ); + is( $holdingbranch->branchcode(), $branch2, "Home branch code matches holdingbranch" ); +}; + # Helper method to set up a Biblio. sub get_biblio { my $bib = MARC::Record->new();