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 <[email protected]>
  • Loading branch information
stelgenhof committed Dec 23, 2024
1 parent 5c0305f commit 1e1a9e8
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 1e1a9e8

Please sign in to comment.