Skip to content

Commit ea7c2db

Browse files
committed
review 3: improved reference. splitted tests for battle of Arica & airforce day
1 parent 302d240 commit ea7c2db

File tree

2 files changed

+13
-7
lines changed

2 files changed

+13
-7
lines changed

holidays/countries/peru.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ class Peru(HolidayBase, ChristianHolidays, InternationalHolidays):
2222
References:
2323
* <https://web.archive.org/web/20250414165243/https://www.gob.pe/feriados/>
2424
* <https://es.wikipedia.org/wiki/Anexo:Días_feriados_en_el_Perú>
25-
* <https://web.archive.org/web/20250716164223/https://img.lpderecho.pe/wp-content/uploads/2023/06/Ley-31788-LPDerecho.pdf>
26-
* <https://web.archive.org/web/20250716164455/https://img.lpderecho.pe/wp-content/uploads/2023/07/Ley-31822-LPDerecho.pdf>
25+
* [Ley N° 31788](https://web.archive.org/web/20250716164223/https://img.lpderecho.pe/wp-content/uploads/2023/06/Ley-31788-LPDerecho.pdf)
26+
* [Ley N° 31822](https://web.archive.org/web/20250716164455/https://img.lpderecho.pe/wp-content/uploads/2023/07/Ley-31822-LPDerecho.pdf)
2727
"""
2828

2929
country = "PE"
@@ -51,13 +51,15 @@ def _populate_public_holidays(self):
5151
# Labor Day.
5252
self._add_labor_day(tr("Día del Trabajo"))
5353

54+
# Added via Ley N° 31788 on June 15th, 2023.
5455
if self._year >= 2024:
5556
# Battle of Arica and Flag Day.
5657
self._add_holiday_jun_7(tr("Batalla de Arica y Día de la Bandera"))
5758

5859
# Saint Peter and Saint Paul's Day.
5960
self._add_saints_peter_and_paul_day(tr("San Pedro y San Pablo"))
6061

62+
# Added via Ley N° 31822 on July 8th, 2023.
6163
if self._year >= 2023:
6264
# Peruvian Air Force Day.
6365
self._add_holiday_jul_23(tr("Día de la Fuerza Aérea del Perú"))

tests/countries/test_peru.py

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,15 @@ def test_2022(self):
6060
"2022-12-25",
6161
)
6262

63-
def test_battle_of_arica_and_narional_flag_day(self):
64-
# Check newly added holidays
65-
self.assertHoliday("2025-06-07", "2025-07-23")
66-
self.assertHolidayName("Batalla de Arica y Día de la Bandera", "2025-06-07")
67-
self.assertHolidayName("Día de la Fuerza Aérea del Perú", "2025-07-23")
63+
def test_battle_of_arica_and_flag_day(self):
64+
name = "Batalla de Arica y Día de la Bandera"
65+
self.assertHolidayName(name, (f"{year}-06-07" for year in range(2024, 2050)))
66+
self.assertNoHolidayName(name, range(1950, 2024))
67+
68+
def test_peruvian_air_force_day(self):
69+
name = "Día de la Fuerza Aérea del Perú"
70+
self.assertHolidayName(name, (f"{year}-07-23" for year in range(2023, 2050)))
71+
self.assertNoHolidayName(name, range(1950, 2023))
6872

6973
def test_l10n_default(self):
7074
self.assertLocalizedHolidays(

0 commit comments

Comments
 (0)