View | Details | Raw Unified | Return to bug 23495
Collapse All | Expand All

(-)a/Koha/Patron.pm (-1 / +3 lines)
Lines 439-445 Returns a Koha::SMS::Provider object representing the patron's SMS provider. Link Here
439
439
440
sub sms_provider {
440
sub sms_provider {
441
    my ( $self ) = @_;
441
    my ( $self ) = @_;
442
    return Koha::SMS::Provider->_new_from_dbic($self->_result->sms_provider);
442
    my $sms_provider_rs = $self->_result->sms_provider;
443
    return unless $sms_provider_rs;
444
    return Koha::SMS::Provider->_new_from_dbic($sms_provider_rs);
443
}
445
}
444
446
445
=head3 guarantor_relationships
447
=head3 guarantor_relationships
(-)a/t/db_dependent/Koha/Patrons.t (-2 / +2 lines)
Lines 89-96 subtest 'library' => sub { Link Here
89
};
89
};
90
90
91
subtest 'sms_provider' => sub {
91
subtest 'sms_provider' => sub {
92
    plan tests => 2;
92
    plan tests => 3;
93
    my $sms_provider = $builder->build({source => 'SmsProvider' });
93
    my $sms_provider = $builder->build({source => 'SmsProvider' });
94
    is( $retrieved_patron_1->sms_provider, undef, '->sms_provider should return undef if none defined' );
94
    $retrieved_patron_1->sms_provider_id( $sms_provider->{id} )->store;
95
    $retrieved_patron_1->sms_provider_id( $sms_provider->{id} )->store;
95
    is_deeply( $retrieved_patron_1->sms_provider->unblessed, $sms_provider, 'Koha::Patron->sms_provider returns the correct SMS provider' );
96
    is_deeply( $retrieved_patron_1->sms_provider->unblessed, $sms_provider, 'Koha::Patron->sms_provider returns the correct SMS provider' );
96
    is( ref($retrieved_patron_1->sms_provider), 'Koha::SMS::Provider', 'Koha::Patron->sms_provider should return a Koha::SMS::Provider object' );
97
    is( ref($retrieved_patron_1->sms_provider), 'Koha::SMS::Provider', 'Koha::Patron->sms_provider should return a Koha::SMS::Provider object' );
97
- 

Return to bug 23495