Bug 29924: (QA follow-up) Remove password_expiration_date from API

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>

Koha/Patron.pm

@@ -1951,6 +1951,7 @@ sub to_api_mapping {
         altcontactsurname   => 'altcontact_surname',
         altcontactstate     => 'altcontact_state',
         altcontactzipcode   => 'altcontact_postal_code',
+        password_expiration_date => undef,
         primary_contact_method => undef,
         secret              => undef,
         auth_method         => undef,

api/v1/swagger/definitions/patron.yaml

@@ -359,13 +359,6 @@ properties:
       - number
       - "null"
     description: Balance of the patron's account
-  password_expiration_date:
-    type:
-      - string
-      - "null"
-    format: date
-    readOnly: true
-    description: day a patron's password expires and must be changed/reset
 additionalProperties: false
 required:
   - surname

api/v1/swagger/paths/patrons.yaml

@@ -189,11 +189,6 @@
         description: Case insensitive search on expiry_date
         required: false
         type: string
-      - name: password_expiration_date
-        in: query
-        description: Case insensitive search on password_expiration_date
-        required: false
-        type: string
       - name: incorrect_address
         in: query
         description: Search on incorrect_address

t/db_dependent/api/v1/patrons.t

@@ -342,7 +342,6 @@ subtest 'add() tests' => sub {
         delete $newpatron->{patron_id};
         delete $newpatron->{restricted};
         delete $newpatron->{anonymized};
-        delete $newpatron->{password_expiration_date};
 
         # Create a library just to make sure its ID doesn't exist on the DB
         my $library_to_delete = $builder->build_object({ class => 'Koha::Libraries' });
@@ -396,7 +395,6 @@ subtest 'add() tests' => sub {
         delete $newpatron->{patron_id};
         delete $newpatron->{restricted};
         delete $newpatron->{anonymized};
-        delete $newpatron->{password_expiration_date};
         $patron_to_delete->delete;
 
         # Set a date field
@@ -622,7 +620,6 @@ subtest 'update() tests' => sub {
         delete $newpatron->{patron_id};
         delete $newpatron->{restricted};
         delete $newpatron->{anonymized};
-        delete $newpatron->{password_expiration_date};
 
         $t->put_ok("//$userid:$password@/api/v1/patrons/-1" => json => $newpatron)
           ->status_is(404)
@@ -701,7 +698,6 @@ subtest 'update() tests' => sub {
         $newpatron->{patron_id} = $unauthorized_patron->to_api->{patron_id};
         $newpatron->{restricted} = $unauthorized_patron->to_api->{restricted};
         $newpatron->{anonymized} = $unauthorized_patron->to_api->{anonymized};
-        $newpatron->{password_expiration_date} = $unauthorized_patron->to_api->{password_expiration_date};
 
         my $got = $result->tx->res->json;
         my $updated_on_got = delete $got->{updated_on};
@@ -733,7 +729,6 @@ subtest 'update() tests' => sub {
         delete $newpatron->{patron_id};
         delete $newpatron->{restricted};
         delete $newpatron->{anonymized};
-        delete $newpatron->{password_expiration_date};
 
         # attempt to update
         $authorized_patron->flags( 2**4 )->store; # borrowers flag = 4