Fix search

uw_hrp/models.py

@@ -1,17 +1,11 @@
 # Copyright 2022 UW-IT, University of Washington
 # SPDX-License-Identifier: Apache-2.0
 
-from datetime import datetime, timezone
 from dateutil.parser import parse
 import json
 from restclients_core import models
 
 
-def get_now():
-    # return time-zone-aware datetime
-    return datetime.now(timezone.utc)
-
-
 def date_to_str(d_obj):
     if d_obj is not None:
         return str(d_obj)

uw_hrp/person.py

@@ -72,7 +72,8 @@ def person_search(**kwargs):
         if "Persons" in data:
             for person_record in data.get("Persons"):
                 persons.append(Person(data=person_record))
-        if "Next" in data and len(data["Next"].get("Href")) > 0:
+        if (data.get("Next") and data["Next"].get("Href") and
+                len(data["Next"]["Href"]) > 0):
             url = data["Next"]["Href"]
         else:
             break

uw_hrp/resources/hrpws/file/hrp/v3/person.json_changed_since_date_2022-12-12_page_size_200

@@ -26,7 +26,7 @@
     "FutureWorker": null
   },
   "Next": {
-    "Href": "/hrp/v3/person.json?name=&page_start=2&page_size=1&worker_wid=&last_name=&first_name=&legal_last_name=&legal_first_name=&preferred_last_name=&preferred_first_name=&position=&retired=&terminated=&current_faculty=&active_appointment=&supervisory_organization=&cost_center=&custom_organization_type=&custom_organization=&location=&future_worker=",
+    "Href": null,
     "ResourceName": null,
     "PageStart": "2",
     "PageSize": "1",

uw_hrp/tests/test_person.py

@@ -84,6 +84,6 @@ def test_get_person_by_regid(self):
                           get_person_by_regid, "000")
 
     def test_person_search(self):
-        pass
-        # persons = person_search(changed_since_date="2022-12-12")
-        # self.assertEqual(len(persons), 0)
+        persons = person_search(changed_since_date="2022-12-12")
+        self.assertEqual(len(persons), 1)
+        self.assertFalse(persons[0].is_active)