Bug 15601: Fix TestBuilder tests
authorJonathan Druart <jonathan.druart@bugs.koha-community.org>
Tue, 19 Jan 2016 15:10:42 +0000 (15:10 +0000)
committerBrendan Gallagher <brendan@bywatersolutions.com>
Sat, 23 Jan 2016 19:01:41 +0000 (19:01 +0000)
Bug 13624 modified the DB structure for overduerules.
The TestBuilder tests (t/db_dependent/TestBuilder.t) based some of its
checks on this structure.

Test plan:
  prove t/db_dependent/TestBuilder.t
Should return green.

Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>

Signed-off-by: Brendan Gallagher brendan@bywatersolutions.com

t/db_dependent/TestBuilder.t

index 638d2ef..a3cb813 100644 (file)
@@ -19,7 +19,7 @@
 
 use Modern::Perl;
 
-use Test::More tests => 42;
+use Test::More tests => 41;
 
 use Koha::Database;
 
@@ -49,8 +49,7 @@ my $my_overduerules_transport_type = {
     message_transport_type => {
         message_transport_type => 'my msg_t_t',
     },
-    letternumber => 1,
-    branchcode   => {
+    overduerules_id => {
         branchcode   => 'codeB',
         categorycode => 'codeC',
     },
@@ -67,11 +66,6 @@ is(
     'build stores the message_transport_type correctly'
 );
 is(
-    $overduerules_transport_type->{letternumber},
-    $my_overduerules_transport_type->{letternumber},
-    'build stores the letternumber correctly'
-);
-is(
     $overduerules_transport_type->{branchcode},
     $my_overduerules_transport_type->{branchcode}->{branchcode},
     'build stores the branchcode correctly'
@@ -102,7 +96,7 @@ is_deeply(
     'build links the branchcode and the categorycode correctly'
 );
 isnt(
-    $overduerules_transport_type->{_fk}->{branchcode}->{letter2},
+    $overduerules_transport_type->{_fk}->{overduerules_id}->{letter2},
     undef,
     'build generates values if they are not given'
 );