Addressed pr comments

Author: Andyg79

tests_utils/test_date_time_utils.py

@@ -32,31 +32,26 @@ def test_add_days():
     assert new_date == date + timedelta(days=5)
 
 
-def test_get_day_of_week_for_today():
+# Valid weekdays for testing get_day_of_week
+VALID_WEEKDAYS = [
+    "Monday",
+    "Tuesday",
+    "Wednesday",
+    "Thursday",
+    "Friday",
+    "Saturday",
+    "Sunday",
+]
+
+
+def test_get_day_of_week_with_specific_date():
     dtu = utils.date_time_utils.DateTimeUtils()
-    date = datetime.now()
-    day_of_week = dtu.get_a_day_of_week(date)
-    assert day_of_week in [
-        "Monday",
-        "Tuesday",
-        "Wednesday",
-        "Thursday",
-        "Friday",
-        "Saturday",
-        "Sunday",
-    ]
-
-
-def test_get_a_day_of_week():
+    date = datetime(2023, 11, 8)  # Known Wednesday
+    day_of_week = dtu.get_day_of_week(date)
+    assert day_of_week in VALID_WEEKDAYS
+
+
+def test_get_day_of_week_with_default_today():
     dtu = utils.date_time_utils.DateTimeUtils()
-    date = datetime(2023, 11, 8)
-    day_of_week = dtu.get_a_day_of_week(date)
-    assert day_of_week in [
-        "Monday",
-        "Tuesday",
-        "Wednesday",
-        "Thursday",
-        "Friday",
-        "Saturday",
-        "Sunday",
-    ]
+    day_of_week = dtu.get_day_of_week()
+    assert day_of_week in VALID_WEEKDAYS

utils/date_time_utils.py

@@ -1,4 +1,5 @@
 from datetime import datetime, timedelta
+from typing import Optional
 import random
 
 
@@ -46,27 +47,18 @@ def add_days(date: datetime, days: float) -> datetime:
         return date + timedelta(days=days)
 
     @staticmethod
-    def get_day_of_week_for_today(date: datetime) -> str:
-        """Gets the day of the week (e.g., Monday, Tuesday) from the specified date.
-
-        Args:
-            date (datetime): The current date using the now function
-
-        Returns:
-            str: The day of the week relating to the specified date.
+    def get_day_of_week(date: Optional[datetime] = None) -> str:
         """
-        return date.strftime("%A")
-
-    @staticmethod
-    def get_a_day_of_week(date: datetime) -> str:
-        """Gets the day of the week (e.g., Monday, Tuesday) from the specified date.
+        Returns the day of the week (e.g., Monday, Tuesday) for the given date.
+        If no date is provided, uses today’s date.
 
         Args:
-            date (datetime): The date for which the day of the week will be returned.
+            date (Optional[datetime]): The date to inspect. Defaults to now.
 
         Returns:
-            str: The day of the week relating to the specified date.
+            str: Day of week corresponding to the date.
         """
+        date = date or datetime.now()
         return date.strftime("%A")
 
     @staticmethod
@@ -99,7 +91,8 @@ def screening_practitioner_appointments_report_timestamp_date_format() -> str:
 
         return DateTimeUtils.format_date(datetime.now(), "%d.%m.%Y at %H:%M:%S")
 
-    def month_string_to_number(self, string: str) -> int:
+    @staticmethod
+    def month_string_to_number(string: str) -> int:
         """
         This is used to convert a month from a string to an integer.
         It accepts the full month or the short version and is not case sensitive
@@ -126,7 +119,8 @@ def month_string_to_number(self, string: str) -> int:
         except Exception:
             raise ValueError("Not a month")
 
-    def generate_unique_weekday_date(self, start_year: int = 2025) -> str:
+    @staticmethod
+    def generate_unique_weekday_date(start_year: int = 2025) -> str:
         """
         Generates a future weekday date from the specified year onward.