Sets url and description to empty when unavailable

Author: shweta-nhs

src/eligibility_signposting_api/views/eligibility.py

@@ -137,9 +137,9 @@ def build_actions(condition: Condition) -> list[eligibility.Action] | None:
             eligibility.Action(
                 actionType=eligibility.ActionType(action.action_type),
                 actionCode=eligibility.ActionCode(action.action_code),
-                description=eligibility.Description(action.action_description),
-                urlLink=eligibility.UrlLink(action.url_link),
-                urlLabel=eligibility.UrlLabel(action.url_label),
+                description=eligibility.Description(action.action_description or ""),
+                urlLabel=eligibility.UrlLabel(action.url_label or ""),
+                urlLink=eligibility.UrlLink(str(action.url_link)) if action.url_link else eligibility.UrlLink(""),
             )
             for action in condition.actions
         ]

src/eligibility_signposting_api/views/response_model/eligibility.py

@@ -2,7 +2,7 @@
 from enum import StrEnum
 from typing import NewType
 
-from pydantic import UUID4, BaseModel, Field, HttpUrl, field_serializer
+from pydantic import UUID4, BaseModel, Field, field_serializer
 from pydantic_core.core_schema import SerializationInfo
 
 LastUpdated = NewType("LastUpdated", datetime)
@@ -50,9 +50,9 @@ class SuitabilityRule(BaseModel):
 class Action(BaseModel):
     action_type: ActionType = Field(..., alias="actionType")
     action_code: ActionCode = Field(..., alias="actionCode")
-    description: Description = Field(default="")
-    url_link: UrlLink = Field(default="", alias="urlLink" )
-    url_label: UrlLabel = Field(default="", alias="urlLabel")
+    description: Description = Field(default=Description(""))
+    url_link: UrlLink = Field(default=UrlLink(""), alias="urlLink")
+    url_label: UrlLabel = Field(default=UrlLabel(""), alias="urlLabel")
 
     model_config = {"populate_by_name": True}
 

tests/fixtures/builders/model/eligibility.py

@@ -3,7 +3,6 @@
 
 from polyfactory import Use
 from polyfactory.factories import DataclassFactory
-from pydantic import HttpUrl
 
 from eligibility_signposting_api.model import eligibility
 from eligibility_signposting_api.model.eligibility import UrlLink

tests/integration/in_process/test_eligibility_endpoint.py

@@ -218,7 +218,15 @@ def test_actionable(
                                             "cohortText": "positive_description",
                                         }
                                     ],
-                                    "actions": [{"actionType": "defaultcomms", "actionCode": "action_code"}],
+                                    "actions": [
+                                        {
+                                            "actionCode": "action_code",
+                                            "actionType": "defaultcomms",
+                                            "description": "",
+                                            "urlLabel": "",
+                                            "urlLink": "",
+                                        }
+                                    ],
                                     "suitabilityRules": [],
                                     "statusText": "Status.actionable",
                                 }
@@ -355,7 +363,15 @@ def test_actionable_when_only_magic_cohort_is_present(
                                             "cohortText": "magic positive description",
                                         }
                                     ],
-                                    "actions": [{"actionType": "defaultcomms", "actionCode": "action_code"}],
+                                    "actions": [
+                                        {
+                                            "actionCode": "action_code",
+                                            "actionType": "defaultcomms",
+                                            "description": "",
+                                            "urlLabel": "",
+                                            "urlLink": "",
+                                        }
+                                    ],
                                     "suitabilityRules": [],
                                     "statusText": "Status.actionable",
                                 }
@@ -511,7 +527,15 @@ def test_actionable(
                                     "condition": "FLU",
                                     "status": "Actionable",
                                     "eligibilityCohorts": [],
-                                    "actions": [{"actionCode": "action_code", "actionType": "defaultcomms"}],
+                                    "actions": [
+                                        {
+                                            "actionCode": "action_code",
+                                            "actionType": "defaultcomms",
+                                            "description": "",
+                                            "urlLabel": "",
+                                            "urlLink": "",
+                                        }
+                                    ],
                                     "suitabilityRules": [],
                                     "statusText": "Status.actionable",
                                 }

tests/unit/views/test_eligibility.py

@@ -11,7 +11,6 @@
 from flask import Flask, Request
 from flask.testing import FlaskClient
 from hamcrest import assert_that, contains_exactly, has_entries, has_length, is_, none
-from pydantic import HttpUrl
 from wireup.integration.flask import get_app_container
 
 from eligibility_signposting_api.audit.audit_service import AuditService
@@ -601,9 +600,9 @@ def test_excludes_nulls_via_build_response(client: FlaskClient):
                     eligibility.Action(
                         actionType=eligibility.ActionType("TYPE_A"),
                         actionCode=eligibility.ActionCode("CODE123"),
-                        description=None,  # Should be excluded
-                        urlLink=None,  # Should be excluded
-                        urlLabel=None,  # Should be excluded
+                        description=eligibility.Description(""),  # Should be an empty string
+                        urlLink=eligibility.UrlLink(""),  # Should be an empty string
+                        urlLabel=eligibility.UrlLabel(""),  # Should be an empty string
                     )
                 ],
             )
@@ -633,9 +632,9 @@ def test_excludes_nulls_via_build_response(client: FlaskClient):
 
         assert action["actionType"] == "TYPE_A"
         assert action["actionCode"] == "CODE123"
-        assert "description" not in action
-        assert "urlLink" not in action
-        assert "urlLabel" not in action
+        assert action["description"] == ""
+        assert action["urlLink"] == ""
+        assert action["urlLabel"] == ""
 
 
 def test_build_response_include_values_that_are_not_null(client: FlaskClient):