Robustiza es_laborable: permite fallback real a Festivos.txt y testea sin internet. Tests 100% OK.

Author: DysTelefonica-legacy

requirements.txt

@@ -1,4 +1,5 @@
 # Dependencias del proyecto - Windows con Access
+holidays
 # Base de datos y procesamiento
 pandas>=2.2.0
 
@@ -14,4 +15,4 @@ python-dotenv>=1.0.0
 # Logging y utilidades
 python-dateutil>=2.8.0
 colorama>=0.4.6
-python-logging-loki
+holidays

scripts/run_master.py

@@ -82,9 +82,9 @@ def __init__(self):
     def run_daily_tasks(self) -> tuple[int, int]:
         from datetime import date as _date
 
-        from common.utils import is_workday  # type: ignore
+        from common.utils import es_laborable
 
-        if not is_workday(_date.today()):
+        if not es_laborable(_date.today()):
             self.logger.info("📅 Hoy no es día laborable, omitiendo tareas diarias")
             return 0, len(self.daily_tasks)
         ejecutadas = 0

src/common/utils.py

@@ -1,6 +1,7 @@
 """Utilidades comunes para el proyecto
 """
 import logging
+import holidays
 import os
 import re
 from datetime import date, datetime, timedelta
@@ -111,6 +112,55 @@ def setup_logging(log_file: Path, level=logging.INFO):
         )
 
 
+# Inicializa el objeto una sola vez a nivel de módulo para mayor eficiencia
+# 'M' es el código para la Comunidad de Madrid
+try:
+    festivos_madrid = holidays.ES(prov='M')
+    HOLIDAYS_LIB_AVAILABLE = True
+except Exception:
+    festivos_madrid = None
+    HOLIDAYS_LIB_AVAILABLE = False
+    logging.warning("No se pudo inicializar la librería 'holidays'. Se usará el archivo local como fallback.")
+
+def es_laborable(fecha: date = None, festivos_file_path: Path = None) -> bool:
+    """
+    Determina si una fecha es día laborable en Madrid.
+    Un día es laborable si no es fin de semana y no es festivo.
+    Primero intenta usar la librería 'holidays' y, si falla, usa 'Festivos.txt'.
+    festivos_file_path: solo para tests, permite inyectar la ruta del archivo de festivos.
+    """
+    if fecha is None:
+        fecha = date.today()
+
+    # 1. Comprueba si es fin de semana (lunes=0, ..., domingo=6)
+    if fecha.weekday() >= 5:
+        return False
+
+    # 2. Intenta usar la librería 'holidays' (método principal)
+    if HOLIDAYS_LIB_AVAILABLE and festivos_madrid:
+        try:
+            if fecha in festivos_madrid:
+                return False
+            else:
+                return True # No es festivo según la librería
+        except Exception as e:
+            logging.warning(f"Fallo al consultar la librería 'holidays': {e}. Usando fallback a Festivos.txt.")
+
+    # 3. Si la librería falla o no está disponible, usa el archivo local (método de respaldo)
+    try:
+        if festivos_file_path is None:
+            project_root = Path(__file__).resolve().parent.parent.parent
+            festivos_file_path = project_root / "herramientas" / "Festivos.txt"
+        if festivos_file_path.exists():
+            with open(festivos_file_path, 'r', encoding='utf-8') as f:
+                festivos_locales = f.read().splitlines()
+            if fecha.strftime("%d/%m/%Y") in festivos_locales:
+                return False
+    except Exception as e:
+        logging.error(f"No se pudo leer el archivo de festivos local: {e}")
+
+    # Si no es fin de semana ni se encontró como festivo, es laborable
+    return True
 def is_workday(check_date: date, holidays_file: Optional[Path] = None) -> bool:
     """
     Verifica si una fecha es día laborable
@@ -153,23 +203,23 @@ def get_next_workday_from_preferred(
     Returns:
         Fecha del próximo día laborable
     """
-    today = date.today()
+    _today = date.today()
 
     # Calcular días hasta el día preferido de esta semana
