Skip to content

Commit

Permalink
test: fix official holidays tests for Portugal
Browse files Browse the repository at this point in the history
All Saints Day was incorrectly set to be present between 2013 and 2015.
This is incorrect as between 2013 and 2015 (inclusive) this holiday did
not happen due to government deliberation.

Signed-off-by: Sacha Telgenhof <me@sachatelgenhof.com>
  • Loading branch information
stelgenhof committed Dec 13, 2024
1 parent 74a719e commit 75125b9
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion tests/Portugal/PortugalTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -51,14 +51,17 @@ public function testOfficialHolidays(): void
'easter',
'goodFriday',
'assumptionOfMary',
'allSaintsDay',
'immaculateConception',
'christmasDay',
'25thApril',
'restorationOfIndependence',
'portugalDay',
];

if ($this->year <= 2012 || $this->year >= 2016) {
$holidays[] = 'allSaintsDay';
}

if (($this->year >= 1910 && $this->year <= 2012) || $this->year >= 2016) {
$holidays[] = 'portugueseRepublic';
}
Expand Down

0 comments on commit 75125b9

Please sign in to comment.