Skip to content

feat: aggiungere il summary validation a status#46

Merged
Gabrymi93 merged 1 commit intomainfrom
feat/status-validation-summary
Mar 14, 2026
Merged

feat: aggiungere il summary validation a status#46
Gabrymi93 merged 1 commit intomainfrom
feat/status-validation-summary

Conversation

@Gabrymi93
Copy link
Member

Contesto

Questa PR chiude #40 migliorando toolkit status con un summary sintetico delle validation gia disponibili nei layer.

Issue collegata:

Cosa cambia

  • status legge i manifest.json dei layer quando presenti
  • segue il file di validation referenziato dal manifest
  • usa il run record solo come fallback
  • aggiunge una sezione validation_summary per clean, mart e cross_year quando presente

La vista nuova mostra in modo compatto:

  • stato del layer (passed / failed / not_validated / unknown)
  • warning ed error count
  • dettagli utili quando deducibili:
    • missing_columns
    • missing_tables
    • missing_outputs

Perimetro

Perimetro volutamente stretto:

  • toolkit/cli/cmd_status.py
  • tests/test_cli_status.py

Nessun cambiamento a:

  • pipeline
  • formato dei JSON
  • manifest o validator

Verifica

Eseguiti localmente:

.venv\Scripts\python.exe -m pytest tests\test_cli_status.py tests\test_cli_all_commands.py -q

Esito riportato: 5 passed

Note minori

  • validation_missing= oggi copre sia file assente sia JSON corrotto
  • layer_status e validation_summary leggono da sorgenti diverse (run record vs file su disco), quindi in edge case possono divergere

Non blocca il merge, ma vale la pena tenerlo esplicito.

@Gabrymi93
Copy link
Member Author

Gabrymi93 commented Mar 14, 2026

Smoke test rapido fatto su due casi reali.

  1. ispra_ru_base (dataset-incubator)
  • il nuovo validation_summary mostra subito clean e mart come passed
  • questo è utile anche quando il run record dell'ultimo giro riflette solo un run mart e la tabella layer resta meno informativa
validation_summary:
  clean: state=passed warnings=0 errors=0
  mart: state=passed warnings=0 errors=0
  1. siope_comparto_2y (siope-comuni)
  • cross_year compare correttamente nella sezione nuova
  • in assenza di validation dedicata viene mostrato come state=not_validated, senza fingere un esito

Nel concreto il cambio rende status molto piu utile come check operativo rapido, senza sostituire i JSON completi.

@Gabrymi93 Gabrymi93 requested a review from matteocavo March 14, 2026 16:50
@Gabrymi93 Gabrymi93 merged commit dcdae74 into main Mar 14, 2026
5 checks passed
@Gabrymi93 Gabrymi93 deleted the feat/status-validation-summary branch March 14, 2026 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cli: mostrare un summary delle validation in status

2 participants