Fixed bug festival would return Shavuos on 7 of other months.

fixes #24
simlist · Aug 24, 2022 · 2868238 · 2868238
1 parent c3f52be
commit 2868238
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/pyluach/utils.py b/pyluach/utils.py
@@ -253,7 +253,7 @@ def _festival(date, israel=False, include_working_days=True):
         return 11
     elif month == 2 and day == 18 and include_working_days:
         return 12
-    elif month == 3 and day == 6 or (not israel and day == 7):
+    elif month == 3 and (day == 6 or (not israel and day == 7)):
         return 13
     elif month == 5 and day == 15 and include_working_days:
         return 14

diff --git a/tests/test_hebrewcal.py b/tests/test_hebrewcal.py
@@ -417,6 +417,8 @@ def test_shavuos(self):
         assert holiday(shavuos, True) == 'Shavuos'
         assert holiday(shavuos + 1, True) is None
         assert festival(shavuos + 1, include_working_days=False) == 'Shavuos'
+        not_shavuos = dates.HebrewDate(5782, 4, 7)
+        assert festival(not_shavuos) is None
 
     def test_tubeav(self):
         tubeav = dates.HebrewDate(5779, 5, 15)