t/db_dependent/Koha/Patrons/Import.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 use Test::More tests => 159;
  22 use Test::Warn;
  23 use Encode qw( encode_utf8 );
  24 use utf8;
  25
  26 # To be replaced by t::lib::Mock
  27 use Test::MockModule;
  28 use Koha::Database;
  29
  30 use File::Temp qw(tempfile tempdir);
  31 my $temp_dir = tempdir('Koha_patrons_import_test_XXXX', CLEANUP => 1, TMPDIR => 1);
  32
  33 use t::lib::Mocks;
  34 use t::lib::TestBuilder;
  35 my $builder = t::lib::TestBuilder->new;
  36
  37 my $schema = Koha::Database->new->schema;
  38 $schema->storage->txn_begin;
  39
  40 # ########## Tests start here #############################
  41 # Given ... we can use the module
  42 BEGIN { use_ok('Koha::Patrons::Import'); }
  43
  44 my $patrons_import = new_ok('Koha::Patrons::Import');
  45
  46 subtest 'test_methods' => sub {
  47     plan tests => 1;
  48
  49     # Given ... we can reach the method(s)
  50     my @methods = ('import_patrons',
  51                    'set_attribute_types',
  52                    'prepare_columns',
  53                    'set_column_keys',
  54                    'generate_patron_attributes',
  55                    'check_branch_code',
  56                    'format_dates',
  57                   );
  58     can_ok('Koha::Patrons::Import', @methods);
  59 };
  60
  61 subtest 'test_attributes' => sub {
  62     plan tests => 1;
  63
  64     my @attributes = ('today_iso', 'text_csv');
  65     can_ok('Koha::Patrons::Import', @attributes);
  66 };
  67
  68 # Tests for Koha::Patrons::Import::import_patrons()
  69 # Given ... nothing much. When ... Then ...
  70 my $result;
  71 warning_is { $result = $patrons_import->import_patrons(undef) }
  72            { carped => 'No file handle passed in!' },
  73            " Koha::Patrons::Import->import_patrons carps if no file handle is passed";
  74 is($result, undef, 'Got the expected undef from import_patrons with nothing much');
  75
  76 # Given ... some params but no file handle.
  77 my $params_0 = { some_stuff => 'random stuff', };
  78
  79 # When ... Then ...
  80 my $result_0;
  81 warning_is { $result_0 = $patrons_import->import_patrons($params_0) }
  82            { carped => 'No file handle passed in!' },
  83            " Koha::Patrons::Import->import_patrons carps if no file handle is passed";
  84 is($result_0, undef, 'Got the expected undef from import_patrons with no file handle');
  85
  86 # Given ... a file handle to file with headers only.
  87 t::lib::Mocks::mock_preference('ExtendedPatronAttributes', 0);
  88 t::lib::Mocks::mock_preference('dateformat', 'us');
  89
  90
  91 my $csv_headers  = 'cardnumber,surname,firstname,title,othernames,initials,streetnumber,streettype,address,address2,city,state,zipcode,country,email,phone,mobile,fax,dateofbirth,branchcode,categorycode,dateenrolled,dateexpiry,userid,password';
  92 my $res_header   = 'cardnumber, surname, firstname, title, othernames, initials, streetnumber, streettype, address, address2, city, state, zipcode, country, email, phone, mobile, fax, dateofbirth, branchcode, categorycode, dateenrolled, dateexpiry, userid, password';
  93 my $csv_one_line = '1000,Nancy,Jenkins,Dr,,NJ,78,Circle,Bunting,El Paso,Henderson,Texas,79984,United States,ajenkins0@sourceforge.net,7-(388)559-6763,3-(373)151-4471,8-(509)286-4001,10/16/1965,CPL,PT,12/28/2014,07/01/2015,jjenkins0,DPQILy';
  94 my $csv_one_line_a = '1001,Nancy,Jenkins,Dr,,NJ,78,Circle,Bunting,El Paso,Henderson,Texas,79984,United States,ajenkins0@sourceforge.net,7-(388)559-6763,3-(373)151-4471,8-(509)286-4001,10/16/1965,CPL,PT,12/28/2014,07/01/2015,jjenkins0,DPQILy';
  95
  96 my $filename_1 = make_csv($temp_dir, $csv_headers, $csv_one_line);
  97 open(my $handle_1, "<", $filename_1) or die "cannot open < $filename_1: $!";
  98 my $params_1 = { file => $handle_1, };
  99
 100 # When ...
 101 my $result_1 = $patrons_import->import_patrons($params_1);
 102
 103 # Then ...
 104 is($result_1->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons with no matchpoint defined');
 105 is(scalar @{$result_1->{errors}}, 0, 'Got the expected 0 size error array from import_patrons with no matchpoint defined');
 106
 107 is($result_1->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons with no matchpoint defined');
 108 is($result_1->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons with no matchpoint defined');
 109 is($result_1->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons with no matchpoint defined');
 110
 111 is($result_1->{feedback}->[1]->{feedback}, 1, 'Got the expected second feedback from import_patrons with no matchpoint defined');
 112 is($result_1->{feedback}->[1]->{name}, 'lastimported', 'Got the expected last imported name from import_patrons with no matchpoint defined');
 113 like($result_1->{feedback}->[1]->{value}, qr/^Nancy \/ \d+/, 'Got the expected second header row value from import_patrons with no matchpoint defined');
 114
 115 is($result_1->{imported}, 1, 'Got the expected 1 imported result from import_patrons with no matchpoint defined');
 116 is($result_1->{invalid}, 0, 'Got the expected 0 invalid result from import_patrons with no matchpoint defined');
 117 is($result_1->{overwritten}, 0, 'Got the expected 0 overwritten result from import_patrons with no matchpoint defined');
 118
 119 # Given ... a valid file handle, a bad matchpoint resulting in invalid card number
 120 my $filename_2 = make_csv($temp_dir, $csv_headers, $csv_one_line);
 121 open(my $handle_2, "<", $filename_2) or die "cannot open < $filename_2: $!";
 122 my $params_2 = { file => $handle_2, matchpoint => 'SHOW_BCODE', };
 123
 124 # When ...
 125 my $result_2 = $patrons_import->import_patrons($params_2);
 126
 127 # Then ...
 128 is($result_2->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons with invalid card number');
 129 is($result_2->{errors}->[0]->{borrowernumber}, undef, 'Got the expected undef borrower number from import patrons with invalid card number');
 130 is($result_2->{errors}->[0]->{cardnumber}, 1000, 'Got the expected 1000 card number from import patrons with invalid card number');
 131 is($result_2->{errors}->[0]->{invalid_cardnumber}, 1, 'Got the expected invalid card number from import patrons with invalid card number');
 132
 133 is($result_2->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons with invalid card number');
 134 is($result_2->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons with invalid card number');
 135 is($result_2->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons with invalid card number');
 136
 137 is($result_2->{imported}, 0, 'Got the expected 0 imported result from import_patrons with invalid card number');
 138 is($result_2->{invalid}, 1, 'Got the expected 1 invalid result from import_patrons with invalid card number');
 139 is($result_2->{overwritten}, 0, 'Got the expected 0 overwritten result from import_patrons with invalid card number');
 140
 141 # Given ... valid file handle, good matchpoint that matches should not overwrite when not set.
 142 my $filename_3 = make_csv($temp_dir, $csv_headers, $csv_one_line);
 143 open(my $handle_3, "<", $filename_3) or die "cannot open < $filename_3: $!";
 144 my $params_3 = { file => $handle_3, matchpoint => 'cardnumber', };
 145
 146 # When ...
 147 my $result_3 = $patrons_import->import_patrons($params_3);
 148
 149 # Then ...
 150 is($result_3->{already_in_db}, 1, 'Got the expected 1 already_in_db from import_patrons with duplicate userid');
 151 is($result_3->{errors}->[0]->{duplicate_userid}, undef, 'No duplicate userid error from import patrons with duplicate userid (it is our own)');
 152 is($result_3->{errors}->[0]->{userid}, undef, 'No duplicate userid error from import patrons with duplicate userid (it is our own)');
 153
 154 is($result_3->{feedback}->[0]->{feedback}, 1, 'Got 1 expected feedback from import_patrons that matched but not overwritten');
 155 is($result_3->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons with duplicate userid');
 156 is($result_3->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons with duplicate userid');
 157
 158 is($result_3->{imported}, 0, 'Got the expected 0 imported result from import_patrons');
 159 is($result_3->{invalid}, 0, 'Got the expected 0 invalid result from import_patrons');
 160 is($result_3->{overwritten}, 0, 'Got the expected 0 overwritten result from import_patrons that matched');
 161
 162 # Given ... valid file handle, good matchpoint that matches should overwrite when set.
 163 my $filename_3a = make_csv($temp_dir, $csv_headers, $csv_one_line);
 164 open(my $handle_3a, "<", $filename_3a) or die "cannot open < $filename_3: $!";
 165 my $params_3a = { file => $handle_3a, matchpoint => 'cardnumber', overwrite_cardnumber => 1};
 166
 167 # When ...
 168 my $result_3a;
 169 warning_is { $result_3a = $patrons_import->import_patrons($params_3a) }
 170            undef,
 171            "No warning raised by import_patrons";
 172
 173 # Then ...
 174 is($result_3a->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons when matched and overwrite set');
 175 is($result_3a->{errors}->[0]->{duplicate_userid}, undef, 'No duplicate userid error from import patrons with duplicate userid (it is our own)');
 176 is($result_3a->{errors}->[0]->{userid}, undef, 'No duplicate userid error from import patrons with duplicate userid (it is our own)');
 177
 178 is($result_3a->{feedback}->[0]->{feedback}, 1, 'Got 1 expected feedback from import_patrons that matched and overwritten');
 179 is($result_3a->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons with duplicate userid');
 180 is($result_3a->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons with duplicate userid');
 181
 182 is($result_3a->{imported}, 0, 'Got the expected 0 imported result from import_patrons');
 183 is($result_3a->{invalid}, 0, 'Got the expected 0 invalid result from import_patrons');
 184 is($result_3a->{overwritten}, 1, 'Got the expected 1 overwritten result from import_patrons that matched');
 185
 186 # Given ... valid file handle, good matchpoint that does not match and conflicting userid.
 187 my $filename_3b = make_csv($temp_dir, $csv_headers, $csv_one_line_a);
 188 open(my $handle_3b, "<", $filename_3b) or die "cannot open < $filename_3: $!";
 189 my $params_3b = { file => $handle_3b, matchpoint => 'cardnumber', };
 190
 191 # When ...
 192 my $result_3b = $patrons_import->import_patrons($params_3b);
 193
 194 # Then ...
 195 is($result_3b->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons with duplicate userid');
 196 is($result_3b->{errors}->[0]->{duplicate_userid}, 1, 'Got the expected duplicate userid error from import patrons with duplicate userid');
 197 is($result_3b->{errors}->[0]->{userid}, 'jjenkins0', 'Got the expected userid error from import patrons with duplicate userid');
 198
 199 is($result_3b->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons with duplicate userid');
 200 is($result_3b->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons with duplicate userid');
 201 is($result_3b->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons with duplicate userid');
 202
 203 is($result_3b->{imported}, 0, 'Got the expected 0 imported result from import_patrons with duplicate userid');
 204 is($result_3b->{invalid}, 1, 'Got the expected 1 invalid result from import_patrons with duplicate userid');
 205 is($result_3b->{overwritten}, 0, 'Got the expected 0 overwritten result from import_patrons with duplicate userid');
 206
 207 # Given ... a new input and mocked C4::Context
 208 t::lib::Mocks::mock_preference('ExtendedPatronAttributes', 1);
 209 my $attribute = $builder->build({ source => "BorrowerAttributeType"});
 210
 211 my $csv_headers_a  = 'cardnumber,surname,firstname,title,othernames,initials,streetnumber,streettype,address,address2,city,state,zipcode,country,email,phone,mobile,fax,dateofbirth,branchcode,categorycode,dateenrolled,dateexpiry,userid,password,patron_attributes';
 212 my $res_header_a   = 'cardnumber, surname, firstname, title, othernames, initials, streetnumber, streettype, address, address2, city, state, zipcode, country, email, phone, mobile, fax, dateofbirth, branchcode, categorycode, dateenrolled, dateexpiry, userid, password, patron_attributes';
 213 my $new_input_line = '1001,Donna,Sullivan,Mrs,Henry,DS,59,Court,Burrows,Reading,Salt Lake City,Pennsylvania,19605,United States,hsullivan1@purevolume.com,3-(864)009-3006,7-(291)885-8423,1-(879)095-5038,09/19/1970,LPL,PT,03/04/2015,07/01/2015,hsullivan1,8j6P6Dmap,'.$attribute->{code}.':1';
 214 my $filename_4 = make_csv($temp_dir, $csv_headers_a, $new_input_line);
 215 open(my $handle_4, "<", $filename_4) or die "cannot open < $filename_4: $!";
 216 my $params_4 = { file => $handle_4, matchpoint => $attribute->{code}, };
 217
 218 # When ...
 219 my $result_4 = $patrons_import->import_patrons($params_4);
 220
 221 # Then ...
 222 is($result_4->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons with extended user');
 223 is(scalar @{$result_4->{errors}}, 0, 'Got the expected 0 size error array from import_patrons with extended user');
 224
 225 is($result_4->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons with extended user');
 226 is($result_4->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons with extended user');
 227 is($result_4->{feedback}->[0]->{value}, $res_header_a, 'Got the expected header row value from import_patrons with extended user');
 228
 229 is($result_4->{feedback}->[1]->{feedback}, 1, 'Got the expected second feedback from import_patrons with extended user');
 230 is($result_4->{feedback}->[1]->{name}, 'attribute string', 'Got the expected attribute string from import_patrons with extended user');
 231 is($result_4->{feedback}->[1]->{value}, $attribute->{code}.':1', 'Got the expected second feedback value from import_patrons with extended user');
 232
 233 is($result_4->{feedback}->[2]->{feedback}, 1, 'Got the expected third feedback from import_patrons with extended user');
 234 is($result_4->{feedback}->[2]->{name}, 'lastimported', 'Got the expected last imported name from import_patrons with extended user');
 235 like($result_4->{feedback}->[2]->{value}, qr/^Donna \/ \d+/, 'Got the expected third feedback value from import_patrons with extended user');
 236
 237 is($result_4->{imported}, 1, 'Got the expected 1 imported result from import_patrons with extended user');
 238 is($result_4->{invalid}, 0, 'Got the expected 0 invalid result from import_patrons with extended user');
 239 is($result_4->{overwritten}, 0, 'Got the expected 0 overwritten result from import_patrons with extended user');
 240
 241 seek $handle_4,0,0; #Reset to verify finding a matched patron works
 242 my $result_4a = $patrons_import->import_patrons($params_4);
 243 is($result_4a->{already_in_db}, 1, 'Got the expected 1 already_in_db from import_patrons with extended user matched');
 244 is(scalar @{$result_4->{errors}}, 0, 'Got the expected 0 size error array from import_patrons with extended user matched');
 245
 246 is($result_4a->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons with extended user matched');
 247 is($result_4a->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons with extended user matched');
 248 is($result_4a->{feedback}->[0]->{value}, $res_header_a, 'Got the expected header row value from import_patrons with extended user matched');
 249
 250 is($result_4a->{feedback}->[1]->{feedback}, 1, 'Got the expected second feedback from import_patrons with extended user matched');
 251 is($result_4a->{feedback}->[1]->{name}, 'attribute string', 'Got the expected attribute string from import_patrons with extended user matched');
 252 is($result_4a->{feedback}->[1]->{value}, $attribute->{code}.':1', 'Got the expected second feedback value from import_patrons with extended user matched');
 253
 254 is($result_4a->{feedback}->[2]->{already_in_db}, '1', 'Got the expected already_in_db from import_patrons with extended user matched');
 255 like($result_4a->{feedback}->[2]->{value}, qr/^Donna \/ \d+/, 'Got the expected third feedback value from import_patrons with extended user matched');
 256
 257 is($result_4a->{imported}, 0, 'Got the expected 0 imported result from import_patrons with extended user matched');
 258 is($result_4a->{invalid}, 0, 'Got the expected 0 invalid result from import_patrons with extended user matched');
 259 is($result_4a->{overwritten}, 0, 'Got the expected 0 overwritten result from import_patrons with extended user matched');
 260
 261 t::lib::Mocks::mock_preference('ExtendedPatronAttributes', 0);
 262
 263 my $surname ='Chloé❤';
 264 # Given ... 3 new inputs. One with no branch code, one with unexpected branch code.
 265 my $input_no_branch   = qq|1002,$surname,Reynolds,Mr,Patricia,JR,12,Hill,Kennedy,Saint Louis,Colorado Springs,Missouri,63131,United States,preynolds2i\@washington.edu,7-(925)314-9514,0-(315)973-8956,4-(510)556-2323,09/18/1967,,PT,05/07/2015,07/01/2015,preynolds2,K3HiDzl|;
 266 my $input_good_branch = qq|1003,$surname,Richardson,Mr,Kimberly,LR,90,Place,Bayside,Atlanta,Erie,Georgia,31190,United States,krichardson3\@pcworld.com,8-(035)185-0387,4-(796)518-3676,3-(644)960-3789,04/13/1954,RPL,PT,06/06/2015,07/01/2015,krichardson3,P3EO0MVRPXbM|;
 267 my $input_na_branch   = qq|1005,$surname,Greene,Mr,Michael,RG,3,Avenue,Grim,Peoria,Jacksonville,Illinois,61614,United States,mgreene5\@seesaa.net,3-(941)565-5752,1-(483)885-8138,4-(979)577-6908,02/09/1957,ZZZ,ST,04/02/2015,07/01/2015,mgreene5,or4ORT6JH|;
 268
 269 my $filename_5 = make_csv($temp_dir, $csv_headers, encode_utf8($input_no_branch), encode_utf8($input_good_branch), encode_utf8($input_na_branch));
 270 open(my $handle_5, "<", $filename_5) or die "cannot open < $filename_5: $!";
 271 my $params_5 = { file => $handle_5, matchpoint => 'cardnumber', };
 272
 273 # When ...
 274 my $result_5 = $patrons_import->import_patrons($params_5);
 275
 276 # Then ...
 277 is($result_5->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons for branch tests');
 278
 279 is($result_5->{errors}->[0]->{missing_criticals}->[0]->{borrowernumber}, 'UNDEF', 'Got the expected undef borrower number error from import patrons for branch tests');
 280 is($result_5->{errors}->[0]->{missing_criticals}->[0]->{key}, 'branchcode', 'Got the expected branch code key from import patrons for branch tests');
 281 is($result_5->{errors}->[0]->{missing_criticals}->[0]->{line}, 2, 'Got the expected 2 line number error from import patrons for branch tests');
 282 is($result_5->{errors}->[0]->{missing_criticals}->[0]->{lineraw}, $input_no_branch."\r\n", 'Got the expected lineraw error from import patrons for branch tests');
 283 is($result_5->{errors}->[0]->{missing_criticals}->[0]->{surname}, $surname, 'Got the expected surname error from import patrons for branch tests');
 284
 285 is($result_5->{errors}->[1]->{missing_criticals}->[0]->{borrowernumber}, 'UNDEF', 'Got the expected undef borrower number error from import patrons for branch tests');
 286 is($result_5->{errors}->[1]->{missing_criticals}->[0]->{branch_map}, 1, 'Got the expected 1 branchmap error from import patrons for branch tests');
 287 is($result_5->{errors}->[1]->{missing_criticals}->[0]->{key}, 'branchcode', 'Got the expected branch code key from import patrons for branch tests');
 288 is($result_5->{errors}->[1]->{missing_criticals}->[0]->{line}, 4, 'Got the expected 4 line number error from import patrons for branch tests');
 289 is($result_5->{errors}->[1]->{missing_criticals}->[0]->{lineraw}, $input_na_branch."\r\n", 'Got the expected lineraw error from import patrons for branch tests');
 290 is($result_5->{errors}->[1]->{missing_criticals}->[0]->{surname}, $surname, 'Got the expected surname error from import patrons for branch tests');
 291 is($result_5->{errors}->[1]->{missing_criticals}->[0]->{value}, 'ZZZ', 'Got the expected ZZZ value error from import patrons for branch tests');
 292
 293 is($result_5->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons for branch tests');
 294 is($result_5->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons for branch tests');
 295 is($result_5->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons for branch tests');
 296
 297 is($result_5->{feedback}->[1]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons for branch tests');
 298 is($result_5->{feedback}->[1]->{name}, 'lastimported', 'Got the expected lastimported name from import_patrons for branch tests');
 299 like($result_5->{feedback}->[1]->{value},  qr/^$surname \/ \d+/, 'Got the expected last imported value from import_patrons with for branch tests');
 300
 301 is($result_5->{imported}, 1, 'Got the expected 1 imported result from import patrons for branch tests');
 302 is($result_5->{invalid}, 2, 'Got the expected 2 invalid result from import patrons for branch tests');
 303 is($result_5->{overwritten}, 0, 'Got the expected 0 overwritten result from import patrons for branch tests');
 304
 305 # Given ... 3 new inputs. One with no category code, one with unexpected category code.
 306 my $input_no_category   = '1006,Christina,Olson,Rev,Kimberly,CO,8,Avenue,Northridge,Lexington,Wilmington,Kentucky,40510,United States,kolson6@dropbox.com,7-(810)636-6048,1-(052)012-8984,8-(567)232-7818,03/26/1952,FFL,,09/07/2014,01/07/2015,kolson6,x5D3qGbLlptx';
 307 my $input_good_category = '1007,Peter,Peters,Mrs,Lawrence,PP,6,Trail,South,Oklahoma City,Topeka,Oklahoma,73135,United States,lpeters7@bandcamp.com,5-(992)205-9318,0-(732)586-9365,3-(448)146-7936,08/16/1983,PVL,T,03/24/2015,07/01/2015,lpeters7,Z19BrQ4';
 308 my $input_na_category   = '1008,Emily,Richards,Ms,Judy,ER,73,Way,Kedzie,Fort Wayne,Phoenix,Indiana,46825,United States,jrichards8@arstechnica.com,5-(266)658-8957,3-(550)500-9107,7-(816)675-9822,08/09/1984,FFL,ZZ,11/09/2014,07/01/2015,jrichards8,D5PvU6H2R';
 309
 310 my $filename_6 = make_csv($temp_dir, $csv_headers, $input_no_category, $input_good_category, $input_na_category);
 311 open(my $handle_6, "<", $filename_6) or die "cannot open < $filename_6: $!";
 312 my $params_6 = { file => $handle_6, matchpoint => 'cardnumber', };
 313
 314 # When ...
 315 my $result_6 = $patrons_import->import_patrons($params_6);
 316
 317 # Then ...
 318 is($result_6->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons for category tests');
 319
 320 is($result_6->{errors}->[0]->{missing_criticals}->[0]->{borrowernumber}, 'UNDEF', 'Got the expected undef borrower number error from import patrons for category tests');
 321 is($result_6->{errors}->[0]->{missing_criticals}->[0]->{key}, 'categorycode', 'Got the expected category code key from import patrons for category tests');
 322 is($result_6->{errors}->[0]->{missing_criticals}->[0]->{line}, 2, 'Got the expected 2 line number error from import patrons for category tests');
 323 is($result_6->{errors}->[0]->{missing_criticals}->[0]->{lineraw}, $input_no_category."\r\n", 'Got the expected lineraw error from import patrons for category tests');
 324 is($result_6->{errors}->[0]->{missing_criticals}->[0]->{surname}, 'Christina', 'Got the expected surname error from import patrons for category tests');
 325
 326 is($result_6->{errors}->[1]->{missing_criticals}->[0]->{borrowernumber}, 'UNDEF', 'Got the expected undef borrower number error from import patrons for category tests');
 327 is($result_6->{errors}->[1]->{missing_criticals}->[0]->{category_map}, 1, 'Got the expected 1 category_map error from import patrons for category tests');
 328 is($result_6->{errors}->[1]->{missing_criticals}->[0]->{key}, 'categorycode', 'Got the expected category code key from import patrons for category tests');
 329 is($result_6->{errors}->[1]->{missing_criticals}->[0]->{line}, 4, 'Got the expected 4 line number error from import patrons for category tests');
 330 is($result_6->{errors}->[1]->{missing_criticals}->[0]->{lineraw}, $input_na_category."\r\n", 'Got the expected lineraw error from import patrons for category tests');
 331 is($result_6->{errors}->[1]->{missing_criticals}->[0]->{surname}, 'Emily', 'Got the expected surname error from import patrons for category tests');
 332 is($result_6->{errors}->[1]->{missing_criticals}->[0]->{value}, 'ZZ', 'Got the expected ZZ value error from import patrons for category tests');
 333
 334 is($result_6->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons for category tests');
 335 is($result_6->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons for category tests');
 336 is($result_6->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons for category tests');
 337
 338 is($result_6->{feedback}->[1]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons for category tests');
 339 is($result_6->{feedback}->[1]->{name}, 'lastimported', 'Got the expected lastimported name from import_patrons for category tests');
 340 like($result_6->{feedback}->[1]->{value},  qr/^Peter \/ \d+/, 'Got the expected last imported value from import_patrons with for category tests');
 341
 342 is($result_6->{imported}, 1, 'Got the expected 1 imported result from import patrons for category tests');
 343 is($result_6->{invalid}, 2, 'Got the expected 2 invalid result from import patrons for category tests');
 344 is($result_6->{overwritten}, 0, 'Got the expected 0 overwritten result from import patrons for category tests');
 345
 346 # Given ... 2 new inputs. One without dateofbirth, dateenrolled and dateexpiry values.
 347 my $input_complete = '1009,Christina,Harris,Dr,Philip,CH,99,Street,Grayhawk,Baton Rouge,Dallas,Louisiana,70810,United States,pharris9@hp.com,9-(317)603-5513,7-(005)062-7593,8-(349)134-1627,06/19/1969,IPT,PT,04/09/2015,07/01/2015,pharris9,NcAhcvvnB';
 348 my $input_no_date  = '1010,Ralph,Warren,Ms,Linda,RW,6,Way,Barby,Orlando,Albany,Florida,32803,United States,lwarrena@multiply.com,7-(579)753-7752,6-(847)086-7566,9-(122)729-8226,26/01/2001,LPL,T,25/01/2001,24/01/2001,lwarrena,tJ56RD4uV';
 349
 350 my $filename_7 = make_csv($temp_dir, $csv_headers, $input_complete, $input_no_date);
 351 open(my $handle_7, "<", $filename_7) or die "cannot open < $filename_7: $!";
 352 my $params_7 = { file => $handle_7, matchpoint => 'cardnumber', };
 353
 354 # When ...
 355 my $result_7 = $patrons_import->import_patrons($params_7);
 356
 357 # Then ...
 358 is($result_7->{already_in_db}, 0, 'Got the expected 0 already_in_db from import_patrons for dates tests');
 359 is(scalar @{$result_7->{errors}}, 1, 'Got the expected 1 error array size from import_patrons for dates tests');
 360 is(scalar @{$result_7->{errors}->[0]->{missing_criticals}}, 3, 'Got the expected 3 missing critical errors from import_patrons for dates tests');
 361
 362 is($result_7->{errors}->[0]->{missing_criticals}->[0]->{bad_date}, 1, 'Got the expected 1 bad_date error from import patrons for dates tests');
 363 is($result_7->{errors}->[0]->{missing_criticals}->[0]->{borrowernumber}, 'UNDEF', 'Got the expected undef borrower number error from import patrons for dates tests');
 364 is($result_7->{errors}->[0]->{missing_criticals}->[0]->{key}, 'dateofbirth', 'Got the expected dateofbirth key from import patrons for dates tests');
 365 is($result_7->{errors}->[0]->{missing_criticals}->[0]->{line}, 3, 'Got the expected 2 line number error from import patrons for dates tests');
 366 is($result_7->{errors}->[0]->{missing_criticals}->[0]->{lineraw}, $input_no_date."\r\n", 'Got the expected lineraw error from import patrons for dates tests');
 367 is($result_7->{errors}->[0]->{missing_criticals}->[0]->{surname}, 'Ralph', 'Got the expected surname error from import patrons for dates tests');
 368
 369 is($result_7->{errors}->[0]->{missing_criticals}->[1]->{key}, 'dateenrolled', 'Got the expected dateenrolled key from import patrons for dates tests');
 370 is($result_7->{errors}->[0]->{missing_criticals}->[2]->{key}, 'dateexpiry', 'Got the expected dateexpiry key from import patrons for dates tests');
 371
 372 is(scalar @{$result_7->{feedback}}, 2, 'Got the expected 2 feedback from import patrons for dates tests');
 373 is($result_7->{feedback}->[0]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons for dates tests');
 374 is($result_7->{feedback}->[0]->{name}, 'headerrow', 'Got the expected header row name from import_patrons for dates tests');
 375 is($result_7->{feedback}->[0]->{value}, $res_header, 'Got the expected header row value from import_patrons for dates tests');
 376
 377 is($result_7->{feedback}->[1]->{feedback}, 1, 'Got the expected 1 feedback from import_patrons for dates tests');
 378 is($result_7->{feedback}->[1]->{name}, 'lastimported', 'Got the expected lastimported from import_patrons for dates tests');
 379 like($result_7->{feedback}->[1]->{value}, qr/^Christina \/ \d+/, 'Got the expected lastimported value from import_patrons for dates tests');
 380
 381 is($result_7->{imported}, 1, 'Got the expected 1 imported result from import patrons for dates tests');
 382 is($result_7->{invalid}, 1, 'Got the expected 1 invalid result from import patrons for dates tests');
 383 is($result_7->{overwritten}, 0, 'Got the expected 0 overwritten result from import patrons for dates tests');
 384
 385 subtest 'test_import_without_cardnumber' => sub {
 386     plan tests => 2;
 387
 388     #Remove possible existing user with a "" as cardnumber
 389     my $blank_card = Koha::Patrons->find({ cardnumber => '' });
 390     $blank_card->delete if $blank_card;
 391
 392     my $branchcode = $builder->build({ source => "Branch"})->{branchcode};
 393     my $categorycode = $builder->build({ source => "Category"})->{categorycode};
 394     my $csv_headers  = 'surname, branchcode, categorycode';
 395     my $res_headers  = 'surname, branchcode, categorycode';
 396     my $csv_nocard_1 = "Squarepants,$branchcode,$categorycode";
 397     my $csv_nocard_2 = "Star,$branchcode,$categorycode";
 398
 399     my $filename_1 = make_csv($temp_dir, $csv_headers, $csv_nocard_1, $csv_nocard_2);
 400     open(my $handle_1, "<", $filename_1) or die "cannot open < $filename_1: $!";
 401     my $params_1 = { file => $handle_1, };
 402
 403     my $defaults = { cardnumber => "" }; #currently all the defaults come as "" if not filled
 404
 405     my $result = $patrons_import->import_patrons($params_1, $defaults);
 406     like($result->{feedback}->[1]->{value}, qr/^Squarepants \/ \d+/, 'First borrower imported as expected');
 407     like($result->{feedback}->[2]->{value}, qr/^Star \/ \d+/, 'Second borrower imported as expected');
 408
 409 };
 410
 411 subtest 'test_import_with_cardnumber_0' => sub {
 412     plan tests => 2;
 413
 414     #Remove possible existing user with a "" as cardnumber
 415     my $zero_card = Koha::Patrons->find({ cardnumber => 0 });
 416     $zero_card->delete if $zero_card;
 417
 418     my $branchcode = $builder->build({ source => "Branch"})->{branchcode};
 419     my $categorycode = $builder->build({ source => "Category"})->{categorycode};
 420     my $csv_headers  = 'cardnumber,surname, branchcode, categorycode';
 421     my $res_headers  = 'cardnumber,surname, branchcode, categorycode';
 422     my $csv_nocard_1 = "0,Squarepants,$branchcode,$categorycode";
 423
 424     my $filename_1 = make_csv($temp_dir, $csv_headers, $csv_nocard_1);
 425     open(my $handle_1, "<", $filename_1) or die "cannot open < $filename_1: $!";
 426     my $params_1 = { file => $handle_1, };
 427
 428     my $defaults = { cardnumber => "" }; #currently all the defaults come as "" if not filled
 429
 430     my $result = $patrons_import->import_patrons($params_1, $defaults);
 431     like($result->{feedback}->[1]->{value}, qr/^Squarepants \/ \d+/, 'First borrower imported as expected');
 432     $zero_card = Koha::Patrons->find({ cardnumber => 0 });
 433     is($zero_card->surname.$zero_card->branchcode.$zero_card->categorycode,'Squarepants'.$branchcode.$categorycode,"Patron with cardnumber 0 is the imported patron");
 434
 435 };
 436
 437 subtest 'test_import_with_password_overwrite' => sub {
 438     plan tests => 8;
 439
 440     #Remove possible existing user to avoid clashes
 441     my $ernest = Koha::Patrons->find({ userid => 'ErnestP' });
 442     $ernest->delete if $ernest;
 443
 444     #Setup our info
 445     my $branchcode = $builder->build({ source => "Branch"})->{branchcode};
 446     my $categorycode = $builder->build({ source => "Category", value => { category_type => 'A'  } })->{categorycode};
 447     my $staff_categorycode = $builder->build({ source => "Category", value => { category_type => 'S'  } })->{categorycode};
 448     my $csv_headers  = 'surname,userid,branchcode,categorycode,password';
 449     my $csv_password = "Worrell,ErnestP,$branchcode,$categorycode,Ernest11";
 450     my $csv_password_change = "Worrell,ErnestP,$branchcode,$categorycode,Vern1234";
 451     my $csv_blank_password = "Worel,ErnestP,$branchcode,$categorycode,";
 452     my $defaults = { cardnumber => "" }; #currently all the defaults come as "" if not filled
 453     my $csv_staff_password_change = "Worrell,ErnestP,$branchcode,$staff_categorycode,Vern1234";
 454
 455     #Make the test files for importing
 456     my $filename_1 = make_csv($temp_dir, $csv_headers, $csv_password);
 457     open(my $handle_1, "<", $filename_1) or die "cannot open < $filename_1: $!";
 458     my $params_1 = { file => $handle_1, matchpoint => 'userid', overwrite_passwords => 1, overwrite_cardnumber => 1};
 459     my $filename_2 = make_csv($temp_dir, $csv_headers, $csv_password_change);
 460     open(my $handle_2, "<", $filename_2) or die "cannot open < $filename_2: $!";
 461     my $params_2 = { file => $handle_2, matchpoint => 'userid', overwrite_passwords => 1, overwrite_cardnumber => 1};
 462
 463     my $filename_3 = make_csv($temp_dir, $csv_headers, $csv_blank_password);
 464     open(my $handle_3, "<", $filename_3) or die "cannot open < $filename_3: $!";
 465     my $params_3 = { file => $handle_3, matchpoint => 'userid', overwrite_passwords => 1, overwrite_cardnumber => 1};
 466
 467     my $filename_4 = make_csv($temp_dir, $csv_headers, $csv_staff_password_change);
 468     open(my $handle_4, "<", $filename_4) or die "cannot open < $filename_4: $!";
 469     my $params_4 = { file => $handle_4, matchpoint => 'userid', overwrite_passwords => 1, overwrite_cardnumber => 1};
 470
 471
 472     my $result = $patrons_import->import_patrons($params_1, $defaults);
 473     like($result->{feedback}->[1]->{value}, qr/^Worrell \/ \d+/, 'First borrower imported as expected');
 474     $ernest = Koha::Patrons->find({ userid => 'ErnestP' });
 475     isnt($ernest->password,'Ernest',"New patron is imported, password is encrypted");
 476
 477     #Save info to double check
 478     my $orig_pass = $ernest->password;
 479
 480     $result = $patrons_import->import_patrons($params_2, $defaults);
 481     $ernest = Koha::Patrons->find({ userid => 'ErnestP' });
 482     isnt($ernest->password,$orig_pass,"New patron is overwritten, password is overwritten");
 483     isnt($ernest->password,'Vern',"Password is overwritten and is encrypted from value provided");
 484
 485     #Save info to check not changed
 486     $orig_pass = $ernest->password;
 487
 488     $result = $patrons_import->import_patrons($params_3, $defaults);
 489     $ernest = Koha::Patrons->find({ userid => 'ErnestP' });
 490     is($ernest->surname,'Worel',"Patron is overwritten, surname changed");
 491     is($ernest->password,$orig_pass,"Patron was overwritten but password is not overwritten if blank");
 492
 493     $ernest->category($staff_categorycode);
 494     $ernest->store;
 495
 496     $result = $patrons_import->import_patrons($params_4, $defaults);
 497     $ernest = Koha::Patrons->find({ userid => 'ErnestP' });
 498     is($ernest->surname,'Worrell',"Patron is overwritten, surname changed");
 499     is($ernest->password,$orig_pass,"Patron is imported, password is not changed for staff");
 500
 501 };
 502
 503
 504 subtest 'test_prepare_columns' => sub {
 505     plan tests => 16;
 506
 507     # Given ... no header row
 508     my %csvkeycol_0;
 509     my @errors_0;
 510
 511     # When ...
 512     my @csvcolumns_0 = $patrons_import->prepare_columns({headerrow => undef, keycol => \%csvkeycol_0, errors => \@errors_0, });
 513
 514     # Then ...
 515     is(scalar @csvcolumns_0, 0, 'Got the expected empty column array from prepare columns with no header row');
 516
 517     is(scalar @errors_0, 1, 'Got the expected 1 entry in error array from prepare columns with no header row');
 518     is($errors_0[0]->{badheader}, 1, 'Got the expected 1 badheader from prepare columns with no header row');
 519     is($errors_0[0]->{line}, 1, 'Got the expected 1 line from prepare columns with no header row');
 520     is($errors_0[0]->{lineraw}, undef, 'Got the expected undef lineraw from prepare columns with no header row');
 521
 522     # Given ... a good header row with plenty of whitespaces
 523     my $headerrow_1 = 'a,    b ,        c,  ,   d';
 524     my %csvkeycol_1;
 525     my @errors_1;
 526
 527     # When ...
 528     my @csvcolumns_1 = $patrons_import->prepare_columns({headerrow => $headerrow_1, keycol => \%csvkeycol_1, errors => \@errors_1, });
 529
 530     # Then ...
 531     is(scalar @csvcolumns_1, 5, 'Got the expected 5 column array from prepare columns');
 532     is($csvcolumns_1[0], 'a', 'Got the expected a header from prepare columns');
 533     is($csvcolumns_1[1], 'b', 'Got the expected b header from prepare columns');
 534     is($csvcolumns_1[2], 'c', 'Got the expected c header from prepare columns');
 535     is($csvcolumns_1[3], '', 'Got the expected empty header from prepare columns');
 536     is($csvcolumns_1[4], 'd', 'Got the expected d header from prepare columns');
 537
 538     is($csvkeycol_1{a}, 0, 'Got the expected 0 value for key a from prepare columns hash');
 539     is($csvkeycol_1{b}, 1, 'Got the expected 1 value for key b from prepare columns hash');
 540     is($csvkeycol_1{c}, 2, 'Got the expected 2 value for key c from prepare columns hash');
 541     is($csvkeycol_1{''}, 3, 'Got the expected 3 value for empty string key from prepare columns hash');
 542     is($csvkeycol_1{d}, 4, 'Got the expected 4 value for key d from prepare columns hash');
 543 };
 544
 545 subtest 'test_set_column_keys' => sub {
 546     plan tests => 5;
 547
 548     # Given ... nothing at all
 549     # When ... Then ...
 550     my $attr_type_0 = $patrons_import->set_attribute_types(undef);
 551     is($attr_type_0, undef, 'Got the expected undef attribute type from set attribute types with nothing');
 552
 553     # Given ... extended but not matchpoint
 554     my $params_1 = { extended => 1, matchpoint => undef, };
 555
 556     # When ... Then ...
 557     my $attr_type_1 = $patrons_import->set_attribute_types($params_1);
 558     is($attr_type_1, undef, 'Got the expected undef attribute type from set attribute types with no matchpoint');
 559
 560     # Given ... extended and unexpected matchpoint
 561     my $params_2 = { extended => 1, matchpoint => 'unexpected', };
 562
 563     # When ... Then ...
 564     my $attr_type_2 = $patrons_import->set_attribute_types($params_2);
 565     is($attr_type_2, undef, 'Got the expected undef attribute type from set attribute types with unexpected matchpoint');
 566
 567     # Given ...
 568     my $code_3   = 'SHOW_BCODE';
 569     my $params_3 = { extended => 1, matchpoint => $code_3, };
 570
 571     # When ...
 572     my $attr_type_3 = $patrons_import->set_attribute_types($params_3);
 573
 574     # Then ...
 575     isa_ok($attr_type_3, 'Koha::Patron::Attribute::Type');
 576     is($attr_type_3->code, $code_3, 'Got the expected code attribute type from set attribute types');
 577 };
 578
 579 subtest 'test_set_column_keys' => sub {
 580     plan tests => 2;
 581
 582     my @columns = Koha::Patrons->columns;
 583     # Given ... nothing at all
 584     # When ... Then ...
 585     my @columnkeys_0 = $patrons_import->set_column_keys(undef);
 586     # -1 because we do not want the borrowernumber column
 587     is(scalar @columnkeys_0, @columns - 1, 'Got the expected array size from set column keys with undef extended');
 588
 589     # Given ... extended.
 590     my $extended = 1;
 591
 592     # When ... Then ...
 593     my @columnkeys_1 = $patrons_import->set_column_keys($extended);
 594     is(scalar @columnkeys_1, @columns - 1 + $extended, 'Got the expected array size from set column keys with extended');
 595 };
 596
 597 subtest 'test_generate_patron_attributes' => sub {
 598     plan tests => 13;
 599
 600     # Given ... nothing at all
 601     # When ... Then ...
 602     my $result_0 = $patrons_import->generate_patron_attributes(undef, undef, undef);
 603     is($result_0, undef, 'Got the expected undef from set patron attributes with nothing');
 604
 605     # Given ... not extended.
 606     my $extended_1 = 0;
 607
 608     # When ... Then ...
 609     my $result_1 = $patrons_import->generate_patron_attributes($extended_1, undef, undef);
 610     is($result_1, undef, 'Got the expected undef from set patron attributes with not extended');
 611
 612     # Given ... NO patrons attributes
 613     my $extended_2          = 1;
 614     my $patron_attributes_2 = undef;
 615     my @feedback_2;
 616
 617     # When ...
 618     my $result_2 = $patrons_import->generate_patron_attributes($extended_2, $patron_attributes_2, \@feedback_2);
 619
 620     # Then ...
 621     is($result_2, undef, 'Got the expected undef from set patron attributes with no patrons attributes');
 622     is(scalar @feedback_2, 0, 'Got the expected 0 size feedback array from set patron attributes with no patrons attributes');
 623
 624     # Given ... some patrons attributes
 625     my $patron_attributes_3 = "homeroom:1150605,grade:01";
 626     my @feedback_3;
 627
 628     # When ...
 629     my $result_3 = $patrons_import->generate_patron_attributes($extended_2, $patron_attributes_3, \@feedback_3);
 630
 631     # Then ...
 632     ok($result_3, 'Got some data back from set patron attributes');
 633     is($result_3->[0]->{code}, 'grade', 'Got the expected first code from set patron attributes');
 634     is($result_3->[0]->{attribute}, '01', 'Got the expected first value from set patron attributes');
 635
 636     is($result_3->[1]->{code}, 'homeroom', 'Got the expected second code from set patron attributes');
 637     is($result_3->[1]->{attribute}, 1150605, 'Got the expected second value from set patron attributes');
 638
 639     is(scalar @feedback_3, 1, 'Got the expected 1 array size from set patron attributes with extended user');
 640     is($feedback_3[0]->{feedback}, 1, 'Got the expected second feedback from set patron attributes with extended user');
 641     is($feedback_3[0]->{name}, 'attribute string', 'Got the expected attribute string from set patron attributes with extended user');
 642     is($feedback_3[0]->{value}, 'homeroom:1150605,grade:01', 'Got the expected feedback value from set patron attributes with extended user');
 643 };
 644
 645 subtest 'test_check_branch_code' => sub {
 646     plan tests => 11;
 647
 648     # Given ... no branch code.
 649     my $borrowerline      = 'some, line';
 650     my $line_number       = 78;
 651     my @missing_criticals = ();
 652
 653     # When ...
 654     $patrons_import->check_branch_code(undef, $borrowerline, $line_number, \@missing_criticals);
 655
 656     # Then ...
 657     is(scalar @missing_criticals, 1, 'Got the expected missing critical array size of 1 from check_branch_code with no branch code');
 658
 659     is($missing_criticals[0]->{key}, 'branchcode', 'Got the expected branchcode key from check_branch_code with no branch code');
 660     is($missing_criticals[0]->{line}, $line_number, 'Got the expected line number from check_branch_code with no branch code');
 661     is($missing_criticals[0]->{lineraw}, $borrowerline, 'Got the expected lineraw value from check_branch_code with no branch code');
 662
 663     # Given ... unknown branch code
 664     my $branchcode_1        = 'unexpected';
 665     my $borrowerline_1      = 'some, line,'.$branchcode_1;
 666     my $line_number_1       = 79;
 667     my @missing_criticals_1 = ();
 668
 669     # When ...
 670     $patrons_import->check_branch_code($branchcode_1, $borrowerline_1, $line_number_1, \@missing_criticals_1);
 671
 672     # Then ...
 673     is(scalar @missing_criticals_1, 1, 'Got the expected missing critical array size of 1 from check_branch_code with unexpected branch code');
 674
 675     is($missing_criticals_1[0]->{branch_map}, 1, 'Got the expected 1 branch_map from check_branch_code with unexpected branch code');
 676     is($missing_criticals_1[0]->{key}, 'branchcode', 'Got the expected branchcode key from check_branch_code with unexpected branch code');
 677     is($missing_criticals_1[0]->{line}, $line_number_1, 'Got the expected line number from check_branch_code with unexpected branch code');
 678     is($missing_criticals_1[0]->{lineraw}, $borrowerline_1, 'Got the expected lineraw value from check_branch_code with unexpected branch code');
 679     is($missing_criticals_1[0]->{value}, $branchcode_1, 'Got the expected value from check_branch_code with unexpected branch code');
 680
 681     # Given ... a known branch code. Relies on database sample data
 682     my $branchcode_2        = 'FFL';
 683     my $borrowerline_2      = 'some, line,'.$branchcode_2;
 684     my $line_number_2       = 80;
 685     my @missing_criticals_2 = ();
 686
 687     # When ...
 688     $patrons_import->check_branch_code($branchcode_2, $borrowerline_2, $line_number_2, \@missing_criticals_2);
 689
 690     # Then ...
 691     is(scalar @missing_criticals_2, 0, 'Got the expected missing critical array size of 0 from check_branch_code');
 692 };
 693
 694 subtest 'test_check_borrower_category' => sub {
 695     plan tests => 11;
 696
 697     # Given ... no category code.
 698     my $borrowerline      = 'some, line';
 699     my $line_number       = 781;
 700     my @missing_criticals = ();
 701
 702     # When ...
 703     $patrons_import->check_borrower_category(undef, $borrowerline, $line_number, \@missing_criticals);
 704
 705     # Then ...
 706     is(scalar @missing_criticals, 1, 'Got the expected missing critical array size of 1 from check_branch_code with no category code');
 707
 708     is($missing_criticals[0]->{key}, 'categorycode', 'Got the expected categorycode key from check_branch_code with no category code');
 709     is($missing_criticals[0]->{line}, $line_number, 'Got the expected line number from check_branch_code with no category code');
 710     is($missing_criticals[0]->{lineraw}, $borrowerline, 'Got the expected lineraw value from check_branch_code with no category code');
 711
 712     # Given ... unknown category code
 713     my $categorycode_1      = 'unexpected';
 714     my $borrowerline_1      = 'some, line, line, '.$categorycode_1;
 715     my $line_number_1       = 791;
 716     my @missing_criticals_1 = ();
 717
 718     # When ...
 719     $patrons_import->check_borrower_category($categorycode_1, $borrowerline_1, $line_number_1, \@missing_criticals_1);
 720
 721     # Then ...
 722     is(scalar @missing_criticals_1, 1, 'Got the expected missing critical array size of 1 from check_branch_code with unexpected category code');
 723
 724     is($missing_criticals_1[0]->{category_map}, 1, 'Got the expected 1 category_map from check_branch_code with unexpected category code');
 725     is($missing_criticals_1[0]->{key}, 'categorycode', 'Got the expected branchcode key from check_branch_code with unexpected category code');
 726     is($missing_criticals_1[0]->{line}, $line_number_1, 'Got the expected line number from check_branch_code with unexpected category code');
 727     is($missing_criticals_1[0]->{lineraw}, $borrowerline_1, 'Got the expected lineraw value from check_branch_code with unexpected category code');
 728     is($missing_criticals_1[0]->{value}, $categorycode_1, 'Got the expected value from check_branch_code with unexpected category code');
 729
 730     # Given ... a known category code. Relies on database sample data.
 731     my $categorycode_2      = 'T';
 732     my $borrowerline_2      = 'some, line,'.$categorycode_2;
 733     my $line_number_2       = 801;
 734     my @missing_criticals_2 = ();
 735
 736     # When ...
 737     $patrons_import->check_borrower_category($categorycode_2, $borrowerline_2, $line_number_2, \@missing_criticals_2);
 738
 739     # Then ...
 740     is(scalar @missing_criticals_2, 0, 'Got the expected missing critical array size of 0 from check_branch_code');
 741 };
 742
 743 subtest 'test_format_dates' => sub {
 744     plan tests => 22;
 745
 746     # Given ... no borrower data.
 747     my $borrowerline      = 'another line';
 748     my $line_number       = 987;
 749     my @missing_criticals = ();
 750     my %borrower;
 751     my $params = {borrower => \%borrower, lineraw => $borrowerline, line => $line_number, missing_criticals => \@missing_criticals, };
 752
 753     # When ...
 754     $patrons_import->format_dates($params);
 755
 756     # Then ...
 757     ok( not(%borrower), 'Got the expected no borrower from format_dates with no dates');
 758     is(scalar @missing_criticals, 0, 'Got the expected missing critical array size of 0 from format_dates with no dates');
 759
 760     # Given ... some good dates
 761     my @missing_criticals_1 = ();
 762     my $dateofbirth_1  = '2016-05-03';
 763     my $dateenrolled_1 = '2016-05-04';
 764     my $dateexpiry_1   = '2016-05-06';
 765     my $borrower_1     = { dateofbirth => $dateofbirth_1, dateenrolled => $dateenrolled_1, dateexpiry => $dateexpiry_1, };
 766     my $params_1       = {borrower => $borrower_1, lineraw => $borrowerline, line => $line_number, missing_criticals => \@missing_criticals_1, };
 767
 768     # When ...
 769     $patrons_import->format_dates($params_1);
 770
 771     # Then ...
 772     is($borrower_1->{dateofbirth}, $dateofbirth_1, 'Got the expected date of birth from format_dates with good dates');
 773     is($borrower_1->{dateenrolled}, $dateenrolled_1, 'Got the expected date of birth from format_dates with good dates');
 774     is($borrower_1->{dateexpiry}, $dateexpiry_1, 'Got the expected date of birth from format_dates with good dates');
 775     is(scalar @missing_criticals_1, 0, 'Got the expected missing critical array size of 0 from check_branch_code with good dates');
 776
 777     # Given ... some very bad dates
 778     my @missing_criticals_2 = ();
 779     my $dateofbirth_2  = '03-2016-05';
 780     my $dateenrolled_2 = '04-2016-05';
 781     my $dateexpiry_2   = '06-2016-05';
 782     my $borrower_2     = { dateofbirth => $dateofbirth_2, dateenrolled => $dateenrolled_2, dateexpiry => $dateexpiry_2, };
 783     my $params_2       = {borrower => $borrower_2, lineraw => $borrowerline, line => $line_number, missing_criticals => \@missing_criticals_2, };
 784
 785     # When ...
 786     $patrons_import->format_dates($params_2);
 787
 788     # Then ...
 789     is($borrower_2->{dateofbirth}, '', 'Got the expected empty date of birth from format_dates with bad dates');
 790     is($borrower_2->{dateenrolled}, '', 'Got the expected emptydate of birth from format_dates with bad dates');
 791     is($borrower_2->{dateexpiry}, '', 'Got the expected empty date of birth from format_dates with bad dates');
 792
 793     is(scalar @missing_criticals_2, 3, 'Got the expected missing critical array size of 3 from check_branch_code with bad dates');
 794     is($missing_criticals_2[0]->{bad_date}, 1, 'Got the expected first bad date flag from check_branch_code with bad dates');
 795     is($missing_criticals_2[0]->{key}, 'dateofbirth', 'Got the expected dateofbirth key from check_branch_code with bad dates');
 796     is($missing_criticals_2[0]->{line}, $line_number, 'Got the expected first line from check_branch_code with bad dates');
 797     is($missing_criticals_2[0]->{lineraw}, $borrowerline, 'Got the expected first lineraw from check_branch_code with bad dates');
 798
 799     is($missing_criticals_2[1]->{bad_date}, 1, 'Got the expected second bad date flag from check_branch_code with bad dates');
 800     is($missing_criticals_2[1]->{key}, 'dateenrolled', 'Got the expected dateenrolled key from check_branch_code with bad dates');
 801     is($missing_criticals_2[1]->{line}, $line_number, 'Got the expected second line from check_branch_code with bad dates');
 802     is($missing_criticals_2[1]->{lineraw}, $borrowerline, 'Got the expected second lineraw from check_branch_code with bad dates');
 803
 804     is($missing_criticals_2[2]->{bad_date}, 1, 'Got the expected third bad date flag from check_branch_code with bad dates');
 805     is($missing_criticals_2[2]->{key}, 'dateexpiry', 'Got the expected dateexpiry key from check_branch_code with bad dates');
 806     is($missing_criticals_2[2]->{line}, $line_number, 'Got the expected third line from check_branch_code with bad dates');
 807     is($missing_criticals_2[2]->{lineraw}, $borrowerline, 'Got the expected third lineraw from check_branch_code with bad dates');
 808 };
 809
 810 # ###### Test utility ###########
 811 sub make_csv {
 812     my ($temp_dir, @lines) = @_;
 813
 814     my ($fh, $filename) = tempfile( DIR => $temp_dir) or die $!;
 815     print $fh $_."\r\n" foreach @lines;
 816     close $fh or die $!;
 817
 818     return $filename;
 819 }
 820
 821 1;