t/db_dependent/Letters.t

   1 #!/usr/bin/perl
   2
   3 # This file is part of Koha.
   4 #
   5 # Copyright (C) 2013 Equinox Software, Inc.
   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 use Test::More tests => 64;
  22 use Test::MockModule;
  23 use Test::Warn;
  24
  25 use MARC::Record;
  26
  27 my %mail;
  28 my $module = new Test::MockModule('Mail::Sendmail');
  29 $module->mock(
  30     'sendmail',
  31     sub {
  32         warn "Fake sendmail";
  33         %mail = @_;
  34     }
  35 );
  36
  37 use_ok('C4::Context');
  38 use_ok('C4::Members');
  39 use_ok('C4::Acquisition');
  40 use_ok('C4::Biblio');
  41 use_ok('C4::Letters');
  42 use t::lib::Mocks;
  43 use t::lib::TestBuilder;
  44 use Koha::Database;
  45 use Koha::DateUtils qw( dt_from_string output_pref );
  46 use Koha::Acquisition::Booksellers;
  47 use Koha::Acquisition::Bookseller::Contacts;
  48 use Koha::Acquisition::Orders;
  49 use Koha::Libraries;
  50 use Koha::Notice::Templates;
  51 use Koha::Patrons;
  52 use Koha::Subscriptions;
  53 my $schema = Koha::Database->schema;
  54 $schema->storage->txn_begin();
  55
  56 my $builder = t::lib::TestBuilder->new;
  57 my $dbh = C4::Context->dbh;
  58 $dbh->{RaiseError} = 1;
  59
  60 $dbh->do(q|DELETE FROM letter|);
  61 $dbh->do(q|DELETE FROM message_queue|);
  62 $dbh->do(q|DELETE FROM message_transport_types|);
  63
  64 my $library = $builder->build({
  65     source => 'Branch',
  66 });
  67 my $patron_category = $builder->build({ source => 'Category' })->{categorycode};
  68 my $date = dt_from_string;
  69 my $borrowernumber = AddMember(
  70     firstname    => 'Jane',
  71     surname      => 'Smith',
  72     categorycode => $patron_category,
  73     branchcode   => $library->{branchcode},
  74     dateofbirth  => $date,
  75     smsalertnumber => undef,
  76 );
  77
  78 my $marc_record = MARC::Record->new;
  79 my( $biblionumber, $biblioitemnumber ) = AddBiblio( $marc_record, '' );
  80
  81
  82
  83 # GetMessageTransportTypes
  84 my $mtts = C4::Letters::GetMessageTransportTypes();
  85 is( @$mtts, 0, 'GetMessageTransportTypes returns the correct number of message types' );
  86
  87 $dbh->do(q|
  88     INSERT INTO message_transport_types( message_transport_type ) VALUES ('email'), ('phone'), ('print'), ('sms')
  89 |);
  90 $mtts = C4::Letters::GetMessageTransportTypes();
  91 is_deeply( $mtts, ['email', 'phone', 'print', 'sms'], 'GetMessageTransportTypes returns all values' );
  92
  93
  94 # EnqueueLetter
  95 is( C4::Letters::EnqueueLetter(), undef, 'EnqueueLetter without argument returns undef' );
  96
  97 my $my_message = {
  98     borrowernumber         => $borrowernumber,
  99     message_transport_type => 'sms',
 100     to_address             => undef,
 101     from_address           => 'from@example.com',
 102 };
 103 my $message_id = C4::Letters::EnqueueLetter($my_message);
 104 is( $message_id, undef, 'EnqueueLetter without the letter argument returns undef' );
 105
 106 delete $my_message->{message_transport_type};
 107 $my_message->{letter} = {
 108     content      => 'a message',
 109     title        => 'message title',
 110     metadata     => 'metadata',
 111     code         => 'TEST_MESSAGE',
 112     content_type => 'text/plain',
 113 };
 114
 115 $message_id = C4::Letters::EnqueueLetter($my_message);
 116 is( $message_id, undef, 'EnqueueLetter without the message type argument argument returns undef' );
 117
 118 $my_message->{message_transport_type} = 'sms';
 119 $message_id = C4::Letters::EnqueueLetter($my_message);
 120 ok(defined $message_id && $message_id > 0, 'new message successfully queued');
 121
 122
 123 # GetQueuedMessages
 124 my $messages = C4::Letters::GetQueuedMessages();
 125 is( @$messages, 1, 'GetQueuedMessages without argument returns all the entries' );
 126
 127 $messages = C4::Letters::GetQueuedMessages({ borrowernumber => $borrowernumber });
 128 is( @$messages, 1, 'one message stored for the borrower' );
 129 is( $messages->[0]->{message_id}, $message_id, 'EnqueueLetter returns the message id correctly' );
 130 is( $messages->[0]->{borrowernumber}, $borrowernumber, 'EnqueueLetter stores the borrower number correctly' );
 131 is( $messages->[0]->{subject}, $my_message->{letter}->{title}, 'EnqueueLetter stores the subject correctly' );
 132 is( $messages->[0]->{content}, $my_message->{letter}->{content}, 'EnqueueLetter stores the content correctly' );
 133 is( $messages->[0]->{message_transport_type}, $my_message->{message_transport_type}, 'EnqueueLetter stores the message type correctly' );
 134 is( $messages->[0]->{status}, 'pending', 'EnqueueLetter stores the status pending correctly' );
 135
 136
 137 # SendQueuedMessages
 138 my $messages_processed = C4::Letters::SendQueuedMessages( { type => 'email' });
 139 is($messages_processed, 0, 'No queued messages processed if type limit passed with unused type');
 140 $messages_processed = C4::Letters::SendQueuedMessages( { type => 'sms' });
 141 is($messages_processed, 1, 'All queued messages processed, found correct number of messages with type limit');
 142 $messages = C4::Letters::GetQueuedMessages({ borrowernumber => $borrowernumber });
 143 is(
 144     $messages->[0]->{status},
 145     'failed',
 146     'message marked failed if tried to send SMS message for borrower with no smsalertnumber set (bug 11208)'
 147 );
 148
 149 # ResendMessage
 150 my $resent = C4::Letters::ResendMessage($messages->[0]->{message_id});
 151 my $message = C4::Letters::GetMessage( $messages->[0]->{message_id});
 152 is( $resent, 1, 'The message should have been resent' );
 153 is($message->{status},'pending', 'ResendMessage sets status to pending correctly (bug 12426)');
 154 $resent = C4::Letters::ResendMessage($messages->[0]->{message_id});
 155 is( $resent, 0, 'The message should not have been resent again' );
 156 $resent = C4::Letters::ResendMessage();
 157 is( $resent, undef, 'ResendMessage should return undef if not message_id given' );
 158
 159 # GetLetters
 160 my $letters = C4::Letters::GetLetters();
 161 is( @$letters, 0, 'GetLetters returns the correct number of letters' );
 162
 163 my $title = q|<<branches.branchname>> - <<status>>|;
 164 my $content = q{Dear <<borrowers.firstname>> <<borrowers.surname>>,
 165 According to our current records, you have items that are overdue.Your library does not charge late fines, but please return or renew them at the branch below as soon as possible.
 166
 167 <<branches.branchname>>
 168 <<branches.branchaddress1>>
 169 URL: <<OPACBaseURL>>
 170
 171 The following item(s) is/are currently <<status>>:
 172
 173 <item> <<count>>. <<items.itemcallnumber>>, Barcode: <<items.barcode>> </item>
 174
 175 Thank-you for your prompt attention to this matter.
 176 Don't forget your date of birth: <<borrowers.dateofbirth>>.
 177 Look at this wonderful biblio timestamp: <<biblio.timestamp>>.
 178 };
 179
 180 $dbh->do( q|INSERT INTO letter(branchcode,module,code,name,is_html,title,content,message_transport_type) VALUES (?,'my module','my code','my name',1,?,?,'email')|, undef, $library->{branchcode}, $title, $content );
 181 $letters = C4::Letters::GetLetters();
 182 is( @$letters, 1, 'GetLetters returns the correct number of letters' );
 183 is( $letters->[0]->{module}, 'my module', 'GetLetters gets the module correctly' );
 184 is( $letters->[0]->{code}, 'my code', 'GetLetters gets the code correctly' );
 185 is( $letters->[0]->{name}, 'my name', 'GetLetters gets the name correctly' );
 186
 187
 188 # getletter
 189 subtest 'getletter' => sub {
 190     plan tests => 16;
 191     t::lib::Mocks::mock_preference('IndependentBranches', 0);
 192     my $letter = C4::Letters::getletter('my module', 'my code', $library->{branchcode}, 'email');
 193     is( $letter->{branchcode}, $library->{branchcode}, 'GetLetters gets the branch code correctly' );
 194     is( $letter->{module}, 'my module', 'GetLetters gets the module correctly' );
 195     is( $letter->{code}, 'my code', 'GetLetters gets the code correctly' );
 196     is( $letter->{name}, 'my name', 'GetLetters gets the name correctly' );
 197     is( $letter->{is_html}, 1, 'GetLetters gets the boolean is_html correctly' );
 198     is( $letter->{title}, $title, 'GetLetters gets the title correctly' );
 199     is( $letter->{content}, $content, 'GetLetters gets the content correctly' );
 200     is( $letter->{message_transport_type}, 'email', 'GetLetters gets the message type correctly' );
 201
 202     my $context = Test::MockModule->new('C4::Context');
 203     $context->mock( 'userenv', sub {
 204         return {
 205             flags  => 1,
 206             branch => "anotherlib" }
 207     });
 208
 209     t::lib::Mocks::mock_preference('IndependentBranches', 1);
 210     $letter = C4::Letters::getletter('my module', 'my code', $library->{branchcode}, 'email');
 211     is( $letter->{branchcode}, $library->{branchcode}, 'GetLetters gets the branch code correctly' );
 212     is( $letter->{module}, 'my module', 'GetLetters gets the module correctly' );
 213     is( $letter->{code}, 'my code', 'GetLetters gets the code correctly' );
 214     is( $letter->{name}, 'my name', 'GetLetters gets the name correctly' );
 215     is( $letter->{is_html}, 1, 'GetLetters gets the boolean is_html correctly' );
 216     is( $letter->{title}, $title, 'GetLetters gets the title correctly' );
 217     is( $letter->{content}, $content, 'GetLetters gets the content correctly' );
 218     is( $letter->{message_transport_type}, 'email', 'GetLetters gets the message type correctly' );
 219
 220     $context->unmock('userenv');
 221 };
 222
 223
 224
 225 # Regression test for Bug 14206
 226 $dbh->do( q|INSERT INTO letter(branchcode,module,code,name,is_html,title,content,message_transport_type) VALUES ('FFL','my module','my code','my name',1,?,?,'print')|, undef, $title, $content );
 227 my $letter14206_a = C4::Letters::getletter('my module', 'my code', 'FFL' );
 228 is( $letter14206_a->{message_transport_type}, 'print', 'Bug 14206 - message_transport_type not passed, correct mtt detected' );
 229 my $letter14206_b = C4::Letters::getletter('my module', 'my code', 'FFL', 'print');
 230 is( $letter14206_b->{message_transport_type}, 'print', 'Bug 14206 - message_transport_type passed, correct mtt detected'  );
 231
 232 # test for overdue_notices.pl
 233 my $overdue_rules = {
 234     letter1         => 'my code',
 235 };
 236 my $i = 1;
 237 my $branchcode = 'FFL';
 238 my $letter14206_c = C4::Letters::getletter('my module', $overdue_rules->{"letter$i"}, $branchcode);
 239 is( $letter14206_c->{message_transport_type}, 'print', 'Bug 14206 - correct mtt detected for call from overdue_notices.pl' );
 240
 241 # GetPreparedLetter
 242 t::lib::Mocks::mock_preference('OPACBaseURL', 'http://thisisatest.com');
 243
 244 my $sms_content = 'This is a SMS for an <<status>>';
 245 $dbh->do( q|INSERT INTO letter(branchcode,module,code,name,is_html,title,content,message_transport_type) VALUES (?,'my module','my code','my name',1,'my title',?,'sms')|, undef, $library->{branchcode}, $sms_content );
 246
 247 my $tables = {
 248     borrowers => $borrowernumber,
 249     branches => $library->{branchcode},
 250     biblio => $biblionumber,
 251 };
 252 my $substitute = {
 253     status => 'overdue',
 254 };
 255 my $repeat = [
 256     {
 257         itemcallnumber => 'my callnumber1',
 258         barcode        => '1234',
 259     },
 260     {
 261         itemcallnumber => 'my callnumber2',
 262         barcode        => '5678',
 263     },
 264 ];
 265 my $prepared_letter = GetPreparedLetter((
 266     module      => 'my module',
 267     branchcode  => $library->{branchcode},
 268     letter_code => 'my code',
 269     tables      => $tables,
 270     substitute  => $substitute,
 271     repeat      => $repeat,
 272 ));
 273 my $retrieved_library = Koha::Libraries->find($library->{branchcode});
 274 my $my_title_letter = $retrieved_library->branchname . qq| - $substitute->{status}|;
 275 my $biblio_timestamp = dt_from_string( GetBiblioData($biblionumber)->{timestamp} );
 276 my $my_content_letter = qq|Dear Jane Smith,
 277 According to our current records, you have items that are overdue.Your library does not charge late fines, but please return or renew them at the branch below as soon as possible.
 278
 279 |.$retrieved_library->branchname.qq|
 280 |.$retrieved_library->branchaddress1.qq|
 281 URL: http://thisisatest.com
 282
 283 The following item(s) is/are currently $substitute->{status}:
 284
 285 <item> 1. $repeat->[0]->{itemcallnumber}, Barcode: $repeat->[0]->{barcode} </item>
 286 <item> 2. $repeat->[1]->{itemcallnumber}, Barcode: $repeat->[1]->{barcode} </item>
 287
 288 Thank-you for your prompt attention to this matter.
 289 Don't forget your date of birth: | . output_pref({ dt => $date, dateonly => 1 }) . q|.
 290 Look at this wonderful biblio timestamp: | . output_pref({ dt => $biblio_timestamp })  . ".\n";
 291
 292 is( $prepared_letter->{title}, $my_title_letter, 'GetPreparedLetter returns the title correctly' );
 293 is( $prepared_letter->{content}, $my_content_letter, 'GetPreparedLetter returns the content correctly' );
 294
 295 $prepared_letter = GetPreparedLetter((
 296     module                 => 'my module',
 297     branchcode             => $library->{branchcode},
 298     letter_code            => 'my code',
 299     tables                 => $tables,
 300     substitute             => $substitute,
 301     repeat                 => $repeat,
 302     message_transport_type => 'sms',
 303 ));
 304 $my_content_letter = qq|This is a SMS for an $substitute->{status}|;
 305 is( $prepared_letter->{content}, $my_content_letter, 'GetPreparedLetter returns the content correctly' );
 306
 307 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('test_date','TEST_DATE','Test dates','A title with a timestamp: <<biblio.timestamp>>','This one only contains the date: <<biblio.timestamp | dateonly>>.');});
 308 $prepared_letter = GetPreparedLetter((
 309     module                 => 'test_date',
 310     branchcode             => '',
 311     letter_code            => 'test_date',
 312     tables                 => $tables,
 313     substitute             => $substitute,
 314     repeat                 => $repeat,
 315 ));
 316 is( $prepared_letter->{content}, q|This one only contains the date: | . output_pref({ dt => $date, dateonly => 1 }) . q|.|, 'dateonly test 1' );
 317
 318 $dbh->do(q{UPDATE letter SET content = 'And also this one:<<timestamp | dateonly>>.' WHERE code = 'test_date';});
 319 $prepared_letter = GetPreparedLetter((
 320     module                 => 'test_date',
 321     branchcode             => '',
 322     letter_code            => 'test_date',
 323     tables                 => $tables,
 324     substitute             => $substitute,
 325     repeat                 => $repeat,
 326 ));
 327 is( $prepared_letter->{content}, q|And also this one:| . output_pref({ dt => $date, dateonly => 1 }) . q|.|, 'dateonly test 2' );
 328
 329 $dbh->do(q{UPDATE letter SET content = 'And also this one:<<timestamp|dateonly >>.' WHERE code = 'test_date';});
 330 $prepared_letter = GetPreparedLetter((
 331     module                 => 'test_date',
 332     branchcode             => '',
 333     letter_code            => 'test_date',
 334     tables                 => $tables,
 335     substitute             => $substitute,
 336     repeat                 => $repeat,
 337 ));
 338 is( $prepared_letter->{content}, q|And also this one:| . output_pref({ dt => $date, dateonly => 1 }) . q|.|, 'dateonly test 3' );
 339
 340 t::lib::Mocks::mock_preference( 'TimeFormat', '12hr' );
 341 my $yesterday_night = $date->clone->add( days => -1 )->set_hour(22);
 342 $dbh->do(q|UPDATE biblio SET timestamp = ? WHERE biblionumber = ?|, undef, $yesterday_night, $biblionumber );
 343 $dbh->do(q{UPDATE letter SET content = 'And also this one:<<timestamp>>.' WHERE code = 'test_date';});
 344 $prepared_letter = GetPreparedLetter((
 345     module                 => 'test_date',
 346     branchcode             => '',
 347     letter_code            => 'test_date',
 348     tables                 => $tables,
 349     substitute             => $substitute,
 350     repeat                 => $repeat,
 351 ));
 352 is( $prepared_letter->{content}, q|And also this one:| . output_pref({ dt => $yesterday_night }) . q|.|, 'dateonly test 3' );
 353
 354 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('claimacquisition','TESTACQCLAIM','Acquisition Claim','Item Not Received','<<aqbooksellers.name>>|<<aqcontacts.name>>|<order>Ordernumber <<aqorders.ordernumber>> (<<biblio.title>>) (<<aqorders.quantity>> ordered)</order>');});
 355 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('orderacquisition','TESTACQORDER','Acquisition Order','Order','<<aqbooksellers.name>>|<<aqcontacts.name>>|<order>Ordernumber <<aqorders.ordernumber>> (<<biblio.title>>) (<<aqorders.quantity>> ordered)</order>');});
 356
 357 # Test that _parseletter doesn't modify its parameters bug 15429
 358 {
 359     my $values = { dateexpiry => '2015-12-13', };
 360     C4::Letters::_parseletter($prepared_letter, 'borrowers', $values);
 361     is( $values->{dateexpiry}, '2015-12-13', "_parseletter doesn't modify its parameters" );
 362 }
 363
 364 my $bookseller = Koha::Acquisition::Bookseller->new(
 365     {
 366         name => "my vendor",
 367         address1 => "bookseller's address",
 368         phone => "0123456",
 369         active => 1,
 370         deliverytime => 5,
 371     }
 372 )->store;
 373 my $booksellerid = $bookseller->id;
 374
 375 Koha::Acquisition::Bookseller::Contact->new( { name => 'John Smith',  phone => '0123456x1', claimacquisition => 1, orderacquisition => 1, booksellerid => $booksellerid } )->store;
 376 Koha::Acquisition::Bookseller::Contact->new( { name => 'Leo Tolstoy', phone => '0123456x2', claimissues      => 1, booksellerid => $booksellerid } )->store;
 377 my $basketno = NewBasket($booksellerid, 1);
 378
 379 my $budgetid = C4::Budgets::AddBudget({
 380     budget_code => "budget_code_test_letters",
 381     budget_name => "budget_name_test_letters",
 382 });
 383
 384 my $bib = MARC::Record->new();
 385 if (C4::Context->preference('marcflavour') eq 'UNIMARC') {
 386     $bib->append_fields(
 387         MARC::Field->new('200', ' ', ' ', a => 'Silence in the library'),
 388     );
 389 } else {
 390     $bib->append_fields(
 391         MARC::Field->new('245', ' ', ' ', a => 'Silence in the library'),
 392     );
 393 }
 394
 395 ($biblionumber, $biblioitemnumber) = AddBiblio($bib, '');
 396 my $order = Koha::Acquisition::Order->new(
 397     {
 398         basketno => $basketno,
 399         quantity => 1,
 400         biblionumber => $biblionumber,
 401         budget_id => $budgetid,
 402     }
 403 )->store;
 404 my $ordernumber = $order->ordernumber;
 405
 406 C4::Acquisition::CloseBasket( $basketno );
 407 my $err;
 408 warning_like {
 409     $err = SendAlerts( 'claimacquisition', [ $ordernumber ], 'TESTACQCLAIM' ) }
 410     qr/^Bookseller .* without emails at/,
 411     "SendAlerts prints a warning";
 412 is($err->{'error'}, 'no_email', "Trying to send an alert when there's no e-mail results in an error");
 413
 414 $bookseller = Koha::Acquisition::Booksellers->find( $booksellerid );
 415 $bookseller->contacts->next->email('testemail@mydomain.com')->store;
 416
 417 # Ensure that the preference 'LetterLog' is set to logging
 418 t::lib::Mocks::mock_preference( 'LetterLog', 'on' );
 419
 420 # SendAlerts needs branchemail or KohaAdminEmailAddress as sender
 421 C4::Context->_new_userenv('DUMMY');
 422 C4::Context->set_userenv( 0, 0, 0, 'firstname', 'surname', $library->{branchcode}, 'My Library', 0, '', '');
 423 t::lib::Mocks::mock_preference( 'KohaAdminEmailAddress', 'library@domain.com' );
 424
 425 {
 426 warning_is {
 427     $err = SendAlerts( 'orderacquisition', $basketno , 'TESTACQORDER' ) }
 428     "Fake sendmail",
 429     "SendAlerts is using the mocked sendmail routine (orderacquisition)";
 430 is($err, 1, "Successfully sent order.");
 431 is($mail{'To'}, 'testemail@mydomain.com', "mailto correct in sent order");
 432 is($mail{'Message'}, 'my vendor|John Smith|Ordernumber ' . $ordernumber . ' (Silence in the library) (1 ordered)', 'Order notice text constructed successfully');
 433
 434 $dbh->do(q{DELETE FROM letter WHERE code = 'TESTACQORDER';});
 435 warning_like {
 436     $err = SendAlerts( 'orderacquisition', $basketno , 'TESTACQORDER' ) }
 437     qr/No orderacquisition TESTACQORDER letter transported by email/,
 438     "GetPreparedLetter warns about missing notice template";
 439 is($err->{'error'}, 'no_letter', "No TESTACQORDER letter was defined.");
 440 }
 441
 442 {
 443 warning_is {
 444     $err = SendAlerts( 'claimacquisition', [ $ordernumber ], 'TESTACQCLAIM' ) }
 445     "Fake sendmail",
 446     "SendAlerts is using the mocked sendmail routine";
 447
 448 is($err, 1, "Successfully sent claim");
 449 is($mail{'To'}, 'testemail@mydomain.com', "mailto correct in sent claim");
 450 is($mail{'Message'}, 'my vendor|John Smith|Ordernumber ' . $ordernumber . ' (Silence in the library) (1 ordered)', 'Claim notice text constructed successfully');
 451 }
 452
 453 {
 454 use C4::Serials;
 455
 456 my $notes = 'notes';
 457 my $internalnotes = 'intnotes';
 458 $dbh->do(q|UPDATE subscription_numberpatterns SET numberingmethod='No. {X}' WHERE id=1|);
 459 my $subscriptionid = NewSubscription(
 460      undef,      "",     undef, undef, undef, $biblionumber,
 461     '2013-01-01', 1, undef, undef,  undef,
 462     undef,      undef,  undef, undef, undef, undef,
 463     1,          $notes,undef, '2013-01-01', undef, 1,
 464     undef,       undef,  0,    $internalnotes,  0,
 465     undef, undef, 0,          undef,         '2013-12-31', 0
 466 );
 467 $dbh->do(q{INSERT INTO letter (module, code, name, title, content) VALUES ('serial','RLIST','Serial issue notification','Serial issue notification','<<biblio.title>>,<<subscription.subscriptionid>>,<<serial.serialseq>>');});
 468 my ($serials_count, @serials) = GetSerials($subscriptionid);
 469 my $serial = $serials[0];
 470
 471 my $borrowernumber = AddMember(
 472     firstname    => 'John',
 473     surname      => 'Smith',
 474     categorycode => $patron_category,
 475     branchcode   => $library->{branchcode},
 476     dateofbirth  => $date,
 477     email        => 'john.smith@test.de',
 478 );
 479 my $subscription = Koha::Subscriptions->find( $subscriptionid );
 480 $subscription->add_subscriber( scalar Koha::Patrons->find( $borrowernumber ) );
 481
 482 my $err2;
 483 warning_is {
 484 $err2 = SendAlerts( 'issue', $serial->{serialid}, 'RLIST' ) }
 485     "Fake sendmail",
 486     "SendAlerts is using the mocked sendmail routine";
 487 is($err2, 1, "Successfully sent serial notification");
 488 is($mail{'To'}, 'john.smith@test.de', "mailto correct in sent serial notification");
 489 is($mail{'Message'}, 'Silence in the library,'.$subscriptionid.',No. 0', 'Serial notification text constructed successfully');
 490 }
 491
 492 subtest 'GetPreparedLetter' => sub {
 493     plan tests => 4;
 494
 495     Koha::Notice::Template->new(
 496         {
 497             module                 => 'test',
 498             code                   => 'test',
 499             branchcode             => '',
 500             message_transport_type => 'email'
 501         }
 502     )->store;
 503     my $letter;
 504     warning_like {
 505         $letter = C4::Letters::GetPreparedLetter(
 506             module      => 'test',
 507             letter_code => 'test',
 508         );
 509     }
 510     qr{^ERROR: nothing to substitute},
 511 'GetPreparedLetter should warn if tables, substiture and repeat are not set';
 512     is( $letter, undef,
 513 'No letter should be returned by GetPreparedLetter if something went wrong'
 514     );
 515
 516     warning_like {
 517         $letter = C4::Letters::GetPreparedLetter(
 518             module      => 'test',
 519             letter_code => 'test',
 520             substitute  => {}
 521         );
 522     }
 523     qr{^ERROR: nothing to substitute},
 524 'GetPreparedLetter should warn if tables, substiture and repeat are not set, even if the key is passed';
 525     is( $letter, undef,
 526 'No letter should be returned by GetPreparedLetter if something went wrong'
 527     );
 528
 529 };
 530
 531
 532
 533 subtest 'TranslateNotices' => sub {
 534     plan tests => 4;
 535
 536     t::lib::Mocks::mock_preference( 'TranslateNotices', '1' );
 537
 538     $dbh->do(
 539         q|
 540         INSERT INTO letter (module, code, branchcode, name, title, content, message_transport_type, lang) VALUES
 541         ('test', 'code', '', 'test', 'a test', 'just a test', 'email', 'default'),
 542         ('test', 'code', '', 'test', 'una prueba', 'solo una prueba', 'email', 'es-ES');
 543     | );
 544     my $substitute = {};
 545     my $letter = C4::Letters::GetPreparedLetter(
 546             module                 => 'test',
 547             tables                 => $tables,
 548             letter_code            => 'code',
 549             message_transport_type => 'email',
 550             substitute             => $substitute,
 551     );
 552     is(
 553         $letter->{title},
 554         'a test',
 555         'GetPreparedLetter should return the default one if the lang parameter is not provided'
 556     );
 557
 558     $letter = C4::Letters::GetPreparedLetter(
 559             module                 => 'test',
 560             tables                 => $tables,
 561             letter_code            => 'code',
 562             message_transport_type => 'email',
 563             substitute             => $substitute,
 564             lang                   => 'es-ES',
 565     );
 566     is( $letter->{title}, 'una prueba',
 567         'GetPreparedLetter should return the required notice if it exists' );
 568
 569     $letter = C4::Letters::GetPreparedLetter(
 570             module                 => 'test',
 571             tables                 => $tables,
 572             letter_code            => 'code',
 573             message_transport_type => 'email',
 574             substitute             => $substitute,
 575             lang                   => 'fr-FR',
 576     );
 577     is(
 578         $letter->{title},
 579         'a test',
 580         'GetPreparedLetter should return the default notice if the one required does not exist'
 581     );
 582
 583     t::lib::Mocks::mock_preference( 'TranslateNotices', '' );
 584
 585     $letter = C4::Letters::GetPreparedLetter(
 586             module                 => 'test',
 587             tables                 => $tables,
 588             letter_code            => 'code',
 589             message_transport_type => 'email',
 590             substitute             => $substitute,
 591             lang                   => 'es-ES',
 592     );
 593     is( $letter->{title}, 'a test',
 594         'GetPreparedLetter should return the default notice if pref disabled but additional language exists' );
 595
 596 };
 597
 598 subtest 'SendQueuedMessages' => sub {
 599
 600     plan tests => 4;
 601     t::lib::Mocks::mock_preference( 'SMSSendDriver', 'Email' );
 602     my $patron = Koha::Patrons->find($borrowernumber);
 603     $dbh->do(q|
 604         INSERT INTO message_queue(borrowernumber, subject, content, message_transport_type, status, letter_code)
 605         VALUES (?, 'subject', 'content', 'sms', 'pending', 'just_a_code')
 606         |, undef, $borrowernumber
 607     );
 608     eval { C4::Letters::SendQueuedMessages(); };
 609     is( $@, '', 'SendQueuedMessages should not explode if the patron does not have a sms provider set' );
 610
 611     my $sms_pro = $builder->build_object({ class => 'Koha::SMS::Providers', value => { domain => 'kidclamp.rocks' } });
 612     ModMember( borrowernumber => $borrowernumber, smsalertnumber => '5555555555', sms_provider_id => $sms_pro->id() );
 613     $message_id = C4::Letters::EnqueueLetter($my_message); #using datas set around line 95 and forward
 614     C4::Letters::SendQueuedMessages();
 615     my $sms_message_address = $schema->resultset('MessageQueue')->search({
 616         borrowernumber => $borrowernumber,
 617         status => 'sent'
 618     })->next()->to_address();
 619     is( $sms_message_address, '5555555555@kidclamp.rocks', 'SendQueuedMessages populates the to address correctly for SMS by email when to_address not set' );
 620     $schema->resultset('MessageQueue')->search({borrowernumber => $borrowernumber,status => 'sent'})->delete(); #clear borrower queue
 621     $my_message->{to_address} = 'fixme@kidclamp.iswrong';
 622     $message_id = C4::Letters::EnqueueLetter($my_message);
 623
 624     my $number_attempted = C4::Letters::SendQueuedMessages({
 625         borrowernumber => -1, # -1 still triggers the borrowernumber condition
 626         letter_code    => 'PASSWORD_RESET',
 627     });
 628     is ( $number_attempted, 0, 'There were no password reset messages for SendQueuedMessages to attempt.' );
 629
 630     C4::Letters::SendQueuedMessages();
 631     $sms_message_address = $schema->resultset('MessageQueue')->search({
 632         borrowernumber => $borrowernumber,
 633         status => 'sent'
 634     })->next()->to_address();
 635     is( $sms_message_address, '5555555555@kidclamp.rocks', 'SendQueuedMessages populates the to address correctly for SMS by email when to_address is set incorrectly' );
 636
 637 };
 638
 639 subtest 'get_item_content' => sub {
 640     plan tests => 2;
 641
 642     t::lib::Mocks::mock_preference('dateformat', 'metric');
 643     t::lib::Mocks::mock_preference('timeformat', '24hr');
 644     my @items = (
 645         {date_due => '2041-01-01 12:34', title => 'a first title', barcode => 'a_first_barcode', author => 'a_first_author', itemnumber => 1 },
 646         {date_due => '2042-01-02 23:45', title => 'a second title', barcode => 'a_second_barcode', author => 'a_second_author', itemnumber => 2 },
 647     );
 648     my @item_content_fields = qw( date_due title barcode author itemnumber );
 649
 650     my $items_content;
 651     for my $item ( @items ) {
 652         $items_content .= C4::Letters::get_item_content( { item => $item, item_content_fields => \@item_content_fields } );
 653     }
 654
 655     my $expected_items_content = <<EOF;
 656 01/01/2041 12:34\ta first title\ta_first_barcode\ta_first_author\t1
 657 02/01/2042 23:45\ta second title\ta_second_barcode\ta_second_author\t2
 658 EOF
 659     is( $items_content, $expected_items_content, 'get_item_content should return correct items info with time (default)' );
 660
 661
 662     $items_content = q||;
 663     for my $item ( @items ) {
 664         $items_content .= C4::Letters::get_item_content( { item => $item, item_content_fields => \@item_content_fields, dateonly => 1, } );
 665     }
 666
 667     $expected_items_content = <<EOF;
 668 01/01/2041\ta first title\ta_first_barcode\ta_first_author\t1
 669 02/01/2042\ta second title\ta_second_barcode\ta_second_author\t2
 670 EOF
 671     is( $items_content, $expected_items_content, 'get_item_content should return correct items info without time (if dateonly => 1)' );
 672 };
 673
 674 subtest 'Test limit parameter for SendQueuedMessages' => sub {
 675     plan tests => 3;
 676
 677     my $dbh = C4::Context->dbh;
 678
 679     my $borrowernumber = AddMember(
 680         firstname    => 'Jane',
 681         surname      => 'Smith',
 682         categorycode => $patron_category,
 683         branchcode   => $library->{branchcode},
 684         dateofbirth  => $date,
 685         smsalertnumber => undef,
 686     );
 687
 688     $dbh->do(q|DELETE FROM message_queue|);
 689     $my_message = {
 690         'letter' => {
 691             'content'      => 'a message',
 692             'metadata'     => 'metadata',
 693             'code'         => 'TEST_MESSAGE',
 694             'content_type' => 'text/plain',
 695             'title'        => 'message title'
 696         },
 697         'borrowernumber'         => $borrowernumber,
 698         'to_address'             => undef,
 699         'message_transport_type' => 'sms',
 700         'from_address'           => 'from@example.com'
 701     };
 702     C4::Letters::EnqueueLetter($my_message);
 703     C4::Letters::EnqueueLetter($my_message);
 704     C4::Letters::EnqueueLetter($my_message);
 705     C4::Letters::EnqueueLetter($my_message);
 706     C4::Letters::EnqueueLetter($my_message);
 707     my $messages_processed = C4::Letters::SendQueuedMessages( { limit => 1 } );
 708     is( $messages_processed, 1,
 709         'Processed 1 message with limit of 1 and 5 unprocessed messages' );
 710     $messages_processed = C4::Letters::SendQueuedMessages( { limit => 2 } );
 711     is( $messages_processed, 2,
 712         'Processed 2 message with limit of 2 and 4 unprocessed messages' );
 713     $messages_processed = C4::Letters::SendQueuedMessages( { limit => 3 } );
 714     is( $messages_processed, 2,
 715         'Processed 2 message with limit of 3 and 2 unprocessed messages' );
 716 };