From 7037e350d6f005c5f66bdba40fcefea471618659 Mon Sep 17 00:00:00 2001 From: Fridolin Somers Date: Tue, 12 Feb 2019 15:52:22 +0100 Subject: [PATCH] Bug 21832: add unit test Adds a check that is_expired exists in service GetPatronInfo. Test plan : Run : prove t/db_dependent/ILSDI_Services.t Signed-off-by: Josef Moravec Signed-off-by: Jonathan Druart Amended patch: remove ref to bug number, we have git for that Signed-off-by: Nick Clemens (cherry picked from commit 20ce110f5c656f51bad262708d4b577568c1a52a) Signed-off-by: Martin Renvoize (cherry picked from commit 4e305856af7b713868206e385fb1abac7a79a9d8) Signed-off-by: Lucas Gass --- t/db_dependent/ILSDI_Services.t | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/t/db_dependent/ILSDI_Services.t b/t/db_dependent/ILSDI_Services.t index 9b6908ff12..210e05d92b 100644 --- a/t/db_dependent/ILSDI_Services.t +++ b/t/db_dependent/ILSDI_Services.t @@ -103,7 +103,7 @@ subtest 'AuthenticatePatron test' => sub { subtest 'GetPatronInfo/GetBorrowerAttributes test for extended patron attributes' => sub { - plan tests => 2; + plan tests => 3; $schema->storage->txn_begin; @@ -228,6 +228,8 @@ subtest 'GetPatronInfo/GetBorrowerAttributes test for extended patron attributes # Check results: is_deeply( $reply->{'attributes'}, [ $cmp ], 'Test GetPatronInfo - show_attributes parameter' ); + ok( exists $reply->{is_expired}, 'There should be the is_expired information'); + # Cleanup $schema->storage->txn_rollback; }; -- 2.39.5