Skip to content

Commit 24f7d7c

Browse files
LeoKHowardLeoKHoward
committed
NPA-5537: Removed excess scenarios
1 parent 9438ef6 commit 24f7d7c

7 files changed

+1
-294
lines changed

‎sandbox/api/constants.py‎

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -74,15 +74,6 @@
7474
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS = (
7575
f"{GET_CONSENT__DIRECTORY}single-mother-child-relationship-unknown-legal-basis.yaml"
7676
)
77-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_BOTH = (
78-
f"{GET_CONSENT__DIRECTORY}single-mother-child-relationship-unknown-legal-basis-include-performer-patient.yaml"
79-
)
80-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_PATIENT = (
81-
f"{GET_CONSENT__DIRECTORY}single-mother-child-relationship-unknown-legal-basis-include-patient.yaml"
82-
)
83-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_PERFORMER = (
84-
f"{GET_CONSENT__DIRECTORY}single-mother-child-relationship-unknown-legal-basis-include-performer.yaml"
85-
)
8677

8778
# GET Consent by ID
8879
GET_CONSENT_BY_ID__INVALID_ID_ERROR = f"{GET_CONSENT__DIRECTORY}ID/errors/invalid-id.yaml"

‎sandbox/api/get_consent.py‎

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,6 @@
2323
INTERNAL_SERVER_ERROR_EXAMPLE,
2424
INVALIDATED_RESOURCE,
2525
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS,
26-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_BOTH,
27-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_PATIENT,
28-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_PERFORMER,
2926
)
3027
from .utils import (
3128
check_for_consent_filtering,
@@ -74,9 +71,6 @@ def get_consent_response() -> Union[dict, tuple]:
7471
return check_for_consent_include_params(
7572
_include,
7673
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS,
77-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_BOTH,
78-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_PATIENT,
79-
GET_CONSENT__SINGLE_MOTHER_CHILD_RELATIONSHIP_UNKNOWN_LEGAL_BASIS_INCLUDE_PERFORMER,
8074
)
8175

8276
elif patient_identifier == "9000000100":

‎sandbox/api/tests/test_get_consent.py‎

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -183,21 +183,6 @@ def test_get_consent_returns_expected_responses__mocked_get_consent(
183183
"./api/examples/GET_Consent/single-mother-child-relationship-unknown-legal-basis.yaml",
184184
200,
185185
),
186-
(
187-
"performer:identifier=9000000999&_include=Consent:patient",
188-
"./api/examples/GET_Consent/single-mother-child-relationship-unknown-legal-basis-include-patient.yaml",
189-
200,
190-
),
191-
(
192-
"performer:identifier=9000000999&_include=Consent:performer",
193-
"./api/examples/GET_Consent/single-mother-child-relationship-unknown-legal-basis-include-performer.yaml",
194-
200,
195-
),
196-
(
197-
"performer:identifier=9000000999&_include=Consent:performer&_include=Consent:patient",
198-
"./api/examples/GET_Consent/single-mother-child-relationship-unknown-legal-basis-include-performer-patient.yaml", # noqa: E501
199-
200,
200-
),
201186
(
202187
"performer:identifier=9000000017&status=test", # Invalid status parameter error
203188
"./api/examples/GET_Consent/errors/invalid-status-parameter.yaml",

‎specification/examples/responses/GET_Consent/single-mother-child-relationship-unknown-legal-basis-include-patient.yaml‎

Lines changed: 0 additions & 81 deletions
This file was deleted.

‎specification/examples/responses/GET_Consent/single-mother-child-relationship-unknown-legal-basis-include-performer-patient.yaml‎

Lines changed: 0 additions & 117 deletions
This file was deleted.

‎specification/examples/responses/GET_Consent/single-mother-child-relationship-unknown-legal-basis-include-performer.yaml‎

Lines changed: 0 additions & 66 deletions
This file was deleted.

‎specification/validated-relationships-service-api.yaml‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -563,6 +563,7 @@ paths:
563563
| A single proxy relationship between consenting adults including details | `performer:identifier=9000000010` and/or `patient:identifier=9000000005&_include=Consent:patient&_include=Consent:proxy` | HTTP Status 200 Bundle containing a single proxy relationship including performer and patient details |
564564
| A single proxy relationship between a mother and child | `performer:identifier=9000000019` and/or `patient:identifier=9000000009` | HTTP Status 200 Bundle containing a single proxy relationship |
565565
| A single proxy relationship between a mother and child including details | `performer:identifier=9000000019` and/or `patient:identifier=9000000009&_include=Consent:patient&_include=Consent:proxy` | HTTP Status 200 Bundle containing a single proxy relationship including performer and patient details |
566+
| A single proxy relationship between a mother and child with legal basis unknown | `performer:identifier=9000000999` and/or `patient:identifier=9000000998` | HTTP Status 200 Bundle containing a single proxy relationship with legal basis unknown |
566567
| Invalid status parameter | `performer:identifier=9000000019` and/or `patient:identifier=9000000009&status=test` | HTTP Status 422 and INVALID_PARAMETER error response |
567568
| Invalid include parameter | `performer:identifier=9000000019` and/or `patient:identifier=9000000009&_include=test` | HTTP Status 422 and INVALID_PARAMETER error response |
568569
| Missing identifier | `patient:identifier=9000000009` | HTTP Status 400 and MISSING_IDENTIFIER_VALUE error response |

0 commit comments

Comments
 (0)