Skip to content

Error correddtion#5

Merged
RobertLopez893 merged 3 commits into
mainfrom
error-correddtion
Jun 19, 2026
Merged

Error correddtion#5
RobertLopez893 merged 3 commits into
mainfrom
error-correddtion

Conversation

@RobertLopez893

Copy link
Copy Markdown
Collaborator

Creo que ya está

@RobertLopez893
RobertLopez893 merged commit 644296d into main Jun 19, 2026
2 of 4 checks passed
eddndev pushed a commit that referenced this pull request Jun 19, 2026
PR #5 dejó main en rojo: `cargo fmt --all -- --check` fallaba en archivos sin
formatear (admin_results.rs, jurado_routes.rs, seed/*) y el test
admin_results_export.rs seguía probando el endpoint CSV que #5 renombró a XLSX
(9/10 en 404). Se formatea el workspace (sin cambios semánticos) y se reescribe
el test contra GET /admin/results/edition/:id/export.xlsx: auth (401/403),
validación (400), edición desconocida (404) y que se produce un .xlsx válido
(firma zip PK) — esto además verifica que el handler de Robert funciona.

Verificado local: fmt clean · clippy 0 · cargo test --workspace --locked verde
· openapi en sync · flutter analyze + 39 tests verdes.
Claude-Session: https://claude.ai/code/session_019ZSsKMAxiM8DENEBf36oWv
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.

1 participant