Skip to content

Commit 4327805

Browse files
Axe fix + sonar fixes
1 parent 5cdcd6c commit 4327805

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

utils/axe.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@ class Axe():
1616
under test to identify any accessibility concerns.
1717
"""
1818

19+
@staticmethod
1920
def run(
2021
page: Page,
2122
filename: str = "",
@@ -56,6 +57,7 @@ def run(
5657
json_report_generated=json_report_generated,
5758
)
5859

60+
@staticmethod
5961
def run_list(
6062
page: Page,
6163
page_list: list[str],

utils/nhs_number_tools.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@ class NHSNumberTools:
88
"""
99
A utility class providing functionality around NHS numbers.
1010
"""
11-
def __init__(self) -> None:
12-
pass
1311

1412
@staticmethod
1513
def _nhs_number_checks(nhs_number: str) -> None:
@@ -21,7 +19,7 @@ def _nhs_number_checks(nhs_number: str) -> None:
2119
"""
2220
if not nhs_number.isnumeric():
2321
raise NHSNumberToolsException("The NHS number provided ({}) is not numeric.".format(nhs_number))
24-
if not len(nhs_number) == 10:
22+
if len(nhs_number) != 10:
2523
raise NHSNumberToolsException("The NHS number provided ({}) is not 10 digits.".format(nhs_number))
2624

2725
@staticmethod

utils/user_tools.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ def retrieve_user(user: str) -> dict:
2626
with open(USERS_FILE, 'r') as file:
2727
user_data = json.loads(file.read())
2828

29-
if not user in user_data:
29+
if user not in user_data:
3030
raise UserToolsException(f"User [{user}] is not present in users.json")
3131

3232
logger.debug(f"Returning user: {user_data[user]}")

0 commit comments

Comments
 (0)