Repository: dr-prodigy/python-holidays

Issue:
October 25 is no longer holiday in Spain Prov=PV
I'm using the WORKDAY integration of Home Assistant to obtain a binary sensor for workdays.
AFAIK python-holidays is being used "under the hood" in this integration
Yesterday October 25th was wrongly marked as holiday.
It's not holiday since 2014.

More info (sorry, in spanish): [https://es.wikipedia.org/wiki/D%C3%ADa_del_Pa%C3%ADs_Vasco#Creaci%C3%B3n_y_desaparici%C3%B3n_de_la_festividad]

Discussion:
coveralls: ## Pull Request Test Coverage Report for [Build 1471793367](https://coveralls.io/builds/44334201)

* **2** of **2**   **(100.0%)**  changed or added relevant lines in **1** file are covered.
* No unchanged relevant lines lost coverage.
* Overall coverage increased (+**0.0005%**) to **98.694%**

---



|  Totals | [![Coverage Status](https://coveralls.io/builds/44334201/badge)](https://coveralls.io/builds/44334201) |
| :-- | --: |
| Change from base [Build 1288461108](https://coveralls.io/builds/43163534): |  0.0005% |
| Covered Lines: | 5505 |
| Relevant Lines: | 5513 |

---
##### 💛  - [Coveralls](https://coveralls.io)


Patch:
diff --git a/holidays/countries/spain.py b/holidays/countries/spain.py
index 6b2c8dd0..3fd94870 100644
--- a/holidays/countries/spain.py
+++ b/holidays/countries/spain.py
@@ -185,7 +185,10 @@ class Spain(HolidayBase):
             elif self.prov == "NC":
                 self._is_observed(date(year, SEP, 27), "Día de Navarra")
             elif self.prov == "PV":
-                self._is_observed(date(year, OCT, 25), "Día del Páis Vasco")
+                if 2011 <= year <= 2013:
+                    self._is_observed(
+                        date(year, OCT, 25), "Día del Páis Vasco"
+                    )
             elif self.prov == "RI":
                 self._is_observed(date(year, JUN, 9), "Día de La Rioja")
 


Test patch:
diff --git a/test/countries/test_spain.py b/test/countries/test_spain.py
index 6108ad57..822ced95 100644
--- a/test/countries/test_spain.py
+++ b/test/countries/test_spain.py
@@ -84,7 +84,6 @@ class TestSpain(unittest.TestCase):
             (9, 11): ["CT"],
             (9, 27): ["NC"],
             (10, 9): ["VC"],
-            (10, 25): ["PV"],
         }
         for prov, prov_holidays in self.prov_holidays.items():
             for year in range(2010, 2025):
@@ -142,3 +141,11 @@ class TestSpain(unittest.TestCase):
                         date(year, *fest_day) in prov_holidays,
                         prov in fest_prov,
                     )
+
+    def test_change_of_province_specific_days(self):
+        prov_holidays = self.prov_holidays["PV"]
+        self.assertNotIn(date(2010, 10, 25), prov_holidays)
+        self.assertIn(date(2011, 10, 25), prov_holidays)
+        self.assertIn(date(2012, 10, 25), prov_holidays)
+        self.assertIn(date(2013, 10, 25), prov_holidays)
+        self.assertNotIn(date(2014, 10, 25), prov_holidays)