Skip to content

Commit cc44ce3

Browse files
committed
Merge branch 'feature/PI-730-update_accredited_system_base_model' into release/2025-01-17
2 parents c6b01c3 + a5e11a9 commit cc44ce3

File tree

5 files changed

+9
-14
lines changed

5 files changed

+9
-14
lines changed

src/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ def _transact_write_items_with_sleep(*args, **kwargs):
5959
return response
6060

6161
def _query_with_sleep(*args, **kwargs):
62-
time.sleep(0.5)
62+
time.sleep(0.2)
6363
response = unpatched_query(*args, **kwargs)
6464
return response
6565

src/layers/domain/repository/questionnaire_repository/v1/tests/test_spine_questionnaires.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ def test_spine_mhs_questionnaires_pass(nhs_mhs: NhsMhs):
9797
mhs_interactions_questionnaire = QuestionnaireRepository().read(
9898
QuestionnaireInstance.SPINE_MHS_MESSAGE_SETS
9999
)
100-
_mhs_data = nhs_mhs.export()
100+
_mhs_data = nhs_mhs.dict()
101101
mhs_data = _apply_field_mapping(
102102
name=QuestionnaireInstance.SPINE_MHS, data=_mhs_data
103103
)

src/layers/sds/domain/nhs_accredited_system.py

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,17 +21,15 @@ class NhsAccreditedSystem(SdsBaseModel):
2121
nhs_requestor_urp: str = Field(alias="nhsrequestorurp")
2222
nhs_date_requested: str = Field(alias="nhsdaterequested")
2323
nhs_id_code: str = Field(alias="nhsidcode")
24-
nhs_mhs_manufacturer_org: Optional[str] = Field(alias="nhsmhsmanufacturerorg")
24+
nhs_mhs_manufacturer_org: str = Field(alias="nhsmhsmanufacturerorg")
2525
nhs_mhs_party_key: str = Field(alias="nhsmhspartykey")
2626
nhs_product_key: str = Field(alias="nhsproductkey")
27-
nhs_product_name: Optional[str] = Field(alias="nhsproductname")
28-
nhs_product_version: Optional[str] = Field(
29-
alias="nhsproductversion"
30-
) # shouldn't be optional?
27+
nhs_product_name: str = Field(alias="nhsproductname", default=None)
28+
nhs_product_version: str = Field(alias="nhsproductversion", default=None)
3129
nhs_as_acf: Optional[set[str]] = Field(alias="nhsasacf")
32-
nhs_as_client: Optional[set[str]] = Field(alias="nhsasclient", default_factory=set)
30+
nhs_as_client: set[str] = Field(alias="nhsasclient", default_factory=set)
3331
nhs_as_svc_ia: set[str] = Field(alias="nhsassvcia")
34-
nhs_temp_uid: Optional[str] = Field(alias="nhstempuid")
32+
nhs_temp_uid: str = Field(alias="nhstempuid", default=None)
3533
description: Optional[str] = Field(alias="description")
3634
nhs_as_category_bag: Optional[set[str]] = Field(alias="nhsascategorybag")
3735

src/layers/sds/domain/nhs_mhs.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ class NhsMhs(SdsBaseModel):
2626
alias="nhscontractpropertytemplatekey"
2727
)
2828
nhs_date_approved: str = Field(alias="nhsdateapproved")
29-
nhs_date_dns_approved: str = Field(alias="nhsdatednsapproved")
29+
nhs_date_dns_approved: str = Field(alias="nhsdatednsapproved", default=None)
3030
nhs_date_requested: str = Field(alias="nhsdaterequested")
3131
nhs_dns_approver: str = Field(alias="nhsdnsapprover")
3232
nhs_ep_interaction_type: InteractionType = Field(alias="nhsepinteractiontype")
@@ -41,7 +41,7 @@ class NhsMhs(SdsBaseModel):
4141
nhs_mhs_in: str = Field(alias="nhsmhsin")
4242
nhs_mhs_ip_address: Optional[str] = Field(alias="nhsmhsipaddress")
4343
nhs_mhs_is_authenticated: Authentication = Field(alias="nhsmhsisauthenticated")
44-
nhs_mhs_manufacturer_org: Optional[str] = Field(alias="nhsmhsmanufacturerorg")
44+
nhs_mhs_manufacturer_org: str = Field(alias="nhsmhsmanufacturerorg")
4545
nhs_mhs_party_key: str = Field(alias="nhsmhspartykey")
4646
nhs_mhs_sn: str = Field(alias="nhsmhssn")
4747
nhs_mhs_svc_ia: str = Field(alias="nhsmhssvcia")

src/layers/sds/epr/bulk_create/tests/conftest.py

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,11 +83,9 @@ def accredited_system_1() -> NhsMhs:
8383
nhsmhspartykey="AAA-123456",
8484
nhsproductname="My EPR Product",
8585
nhsproductkey="key-123",
86-
nhsproductversion=None,
8786
nhsasclient={"ABC", "CDE", "EFG"},
8887
nhsassvcia={"interaction-id-1", "interaction-id-2"},
8988
nhsmhsmanufacturerorg="AAA",
90-
nhstempuid=None,
9189
)
9290

9391

@@ -106,7 +104,6 @@ def accredited_system_2() -> NhsMhs:
106104
nhsidcode="BBB",
107105
nhsmhspartykey="AAA-456789",
108106
nhsproductkey="key-123",
109-
nhsproductversion=None,
110107
nhsasclient={"ABC", "JKL", "LMN"},
111108
nhsassvcia={"interaction-id-3", "interaction-id-4"},
112109
nhsmhsmanufacturerorg="AAA",

0 commit comments

Comments
 (0)