diff --git a/utils/axe.py b/utils/axe.py index 22662e01..6d73352b 100644 --- a/utils/axe.py +++ b/utils/axe.py @@ -16,6 +16,7 @@ class Axe(): under test to identify any accessibility concerns. """ + @staticmethod def run( page: Page, filename: str = "", @@ -56,6 +57,7 @@ def run( json_report_generated=json_report_generated, ) + @staticmethod def run_list( page: Page, page_list: list[str], diff --git a/utils/date_time_utils.py b/utils/date_time_utils.py index 5a8bf2ad..ec823e2c 100644 --- a/utils/date_time_utils.py +++ b/utils/date_time_utils.py @@ -6,9 +6,6 @@ class DateTimeUtils: A utility class for doing common actions with datetimes. """ - def __init__(self): - pass - @staticmethod def current_datetime(format_date: str = "%d/%m/%Y %H:%M") -> str: """Gets the current datetime in the specified format. diff --git a/utils/nhs_number_tools.py b/utils/nhs_number_tools.py index 0fc8882a..2b55ae0c 100644 --- a/utils/nhs_number_tools.py +++ b/utils/nhs_number_tools.py @@ -8,8 +8,6 @@ class NHSNumberTools: """ A utility class providing functionality around NHS numbers. """ - def __init__(self) -> None: - pass @staticmethod def _nhs_number_checks(nhs_number: str) -> None: @@ -21,7 +19,7 @@ def _nhs_number_checks(nhs_number: str) -> None: """ if not nhs_number.isnumeric(): raise NHSNumberToolsException("The NHS number provided ({}) is not numeric.".format(nhs_number)) - if not len(nhs_number) == 10: + if len(nhs_number) != 10: raise NHSNumberToolsException("The NHS number provided ({}) is not 10 digits.".format(nhs_number)) @staticmethod diff --git a/utils/user_tools.py b/utils/user_tools.py index 85f19f22..c1694cc2 100644 --- a/utils/user_tools.py +++ b/utils/user_tools.py @@ -26,7 +26,7 @@ def retrieve_user(user: str) -> dict: with open(USERS_FILE, 'r') as file: user_data = json.loads(file.read()) - if not user in user_data: + if user not in user_data: raise UserToolsException(f"User [{user}] is not present in users.json") logger.debug(f"Returning user: {user_data[user]}")