Good Friday is not an official holiday in Brazil

CHANGELOG.md

@@ -38,6 +38,7 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/) and this p
 - Liberation Day is official in the Netherlands [\#169](https://github.com/azuyalabs/yasumi/pull/169) ([c960657](https://github.com/c960657))
 - Typos in Easter Monday and Republic Day for the 'it_IT' locale [\#171](https://github.com/azuyalabs/yasumi/pull/171) ([c960657](https://github.com/c960657))
 - Corrected the name of the Emperors Birthday function and variable.
+- Good Friday is not official in Brazil [\#174](https://github.com/azuyalabs/yasumi/pull/174) ([c960657](https://github.com/c960657))
 
 ### Removed
 - Unused constants.

src/Yasumi/Provider/Brazil.php

@@ -52,7 +52,7 @@ public function initialize(): void
         $this->addHoliday($this->christmasDay($this->year, $this->timezone, $this->locale));
         $this->addHoliday($this->easter($this->year, $this->timezone, $this->locale, Holiday::TYPE_OBSERVANCE));
         $this->addHoliday($this->corpusChristi($this->year, $this->timezone, $this->locale, Holiday::TYPE_OBSERVANCE));
-        $this->addHoliday($this->goodFriday($this->year, $this->timezone, $this->locale));
+        $this->addHoliday($this->goodFriday($this->year, $this->timezone, $this->locale, Holiday::TYPE_OBSERVANCE));
         $this->addHoliday($this->ashWednesday($this->year, $this->timezone, $this->locale, Holiday::TYPE_OBSERVANCE));
 
         /**

tests/Brazil/BrazilTest.php

@@ -33,7 +33,6 @@ public function testOfficialHolidays(): void
     {
         $this->assertDefinedHolidays([
             'newYearsDay',
-            'goodFriday',
             'tiradentesDay',
             'internationalWorkersDay',
             'independenceDay',
@@ -53,6 +52,7 @@ public function testObservedHolidays(): void
         $this->assertDefinedHolidays([
             'carnavalMonday',
             'carnavalTuesday',
+            'goodFriday',
             'easter',
             'corpusChristi',
             'ashWednesday'

tests/Brazil/GoodFridayTest.php

@@ -68,6 +68,6 @@ public function testTranslation(): void
      */
     public function testHolidayType(): void
     {
-        $this->assertHolidayType(self::REGION, self::HOLIDAY, $this->generateRandomYear(), Holiday::TYPE_OFFICIAL);
+        $this->assertHolidayType(self::REGION, self::HOLIDAY, $this->generateRandomYear(), Holiday::TYPE_OBSERVANCE);
     }
 }