Merge branch 'GREG-337-show-created-and-updated-fields' into 'master'

GREG-337: Add fields created and updated to endpoints

See merge request !391

greg/api/serializers/consent_type.py

@@ -27,7 +27,10 @@ class ConsentTypeSerializer(ModelSerializer):
             "user_allowed_to_change",
             "valid_from",
             "choices",
+            "created",
+            "updated",
         )
+        read_only_fields = ["created", "updated"]
 
 
 class ConsentTypeSerializerBrief(ModelSerializer):

greg/api/serializers/organizational_unit.py

@@ -33,5 +33,7 @@ class SponsorOrgUnitsSerializer(ModelSerializer):
             "hierarchical_access",
             "automatic",
             "source",
+            "created",
+            "updated",
         ]
-        read_only_fields = ["sponsor", "organizational_unit"]
+        read_only_fields = ["sponsor", "organizational_unit", "created", "updated"]

greg/api/serializers/person.py

@@ -24,8 +24,12 @@ class PersonSerializer(serializers.ModelSerializer):
             "roles",
             "consents",
             "meta",
+            "created",
+            "updated",
         ]
 
+        read_only_fields = ["created", "updated"]
+
 
 class PersonSearchSerializer(serializers.ModelSerializer):
     person_id = serializers.CharField(source="id")

greg/api/serializers/sponsor.py

@@ -9,4 +9,14 @@ class SponsorSerializer(serializers.ModelSerializer):
 
     class Meta:
         model = Sponsor
-        fields = ["id", "feide_id", "first_name", "last_name", "work_email", "orgunits"]
+        fields = [
+            "id",
+            "feide_id",
+            "first_name",
+            "last_name",
+            "work_email",
+            "orgunits",
+            "created",
+            "updated",
+        ]
+        read_only_fields = ["created", "updated"]

greg/tests/api/test_consent_type.py

@@ -38,4 +38,6 @@ def test_get_consent_type(client, consent_type_foo: ConsentType):
                 "text_nn": "Nei",
             },
         ],
+        "created": consent_type_foo.created.strftime("%Y-%m-%dT%H:%M:%S.%fZ"),
+        "updated": consent_type_foo.updated.strftime("%Y-%m-%dT%H:%M:%S.%fZ"),
     }

greg/tests/api/test_person.py

@@ -122,18 +122,10 @@ def test_person_create(client):
     url = reverse("v1:person-list")
     response = client.post(url, {"first_name": "foo木👍أ", "last_name": "غbar水"})
     results = response.json()
-    assert results == {
-        "id": 1,
-        "first_name": "foo木👍أ",
-        "last_name": "غbar水",
-        "gender": None,
-        "date_of_birth": None,
-        "registration_completed_date": None,
-        "identities": [],
-        "roles": [],
-        "consents": [],
-        "meta": None,
-    }
+    assert results.get("first_name") == "foo木👍أ"
+    assert results.get("last_name") == "غbar水"
+    assert results.get("created") is not None
+    assert results.get("updated") is not None
 
 
 @pytest.mark.django_db

gregui/tests/api/views/test_consent.py

@@ -25,6 +25,8 @@ def test_get_consents(client, consent_type_foo, consent_type_bar):
             "name_nn": "F",
             "user_allowed_to_change": True,
             "valid_from": "2018-01-20",
+            "created": consent_type_foo.created.strftime("%Y-%m-%dT%H:%M:%S.%fZ"),
+            "updated": consent_type_foo.updated.strftime("%Y-%m-%dT%H:%M:%S.%fZ"),
         },
         {
             "choices": [
@@ -41,5 +43,7 @@ def test_get_consents(client, consent_type_foo, consent_type_bar):
             "name_nn": "B",
             "user_allowed_to_change": True,
             "valid_from": "2018-01-20",
+            "created": consent_type_bar.created.strftime("%Y-%m-%dT%H:%M:%S.%fZ"),
+            "updated": consent_type_bar.updated.strftime("%Y-%m-%dT%H:%M:%S.%fZ"),
         },
     ]