Skip to content

test: Sprint test-3 lite — 3 tests E2E con shinytest2#66

Merged
pablotis merged 1 commit intostagingfrom
chore/tests-sprint-3
May 7, 2026
Merged

test: Sprint test-3 lite — 3 tests E2E con shinytest2#66
pablotis merged 1 commit intostagingfrom
chore/tests-sprint-3

Conversation

@pablotis
Copy link
Copy Markdown
Collaborator

@pablotis pablotis commented May 7, 2026

Summary

Sprint test-3 versión recortada (vs scope original de 5-7 tests + Codecov), con foco en cubrir el smoke + regresión del toggle Tipo de dúo + render de output post-navegación. ROI suficiente para el costo de mantenimiento de E2E.

3 tests E2E (7 expects):

Infra:

  • Workflow tests-e2e.yml separado con workflow_dispatch + cron semanal (domingo 06:00 UTC). NO corre en cada PR.
  • Guard RUN_E2E=true: suite default rápida (skip de E2E).
  • Helper new_app() con skips defensivos.

Diferidos a futuro:

Suite total: 192 tests (185 unit + 7 E2E con RUN_E2E=true).

Test plan

  • Suite default sigue rápida (Rscript tests/testthat.R → 185 PASS + 3 SKIP).
  • Suite full pasa local (RUN_E2E=true Rscript tests/testthat.R → 192 PASS).
  • CI tests-unit.yml sigue verde (no debería verse afectado por el guard).
  • Workflow tests-e2e.yml ejecutable manualmente desde Actions UI (probar post-merge).

🤖 Generated with Claude Code

Versión recortada del Sprint test-3 (scope inicial 5-7 tests + Codecov)
con foco en cubrir el smoke y la regresión del toggle Tipo de dúo. ROI
suficiente para el costo de mantenimiento de E2E.

Tests cubiertos:
- smoke: app levanta y registra el input tipo_duo con default
  "trimestral".
- toggle Tipo de dúo: cambiar trimestral ↔ anual se refleja en el
  reactive state via input. Cubre regresión #44.
- módulo Calidad: tras navegar al nav_panel, el output kpi_encontrado
  renderiza un valor numérico válido (forma "X.X%" o "—").

Infraestructura:
- Workflow tests-e2e.yml separado con workflow_dispatch + cron
  semanal (domingo 06:00 UTC). NO corre en cada PR. tests-unit.yml
  sigue siendo la barrera obligatoria.
- Guard RUN_E2E=true env var: la suite default (Rscript tests/testthat.R)
  salta los E2E manteniéndose rápida (~30s vs +3min).
- Helper new_app() encapsula AppDriver$new + skips defensivos
  (paquetes ausentes, datasets faltantes, env var).

Diferidos a futuro (documentados en ROADMAP):
- E2E de descarga (quirk de Chromote con downloadHandler que copia
  archivos vía file.copy).
- Codecov.
- E2E de regresión #40 line charts (requiere snapshot testing).

Suite total: 192 tests (185 unit + 7 E2E con RUN_E2E=true).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@pablotis pablotis merged commit 3d25474 into staging May 7, 2026
1 check passed
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