Bug 18292: Remove return 1 statements in tests
[koha.git] / t / db_dependent / Koha / Patron / Messages.t
1 #!/usr/bin/perl
2
3 # Copyright 2015 Koha Development team
4 #
5 # This file is part of Koha
6 #
7 # Koha is free software; you can redistribute it and/or modify it
8 # under the terms of the GNU General Public License as published by
9 # the Free Software Foundation; either version 3 of the License, or
10 # (at your option) any later version.
11 #
12 # Koha is distributed in the hope that it will be useful, but
13 # WITHOUT ANY WARRANTY; without even the implied warranty of
14 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
15 # GNU General Public License for more details.
16 #
17 # You should have received a copy of the GNU General Public License
18 # along with Koha; if not, see <http://www.gnu.org/licenses>.
19
20 use Modern::Perl;
21
22 use Test::More tests => 11;
23
24 use C4::Context;
25 use C4::Log;
26 use Koha::Patron::Message;
27 use Koha::Patron::Messages;
28 use Koha::Patrons;
29 use Koha::Database;
30
31 use t::lib::Mocks;
32 use t::lib::TestBuilder;
33
34 my $schema = Koha::Database->new->schema;
35 $schema->storage->txn_begin;
36
37 my $builder        = t::lib::TestBuilder->new;
38 my $library        = $builder->build( { source => 'Branch' } );
39 my $patron         = $builder->build( { source => 'Borrower', values => { branchcode => $library->{branchcode} } } );
40 my $patron_2       = $builder->build( { source => 'Borrower' } );
41 my $nb_of_logaction = get_nb_of_logactions();
42 my $nb_of_messages = Koha::Patron::Messages->search->count;
43
44 t::lib::Mocks::mock_preference('BorrowersLog', 0);
45 my $new_message_1  = Koha::Patron::Message->new(
46     {   borrowernumber => $patron->{borrowernumber},
47         branchcode     => $library->{branchcode},
48         message_type   => 'L',
49         message        => 'my message 1',
50     }
51 )->store;
52 is( get_nb_of_logactions(), $nb_of_logaction, 'With BorrowersLog off, no new log should have been added' );
53
54 my $context = new Test::MockModule('C4::Context');
55 $context->mock( 'userenv', sub {
56     return {
57         number => $patron_2->{borrowernumber},
58     };
59 });
60
61 t::lib::Mocks::mock_preference('BorrowersLog', 1);
62 my $new_message_2  = Koha::Patron::Message->new(
63     {   borrowernumber => $patron->{borrowernumber},
64         branchcode     => $library->{branchcode},
65         message_type   => 'B',
66         message        => 'my message 2',
67     }
68 )->store;
69 is( get_nb_of_logactions(), $nb_of_logaction + 1, 'With BorrowersLog on, 1 new log should have been added when adding a new message' );
70
71 like( $new_message_1->message_id, qr|^\d+$|, 'Adding a new message should have set the message_id');
72 is( Koha::Patron::Messages->search->count, $nb_of_messages + 2, 'The 2 messages should have been added' );
73
74 my $retrieved_message_1 = Koha::Patron::Messages->find( $new_message_1->message_id );
75 is( $retrieved_message_1->message, $new_message_1->message, 'Find a message by id should return the correct message' );
76 is( $retrieved_message_1->manager_id, undef, 'Manager id should not be filled in when it is not defined in userenv' );
77 my $retrieved_message_2 = Koha::Patron::Messages->find( $new_message_2->message_id );
78 is( $retrieved_message_2->manager_id, $patron_2->{borrowernumber}, 'Manager id should be filled in when it is defined in userenv' );
79
80 t::lib::Mocks::mock_preference('BorrowersLog', 0);
81 $retrieved_message_1->delete;
82 is( Koha::Patron::Messages->search->count, $nb_of_messages + 1, 'Delete should have deleted the message 1' );
83 is( get_nb_of_logactions(), $nb_of_logaction + 1, 'With BorrowersLog off, no new log should have been added when deleting a new message' );
84
85 t::lib::Mocks::mock_preference('BorrowersLog', 1);
86 $new_message_2->delete;
87 is( Koha::Patron::Messages->search->count, $nb_of_messages, 'Delete should have deleted the message 2' );
88 is( get_nb_of_logactions(), $nb_of_logaction + 2, 'With BorrowersLog on, 1 new log should have been added when deleting a new message' );
89
90 $schema->storage->txn_rollback;
91
92 sub get_nb_of_logactions {
93     return scalar( @{ C4::Log::GetLogs( undef, undef, undef, ['MEMBERS'] ) } );
94 }
95