| Lines 282-288
          HERE
      
      
        Link Here | 
        
          | 282 |     }; | 282 |     }; | 
        
          | 283 |  | 283 |  | 
        
          | 284 |     subtest 'set message_transport_types' => sub { | 284 |     subtest 'set message_transport_types' => sub { | 
          
            
              | 285 |         plan tests => 12; | 285 |         plan tests => 18; | 
        
          | 286 |  | 286 |  | 
        
          | 287 |         $schema->storage->txn_begin; | 287 |         $schema->storage->txn_begin; | 
        
          | 288 |  | 288 |  | 
  
    | Lines 339-348
          HERE
      
      
        Link Here | 
        
          | 339 |         }); | 339 |         }); | 
        
          | 340 |         is($stored_transports->count, 2, 'Two transports selected'); | 340 |         is($stored_transports->count, 2, 'Two transports selected'); | 
        
          | 341 |  | 341 |  | 
          
            
              | 342 |         # Test email and smsalertnumber validation | 342 |         # Test contact information validation | 
        
          | 343 |         eval { Koha::Patron::Message::Transport::Types->new({ | 343 |         eval { Koha::Patron::Message::Transport::Types->new({ | 
        
          | 344 |                 message_transport_type => 'email' | 344 |                 message_transport_type => 'email' | 
        
          | 345 |             })->store }; | 345 |             })->store }; | 
            
              |  |  | 346 |         eval { Koha::Patron::Message::Transport::Types->new({ | 
            
              | 347 |                 message_transport_type => 'itiva' | 
            
              | 348 |             })->store }; | 
            
              | 349 |         eval { Koha::Patron::Message::Transport::Types->new({ | 
            
              | 350 |                 message_transport_type => 'phone' | 
            
              | 351 |             })->store }; | 
        
          | 346 |         eval { Koha::Patron::Message::Transport::Types->new({ | 352 |         eval { Koha::Patron::Message::Transport::Types->new({ | 
        
          | 347 |                 message_transport_type => 'sms' | 353 |                 message_transport_type => 'sms' | 
        
          | 348 |             })->store }; | 354 |             })->store }; | 
  
    | Lines 351-362
          HERE
      
      
        Link Here | 
        
          | 351 |             message_transport_type => 'email', | 357 |             message_transport_type => 'email', | 
        
          | 352 |             is_digest              => 1 | 358 |             is_digest              => 1 | 
        
          | 353 |         })->store; | 359 |         })->store; | 
            
              |  |  | 360 |         Koha::Patron::Message::Transport->new({ | 
            
              | 361 |             message_attribute_id   => $preference->message_attribute_id, | 
            
              | 362 |             message_transport_type => 'itiva', | 
            
              | 363 |             is_digest              => 1 | 
            
              | 364 |         })->store; | 
            
              | 365 |         Koha::Patron::Message::Transport->new({ | 
            
              | 366 |             message_attribute_id   => $preference->message_attribute_id, | 
            
              | 367 |             message_transport_type => 'phone', | 
            
              | 368 |             is_digest              => 1 | 
            
              | 369 |         })->store; | 
        
          | 354 |         Koha::Patron::Message::Transport->new({ | 370 |         Koha::Patron::Message::Transport->new({ | 
        
          | 355 |             message_attribute_id   => $preference->message_attribute_id, | 371 |             message_attribute_id   => $preference->message_attribute_id, | 
        
          | 356 |             message_transport_type => 'sms', | 372 |             message_transport_type => 'sms', | 
        
          | 357 |             is_digest              => 1 | 373 |             is_digest              => 1 | 
        
          | 358 |         })->store; | 374 |         })->store; | 
          
            
              | 359 |         $patron->set({ email => '', smsalertnumber => '' })->store; | 375 |         t::lib::Mocks::mock_preference('TalkingTechItivaPhoneNotification', 0); | 
            
              |  |  | 376 |         $patron->set({ email => '', phone => '', smsalertnumber => '' })->store; | 
        
          | 360 |         eval { | 377 |         eval { | 
        
          | 361 |             $preference->message_transport_types('email')->store; | 378 |             $preference->message_transport_types('email')->store; | 
        
          | 362 |         }; | 379 |         }; | 
  
    | Lines 367-372
          HERE
      
      
        Link Here | 
        
          | 367 |             .' tells us it which message name it was.'); | 384 |             .' tells us it which message name it was.'); | 
        
          | 368 |         is ($@->borrowernumber, $patron->borrowernumber, 'The previous exception ' | 385 |         is ($@->borrowernumber, $patron->borrowernumber, 'The previous exception ' | 
        
          | 369 |             .' tells us which patron it was.'); | 386 |             .' tells us which patron it was.'); | 
            
              |  |  | 387 |         eval { | 
            
              | 388 |             $preference->message_transport_types('itiva')->store; | 
            
              | 389 |         }; | 
            
              | 390 |         is (ref $@, 'Koha::Exceptions::Patron::Message::Preference::TalkingTechItivaPhoneNotificationRequired', | 
            
              | 391 |             'Adding a message preference with invalid message_transport_type' | 
            
              | 392 |             .' => Koha::Exceptions::Patron::Message::Preference::TalkingTechItivaPhoneNotificationRequired'); | 
            
              | 393 |         is ($@->message_name, $preference->message_name, 'The previous exception ' | 
            
              | 394 |             .' tells us it which message name it was.'); | 
            
              | 395 |         is ($@->borrowernumber, $patron->borrowernumber, 'The previous exception ' | 
            
              | 396 |             .' tells us which patron it was.'); | 
            
              | 397 |         eval { | 
            
              | 398 |             $preference->message_transport_types('phone')->store; | 
            
              | 399 |         }; | 
            
              | 400 |         is (ref $@, 'Koha::Exceptions::Patron::Message::Preference::PhoneNumberRequired', | 
            
              | 401 |             'Adding a message preference with invalid message_transport_type' | 
            
              | 402 |             .' => Koha::Exceptions::Patron::Message::Preference::PhoneNumberRequired'); | 
            
              | 403 |         is ($@->message_name, $preference->message_name, 'The previous exception ' | 
            
              | 404 |             .' tells us it which message name it was.'); | 
            
              | 405 |         is ($@->borrowernumber, $patron->borrowernumber, 'The previous exception ' | 
            
              | 406 |             .' tells us which patron it was.'); | 
        
          | 370 |         eval { | 407 |         eval { | 
        
          | 371 |             $preference->message_transport_types('sms')->store; | 408 |             $preference->message_transport_types('sms')->store; | 
        
          | 372 |         }; | 409 |         }; | 
            
              | 373 | -  |  |  |