diff --git a/t/db_dependent/Koha/Patron.t b/t/db_dependent/Koha/Patron.t index 50d28e7d3c..196579a099 100755 --- a/t/db_dependent/Koha/Patron.t +++ b/t/db_dependent/Koha/Patron.t @@ -134,7 +134,7 @@ subtest 'relationships_debt() tests' => sub { sub _test_combinations { my ( $patrons, $parent1_debt, $parent2_debt, $child1_debt, $child2_debt ) = @_; - diag("Testing with parent 1 debt $parent1_debt | Parent 2 debt $parent2_debt | Child 1 debt $child1_debt | Child 2 debt $child2_debt"); + note("Testing with parent 1 debt $parent1_debt | Parent 2 debt $parent2_debt | Child 1 debt $child1_debt | Child 2 debt $child2_debt"); # Options # P1 => P1 + C1 + C2 ( - P1 ) ( + P2 ) # P2 => P2 + C1 + C2 ( - P2 ) ( + P1 ) @@ -145,7 +145,7 @@ sub _test_combinations { for my $i ( 0 .. 7 ) { my ( $only_this_guarantor, $include_guarantors, $include_this_patron ) = split '', sprintf( "%03b", $i ); - diag("---------------------"); + note("---------------------"); for my $patron ( @$patrons ) { if ( $only_this_guarantor && !$patron->guarantee_relationships->count ) diff --git a/t/db_dependent/SIP/Patron.t b/t/db_dependent/SIP/Patron.t index f0a1f26dce..15b8827da2 100755 --- a/t/db_dependent/SIP/Patron.t +++ b/t/db_dependent/SIP/Patron.t @@ -272,12 +272,16 @@ subtest "fine_items tests" => sub { is( @$fine_items, 0, "Got zero fine items" ); }; +$schema->storage->txn_rollback; + subtest "NoIssuesChargeGuarantorsWithGuarantees tests" => sub { plan tests => 1; t::lib::Mocks::mock_preference( 'borrowerRelationship', 'parent' ); + $schema->storage->txn_begin; + my $patron = $builder->build_object({ class => 'Koha::Patrons' }); my $child = $builder->build_object({ class => 'Koha::Patrons' }); $child->add_guarantor({ guarantor_id => $patron->borrowernumber, relationship => 'parent' }); @@ -308,5 +312,5 @@ subtest "NoIssuesChargeGuarantorsWithGuarantees tests" => sub { is( $sip_patron->fines_amount, 11.11,"Guarantee fines correctly included"); + $schema->storage->txn_rollback; }; -$schema->storage->txn_rollback;