Improve unit test coverage

scripts/main.py

@@ -359,11 +359,13 @@ def calc_release(next_release_date: datetime.date):
 
 
 # Calculates all releases starting from FIRST_NEW_RELEASE until current date
-def calc_all_releases(first_to_calc: datetime.date):
+def calc_all_releases(
+    first_to_calc: datetime.date, last_to_calc: datetime.date = datetime.date.today()
+):
     next_release_date = first_to_calc
     time_started = datetime.datetime.now()
     n_releases_calculated = 0
-    last_day_to_calc = datetime.date.today() + datetime.timedelta(days=7)
+    last_day_to_calc = last_to_calc + datetime.timedelta(days=7)
     while next_release_date <= last_day_to_calc:
         calc_release(next_release_date=next_release_date)
         n_releases_calculated += 1

scripts/tools.py

@@ -79,7 +79,7 @@ def get_releases_difference(release1: datetime.date, release2: datetime.date) ->
             )
         if (release >= FIRST_NEW_RELEASE) and (release.weekday() != THURSDAY):
             raise AssertionError(
-                f"{release} is before {FIRST_NEW_RELEASE} and is not on Thursday."
+                f"{release} is after {FIRST_NEW_RELEASE} and is not on Thursday."
             )
     if release1 <= LAST_OLD_RELEASE < FIRST_NEW_RELEASE <= release2:
         return (

tests/test_release.py

@@ -1,5 +1,5 @@
 import unittest
-from datetime import date
+from datetime import date, timedelta
 
 from dotenv import load_dotenv
 
@@ -9,7 +9,7 @@
 
 django.setup()
 
-from scripts.main import calc_release
+from scripts.main import calc_release, calc_all_releases
 from b.models import (
     Team_rating,
     Tournament_in_release,
@@ -25,6 +25,7 @@ class TestReleases(unittest.TestCase):
     def setUpClass(cls):
         release_date = date(2021, 9, 16)
         calc_release(release_date)
+        calc_all_releases(release_date, release_date + timedelta(days=14))
         cls.release = Release.objects.get(date=release_date)
 
     def test_team_rating_values(self):

tests/test_tools.py

@@ -28,6 +28,36 @@ def test_get_releases_difference(self):
             tools.get_releases_difference(
                 datetime.date(2020, 3, 21), datetime.date(2021, 9, 16)
             )
+        # release1 is after release2
+        with self.assertRaises(AssertionError) as _:
+            tools.get_releases_difference(
+                datetime.date(2021, 9, 16), datetime.date(2020, 3, 21)
+            )
+        # release1 is between LAST_OLD_RELEASE and FIRST_NEW_RELEASE
+        with self.assertRaises(AssertionError) as _:
+            tools.get_releases_difference(
+                datetime.date(2020, 5, 21), datetime.date(2021, 9, 16)
+            )
+        # release2 is between LAST_OLD_RELEASE and FIRST_NEW_RELEASE
+        with self.assertRaises(AssertionError) as _:
+            tools.get_releases_difference(
+                datetime.date(2020, 3, 21), datetime.date(2021, 8, 16)
+            )
+        # both release1 and release2 is between LAST_OLD_RELEASE and FIRST_NEW_RELEASE
+        with self.assertRaises(AssertionError) as _:
+            tools.get_releases_difference(
+                datetime.date(2020, 5, 21), datetime.date(2021, 8, 16)
+            )
+        # release1 is before LAST_OLD_RELEASE and is not on Friday.
+        with self.assertRaises(AssertionError) as _:
+            tools.get_releases_difference(
+                datetime.date(2020, 3, 23), datetime.date(2021, 9, 16)
+            )
+        # release1 is after FIRST_NEW_RELEASE and is not on Thursday.
+        with self.assertRaises(AssertionError) as _:
+            tools.get_releases_difference(
+                datetime.date(2020, 3, 20), datetime.date(2021, 9, 17)
+            )
 
     def test_get_age_in_weeks(self):
         self.assertEqual(
@@ -52,6 +82,11 @@ def test_get_age_in_weeks(self):
             tools.get_age_in_weeks(
                 datetime.date(2021, 9, 13), datetime.date(2021, 9, 9)
             )
+        # tournament_end is between old releases and new releases.
+        with self.assertRaises(AssertionError) as _:
+            tools.get_age_in_weeks(
+                datetime.date(2021, 8, 13), datetime.date(2021, 8, 9)
+            )
 
     def test_get_prev_release_date(self):
         self.assertEqual(
@@ -68,6 +103,12 @@ def test_get_prev_release_date(self):
         )
         with self.assertRaises(AssertionError) as _:
             tools.get_prev_release_date(datetime.date(2021, 9, 13))
+        # release_date is between old releases and new releases."
+        with self.assertRaises(AssertionError) as _:
+            tools.get_prev_release_date(datetime.date(2021, 8, 13))
+        # release_date is old but not on Friday."
+        with self.assertRaises(AssertionError) as _:
+            tools.get_prev_release_date(datetime.date(2020, 4, 1))
 
 
 if __name__ == "__main__":