Skip to content

Commit b4fc45f

Browse files
authored
Merge pull request #748 from NHSDigital/export-environment-to-smoke-test
Export `ENVIRONMENT` prior to running smoke test
2 parents 2330c8e + 5d3b6c7 commit b4fc45f

File tree

2 files changed

+3
-15
lines changed

2 files changed

+3
-15
lines changed

scripts/python/smoke_test/notifications_smoke_test.py

Lines changed: 2 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,8 @@
1515
def test_notifications():
1616
logging.info("Running notifications smoke test")
1717

18-
environment, storage_account, resource_group_name = configure()
18+
environment = os.getenv("ENVIRONMENT")
19+
resource_group_name = f"rg-manbrs-{environment}-container-app-uks"
1920

2021
if environment == "prod":
2122
return
@@ -38,20 +39,6 @@ def test_notifications():
3839
logging.info("Finished notifications smoke test")
3940

4041

41-
def configure():
42-
environment = os.getenv("ENVIRONMENT", "dev")
43-
pr_number = os.getenv("PR_NUMBER", "")
44-
storage_account = f"stmanbrs{environment}uks"
45-
46-
if pr_number != "":
47-
environment = f"pr-{pr_number}"
48-
storage_account = f"stmanbrspr{pr_number}uks"
49-
50-
resource_group_name = f"rg-manbrs-{environment}-container-app-uks"
51-
52-
return (environment, storage_account, resource_group_name)
53-
54-
5542
def setup_mesh_inbox_test_data(environment: str, resource_group_name: str):
5643
populate_mesh_env_vars(environment, resource_group_name)
5744
mesh_client().send_message(

scripts/terraform/terraform.mk

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ terraform-validate: terraform-init-no-backend # Validate Terraform changes - mak
7777

7878
notifications-smoke-test:
7979
if test "${RUN_NOTIFICATIONS_SMOKE_TEST}" == "true"; then
80+
export ENVIRONMENT
8081
pip install pytest mesh-client
8182
pytest -vv scripts/python/smoke_test/notifications_smoke_test.py
8283
else

0 commit comments

Comments
 (0)