Skip to content

Commit 64ac832

Browse files
NRL-2100 Attempting to get public smoke tests working
1 parent a70630b commit 64ac832

5 files changed

Lines changed: 23 additions & 5 deletions

File tree

tests/smoke/conftest.py

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
import pytest
55

66
from scripts.aws_session_assume import get_boto_session
7-
from tests.smoke.environment import EnvironmentConfig, SmokeTestParameters
7+
from tests.smoke.environment import ConnectMode, EnvironmentConfig, SmokeTestParameters
88
from tests.utilities.api_clients import ConsumerTestClient, ProducerTestClient
99

1010

@@ -55,7 +55,11 @@ def producer_client_v1(
5555
environment_config: EnvironmentConfig, smoke_test_parameters: SmokeTestParameters
5656
) -> ProducerTestClient:
5757
config = environment_config.to_client_config(smoke_test_parameters)
58-
config.connection_metadata.ods_code = "SMOKETESTV1"
58+
if environment_config.connect_mode == ConnectMode.INTERNAL:
59+
config.connection_metadata.ods_code = smoke_test_parameters.v1_ods_code
60+
config.custom_headers["NHSD-End-User-Organisation-ODS"] = (
61+
smoke_test_parameters.v1_ods_code
62+
)
5963
return ProducerTestClient(config=config)
6064

6165

@@ -65,6 +69,11 @@ def consumer_client_v1(
6569
) -> ConsumerTestClient:
6670
config = environment_config.to_client_config(smoke_test_parameters)
6771
config.connection_metadata.ods_code = "SMOKETESTV1"
72+
if environment_config.connect_mode == ConnectMode.INTERNAL:
73+
config.connection_metadata.ods_code = smoke_test_parameters.v1_ods_code
74+
config.custom_headers["NHSD-End-User-Organisation-ODS"] = (
75+
smoke_test_parameters.v1_ods_code
76+
)
6877
return ConsumerTestClient(config=config)
6978

7079

tests/smoke/environment.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ def __init__(self, parameters: dict[str, str]):
1515
self.nrlf_app_id = parameters.get("nrlf_app_id")
1616
self.ods_code = parameters.get("ods_code")
1717
self.test_nhs_numbers = parameters.get("test_nhs_numbers").split(",")
18+
self.v1_ods_code = "SMOKETESTV1"
1819

1920

2021
class ConnectMode(Enum):
@@ -59,6 +60,8 @@ def to_client_config(self, parameters: SmokeTestParameters):
5960
custom_headers={
6061
"X-Request-Id": smoketest_id,
6162
"NHSD-Correlation-Id": f"{smoketest_id}.smoketest.{self.stack_name}.{self.env_name}",
63+
"X-Proxygen-App-NRL-App-ID": parameters.nrlf_app_id,
64+
"NHSD-End-User-Organisation-ODS": parameters.ods_code,
6265
},
6366
connection_metadata=connection_metadata,
6467
client_cert=(

tests/smoke/scenarios/consumer_search_read_v1.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import pytest
44

5+
from tests.smoke.environment import SmokeTestParameters
56
from tests.smoke.setup import build_document_reference, upsert_test_pointer
67
from tests.utilities.api_clients import ConsumerTestClient, ProducerTestClient
78

@@ -10,8 +11,9 @@
1011
def test_data_v1(
1112
test_nhs_numbers: list[str],
1213
producer_client_v1: ProducerTestClient,
14+
smoke_test_parameters: SmokeTestParameters,
1315
) -> Generator[str, Any, None]:
14-
test_ods_code = producer_client_v1.config.connection_metadata.ods_code
16+
test_ods_code = smoke_test_parameters.v1_ods_code
1517
test_pointers = [
1618
upsert_test_pointer(
1719
f"{test_ods_code}-smoketest_consumer_search_read_pointer_{n}",

tests/smoke/scenarios/producer_crud_v1.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,17 @@
1+
from tests.smoke.environment import SmokeTestParameters
12
from tests.smoke.setup import build_document_reference
23
from tests.utilities.api_clients import ProducerTestClient
34

45

56
def test_smoke_producer_crud_v1(
67
producer_client_v1: ProducerTestClient,
78
test_nhs_numbers: list[str],
9+
smoke_test_parameters: SmokeTestParameters,
810
):
911
"""
1012
Smoke test scenario for producer CRUD behavior
1113
"""
12-
test_ods_code = producer_client_v1.config.connection_metadata.ods_code
14+
test_ods_code = smoke_test_parameters.v1_ods_code
1315
test_docref = build_document_reference(
1416
nhs_number=test_nhs_numbers[0], custodian=test_ods_code
1517
)

tests/smoke/scenarios/producer_search_read_v1.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import pytest
44

5+
from tests.smoke.environment import SmokeTestParameters
56
from tests.smoke.setup import build_document_reference, upsert_test_pointer
67
from tests.utilities.api_clients import ProducerTestClient
78

@@ -10,8 +11,9 @@
1011
def test_data(
1112
test_nhs_numbers: list[str],
1213
producer_client_v1: ProducerTestClient,
14+
smoke_test_parameters: SmokeTestParameters,
1315
) -> Generator[str, Any, None]:
14-
test_ods_code = producer_client_v1.config.connection_metadata.ods_code
16+
test_ods_code = smoke_test_parameters.v1_ods_code
1517
test_pointers = [
1618
upsert_test_pointer(
1719
f"{test_ods_code}-smoketest_producer_count_search_read_pointer_{n}",

0 commit comments

Comments
 (0)