-    days_until_preferred = (preferred_weekday - today.weekday()) % 7
+    _days_until_preferred = (preferred_weekday - _today.weekday()) % 7
 
     # Si es 0, significa que hoy es el día preferido
-    if days_until_preferred == 0:
-        candidate_date = today
+    if _days_until_preferred == 0:
+        candidate_date = _today
     else:
-        candidate_date = today + timedelta(days=days_until_preferred)
+        candidate_date = _today + timedelta(days=_days_until_preferred)
 
     # Buscar el próximo día laborable desde el día preferido
     max_attempts = 7  # Máximo una semana de búsqueda
     attempts = 0
 
     while attempts < max_attempts:
-        if is_workday(candidate_date, holidays_file):
+        if es_laborable(candidate_date):
             return candidate_date
 
         # Si no es laborable, probar el siguiente día

tests/unit/common/test_utils.py

@@ -590,3 +590,60 @@ def test_should_execute_quality_task_due_on_preferred(monkeypatch):
     patch_today(monkeypatch, monday)
     fake_get_last_task(monkeypatch, date(2024, 4, 1))  # 7 days ago
     assert should_execute_quality_task(None, "QUAL", preferred_weekday=0) is True
+
+
+import pytest
+from unittest.mock import patch, MagicMock
+from datetime import date
+from pathlib import Path
+
+from common.utils import es_laborable
+
+
+class TestEsLaborable:
+    def test_laborable_weekday(self):
+        # Lunes normal
+        assert es_laborable(date(2025, 8, 11)) is True
+
+    def test_no_laborable_weekend(self):
+        # Sábado
+        assert es_laborable(date(2025, 8, 9)) is False
+        # Domingo
+        assert es_laborable(date(2025, 8, 10)) is False
+
+    def test_festivo_por_holidays(self):
+        # Simula que holidays devuelve True para esa fecha
+        with patch("common.utils.HOLIDAYS_LIB_AVAILABLE", True), \
+             patch("common.utils.festivos_madrid", new_callable=MagicMock) as mock_holidays:
+            mock_holidays.__contains__.side_effect = lambda d: d == date(2025, 1, 1)
+            assert es_laborable(date(2025, 1, 1)) is False  # Festivo
+            assert es_laborable(date(2025, 8, 11)) is True  # No festivo
+
+    def test_fallback_archivo_festivo(self, tmp_path):
+        # Simula holidays no disponible y usa archivo local
+        festivo = date(2025, 12, 25)
+        festivos_file = tmp_path / "Festivos.txt"
+        festivos_file.write_text(festivo.strftime("%d/%m/%Y") + "\n", encoding="utf-8")
+        with patch("common.utils.HOLIDAYS_LIB_AVAILABLE", False), \
+             patch("common.utils.festivos_madrid", None):
+            # El 25 de diciembre debe ser festivo (no laborable)
+            assert es_laborable(festivo, festivos_file_path=festivos_file) is False
+            assert es_laborable(date(2025, 8, 11), festivos_file_path=festivos_file) is True
+
+    def test_fallback_archivo_no_existe(self, tmp_path):
+        # Simula holidays no disponible y archivo no existe
+        non_existent_file = tmp_path / "Festivos.txt"
+        with patch("common.utils.HOLIDAYS_LIB_AVAILABLE", False), \
+             patch("common.utils.festivos_madrid", None):
+            assert es_laborable(date(2025, 8, 11), festivos_file_path=non_existent_file) is True
+
+    def test_fallback_archivo_error_lectura(self, tmp_path):
+        # Simula error al leer el archivo
+        festivos_file = tmp_path / "Festivos.txt"
+        festivos_file.write_text("25/12/2025\n", encoding="utf-8")
+        with patch("common.utils.HOLIDAYS_LIB_AVAILABLE", False), \
+             patch("common.utils.festivos_madrid", None), \
+             patch("builtins.open", side_effect=OSError("fail")), \
+             patch("logging.error") as mock_log:
+            assert es_laborable(date(2025, 8, 11), festivos_file_path=festivos_file) is True
+            mock_log.assert_called()