Merge pull request #564 from NHSDigital/release/2025-04-01

Release/2025 04 01

Author: megan-bower4

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 2025-04-01
+- [PI-848] Add info box to swaager explaining prodID usage in non-prod envs
+- [PI-870] Sonarcloud fixes
+
 ## 2025-03-31
 - [PI-835] Clean up deployment policies
 - [PI-861] Remove SDS secret access

VERSION

@@ -1 +1 @@
-2025.03.31
+2025.04.01

changelog/2025-04-01.md

@@ -0,0 +1,2 @@
+- [PI-848] Add info box to swaager explaining prodID usage in non-prod envs
+- [PI-870] Sonarcloud fixes

infrastructure/swagger/02_info.yaml

@@ -10,6 +10,22 @@ info:
     name: MIT
     url: https://github.com/NHSDigital/connecting-party-manager/blob/main/LICENCE.md
   description: |
+    <div class="nhsd-m-emphasis-box nhsd-m-emphasis-box--emphasis nhsd-!t-margin-bottom-6" aria-label="Highlighted Information">
+        <div class="nhsd-a-box nhsd-a-box--border-blue">
+            <div class="nhsd-m-emphasis-box__image-box">
+                <figure class="nhsd-a-image">
+                    <picture class="nhsd-a-image__picture">
+                        <img src="http://digital.nhs.uk/binaries/content/gallery/icons/info.svg?colour=231f20" alt="" style="object-fit:fill">
+                    </picture>
+                </figure>
+            </div>
+            <div class="nhsd-m-emphasis-box__content-box">
+                <div data-uipath="website.contentblock.emphasis.content" class="nhsd-t-word-break"><p class="nhsd-t-body">Product IDs created in non-production environments follow the same format as production, but are only valid in the environment in which they were created.</p></div>
+            </div>
+        </div>
+    </div>
+    <hr class="nhsd-a-horizontal-rule">
+
     ## Overview
 
     Use this API to access the Connecting Party Manager (CPM) service - an internal service for registering and managing details of IT systems and applications that connect to our APIs – sometimes known as ‘connecting parties’ and referred to herein as ‘products’.

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "connecting-party-manager"
-version = "2025.03.31"
+version = "2025.04.01"
 description = "Repository for the Connecting Party Manager API and related services"
 authors = ["NHS England"]
 license = "LICENSE.md"

src/api/tests/feature_tests/environment.py

@@ -82,22 +82,6 @@ def before_feature(context: Context, feature: Feature):
         name=feature.name,
         description=" ".join(feature.description),
     )
-    cpm_scenarios = [
-        "Create Product Team - success scenarios",
-        "Create Product Team - failure scenarios",
-        "Read Product Team - success scenarios",
-        "Read Product Team - failure scenarios",
-        "Delete Product Team - success scenarios",
-        "Delete Product Team - failure scenarios",
-        "Create CPM Product - success scenarios",
-        "Create CPM Product - failure scenarios",
-        "Read CPM Product - success scenarios",
-        "Read CPM Product - failure scenarios",
-        "Delete CPM Product - success scenarios",
-        "Delete CPM Product - failure scenarios",
-        "Search Products - success scenarios",
-        "Search Products - failures scenarios",
-    ]
     if context.test_mode is TestMode.INTEGRATION:
         table = "dynamodb_cpm_table_name.value"
         context.table_name = read_terraform_output(table)

src/api/tests/feature_tests/steps/context.py

@@ -1,5 +1,6 @@
 from contextlib import AbstractContextManager
 from dataclasses import dataclass
+from typing import Optional
 
 from behave.model import Table
 from behave.runner import Context as BehaveContext
@@ -17,19 +18,19 @@
 @dataclass
 class Context(BehaveContext):
     base_url: str
-    headers: dict[str, dict[str, str]] = None
-    response: Response = None
-    table: Table = None
-    test_mode: TestMode = None
-    table_name: str = None
-    session: AbstractContextManager = None
-    dynamodb_client: DynamoDBClient = None
-    workspace: str = None
-    environment: str = None
-    workspace_type: str = None
-    api_key: str = None
-    notes: dict[str, str] = None
-    postman_endpoint: str = None
+    headers: Optional[dict[str, dict[str, str]]]
+    response: Optional[Response]
+    table: Optional[Table]
+    test_mode: Optional[TestMode]
+    table_name: Optional[str]
+    session: Optional[AbstractContextManager]
+    dynamodb_client: Optional[DynamoDBClient]
+    workspace: Optional[str]
+    environment: Optional[str]
+    workspace_type: Optional[str]
+    api_key: Optional[str]
+    notes: Optional[dict[str, str]]
+    postman_endpoint: Optional[str]
 
     postman_collection: PostmanCollection = None
     postman_feature: PostmanItem = None

src/api/tests/feature_tests/steps/requests.py

@@ -1,5 +1,4 @@
 import json as _json
-import time
 from contextlib import contextmanager
 from dataclasses import dataclass
 from unittest import mock
@@ -9,7 +8,6 @@
 from domain.response.aws_lambda_response import AwsLambdaResponse
 from event.json import json_loads
 from requests import HTTPError, Response, request
-from requests.exceptions import SSLError
 
 from api.tests.feature_tests.steps.data import DUMMY_CONTEXT
 from api.tests.feature_tests.steps.endpoint_lambda_mapping import (
@@ -35,21 +33,6 @@ def _parse_url(base_url: str, endpoint: str) -> str:
     return url
 
 
-@contextmanager
-def retry_on_ssl_error(sleep_time: int = 3, max_retries=5):
-    retries = 0
-    while True:
-        try:
-            yield
-        except SSLError:
-            if retries == max_retries:
-                raise
-            time.sleep(sleep_time)
-            retries += 1
-        finally:
-            break
-
-
 def make_request(
     base_url: str,
     http_method: str,
@@ -62,10 +45,9 @@ def make_request(
     json = body if type(body) is dict else None
     data = None if type(body) is dict else body
 
-    with retry_on_ssl_error():
-        response = request(
-            method=http_method, url=url, headers=headers, json=json, data=data
-        )
+    response = request(
+        method=http_method, url=url, headers=headers, json=json, data=data
+    )
 
     if raise_for_status:
         try:

src/layers/api_utils/api_step_chain/__init__.py

@@ -1,6 +1,6 @@
 from types import ModuleType
 
-from api_utils.versioning.constants import VERSIONING_STEP_ARGS
+from api_utils.versioning.constants import VersioningStepArgs
 from api_utils.versioning.steps import (
     get_largest_possible_version,
     get_steps_for_requested_version,
@@ -27,8 +27,8 @@ def execute_step_chain(
     )
     version_chain.run(
         init={
-            VERSIONING_STEP_ARGS.EVENT: event,
-            VERSIONING_STEP_ARGS.VERSIONED_STEPS: versioned_steps,
+            VersioningStepArgs.EVENT: event,
+            VersioningStepArgs.VERSIONED_STEPS: versioned_steps,
         }
     )
 

src/layers/api_utils/versioning/constants.py

@@ -3,6 +3,6 @@
 VERSION_RE = re.compile(r"^v(\d+)$")
 
 
-class VERSIONING_STEP_ARGS:
+class VersioningStepArgs:
     VERSIONED_STEPS = "versioned_steps"
     EVENT = "event"