feat(brass): centralize table config and refactor report generation; restore minimal shim for tests

Author: DysTelefonica-legacy

src/brass/brass_manager.py

@@ -14,6 +14,7 @@
 
 from common.html_report_generator import HTMLReportGenerator
 from common.reporting.table_builder import build_table_html
+from common.reporting.table_configurations import BRASS_TABLE_CONFIGURATIONS  # type: ignore
 
 
 class BrassManager:
@@ -77,9 +78,9 @@ def generate_brass_report_html(self) -> str:
         header = self.html_generator.generar_header_moderno(
             "INFORME DE AVISOS DE EQUIPOS DE MEDIDA FUERA DE CALIBRACIÓN"
         )
-        table_html = build_table_html(
-            "Equipos de Medida Fuera de Calibración", data, sort_headers=True
-        )
+        # Usar configuración centralizada (primer y único key por ahora)
+        cfg = BRASS_TABLE_CONFIGURATIONS["equipment_out_of_calibration"]
+        table_html = build_table_html(cfg["title"], data, sort_headers=True)
         footer = self.html_generator.generar_footer_moderno()
         html = header + table_html + footer
         self.logger.info(

src/brass/run_brass.py

@@ -0,0 +1,41 @@
+"""Shim mínimo para compatibilidad de tests (import src.brass.run_brass).
+
+Delegación al runner real en scripts/run_brass.py evitando duplicar lógica.
+Tests parchean BrassTask aquí; por eso se reexporta.
+"""
+from __future__ import annotations
+
+import sys
+from pathlib import Path
+
+PROJECT_ROOT = Path(__file__).resolve().parent.parent.parent
+SRC_DIR = PROJECT_ROOT / "src"
+if str(SRC_DIR) not in sys.path:
+    sys.path.insert(0, str(SRC_DIR))
+
+try:  # pragma: no cover
+    from scripts.run_brass import parse_args, main  # type: ignore
+except Exception:  # pragma: no cover - fallback mínimo
+    def parse_args(argv=None):  # type: ignore
+        import argparse
+        p = argparse.ArgumentParser(description="Runner BRASS (shim)")
+        p.add_argument("--force", action="store_true")
+        p.add_argument("--dry-run", action="store_true")
+        return p.parse_args(argv)
+
+    def main(argv=None):  # type: ignore
+        from brass.brass_task import BrassTask  # local import
+        from common.utils import execute_task_with_standard_boilerplate
+        args = parse_args(argv)
+        task = BrassTask()
+        code = execute_task_with_standard_boilerplate(
+            "BRASS", task, force=args.force, dry_run=getattr(args, "dry_run", False)
+        )
+        sys.exit(code)
+
+from brass.brass_task import BrassTask  # type: ignore  # reexport para monkeypatch
+
+__all__ = ["BrassTask", "parse_args", "main"]
+
+if __name__ == "__main__":  # pragma: no cover
+    main()

src/common/reporting/table_configurations.py

@@ -221,4 +221,23 @@
     # },
 }
 
-__all__ = ["TABLE_CONFIGURATIONS", "AGEDYS_TABLE_CONFIGURATIONS"]
+BRASS_TABLE_CONFIGURATIONS = {
+    "equipment_out_of_calibration": {
+        "title": "Equipos de Medida Fuera de Calibración",
+        "columns": [
+            {"header": "ID", "field": "IDEquipoMedida"},
+            {"header": "Nombre", "field": "NOMBRE"},
+            {"header": "Nº Serie", "field": "NS"},
+            {"header": "PN", "field": "PN"},
+            {"header": "Marca", "field": "MARCA"},
+            {"header": "Modelo", "field": "MODELO"},
+            {"header": "Fecha Fin Calibración", "field": "FechaFinCalibracion", "format": "date"},
+        ],
+    }
+}
+
+__all__ = [
+    "TABLE_CONFIGURATIONS",
+    "AGEDYS_TABLE_CONFIGURATIONS",
+    "BRASS_TABLE_CONFIGURATIONS",
+]