feat: CAT-002 Regla de Oro Rama vs Hoja + validaciones #35

dmolinari · 2026-04-19T11:41:45Z

dmolinari commented

2026-04-19 11:41:45 +00:00

Summary

Regla de Oro enforced en Application layer (Create/Move guards + GetTree tieneAvisos)
IAvisoQueryRepository con NullAvisoQueryRepository stub (PRD-002 reemplaza binding)
2 nuevas domain exceptions → 409 Conflict (RubroPadreEsHojaConAvisosException activa; RubroEsRamaConHijosActivosException definida/mapeada, consumidor en PRD-002)
Frontend: tieneAvisos?: boolean en RubroTreeNode + disable "Agregar subrubro" con tooltip cuando === true
1450 tests green (+31 vs baseline 1419), 0 regresiones

Test plan

Backend Application: dotnet test tests/SIGCM2.Application.Tests/ → 844/844
Backend Api: dotnet test tests/SIGCM2.Api.Tests/ → 254/254 (sequential per #29)
Frontend: cd src/web && npx vitest run → 352/352
Smoke manual post-merge: crear rubro hijo bajo hoja (hoy OK; cuando Aviso exista, 409)

Warnings resueltos (tracking)

W1 — e2e 409 gap por RSA singleton sealed en TestWebApplicationFactory: abierto como follow-up #36 (label followup). Prioridad antes de PRD-002. Cobertura actual compositiva (handler unit + ExceptionFilter unit + DI wiring test en /arbol).
W2 — título tooltip divergente de spec: resuelto en commit a0a1874 — test aprieta match exacto del literal de impl ("El rubro contiene avisos asignados. Muévalos antes de agregar sub-rubros.").
W3 — commit c3 duplicado en mensaje: cosmetic, sin acción (requeriría rebase destructivo).
W4 — NullAvisoQueryRepository.CountAvisosBatchAsync devuelve {} vs spec {1:0,...}: design-intentional, todos los consumers usan GetValueOrDefault(id, 0). Cerrado.

Follow-ups

#36 — [CAT-002 / Infra] TestWebApplicationFactory: desacoplar RSA singleton (bloquea e2e override en PRD-002 y otros)

Artifacts SDD (engram)

sdd/cat-002-regla-oro-rama-vs-hoja/explore
sdd/cat-002-regla-oro-rama-vs-hoja/proposal
sdd/cat-002-regla-oro-rama-vs-hoja/spec
sdd/cat-002-regla-oro-rama-vs-hoja/design
sdd/cat-002-regla-oro-rama-vs-hoja/tasks
sdd/cat-002-regla-oro-rama-vs-hoja/apply-progress
sdd/cat-002-regla-oro-rama-vs-hoja/verify-report
sdd/cat-002-regla-oro-rama-vs-hoja/archive-report

## Summary - Regla de Oro enforced en Application layer (Create/Move guards + GetTree tieneAvisos) - `IAvisoQueryRepository` con `NullAvisoQueryRepository` stub (PRD-002 reemplaza binding) - 2 nuevas domain exceptions → 409 Conflict (`RubroPadreEsHojaConAvisosException` activa; `RubroEsRamaConHijosActivosException` definida/mapeada, consumidor en PRD-002) - Frontend: `tieneAvisos?: boolean` en `RubroTreeNode` + disable "Agregar subrubro" con tooltip cuando `=== true` - **1450 tests green (+31 vs baseline 1419), 0 regresiones** ## Test plan - [x] Backend Application: `dotnet test tests/SIGCM2.Application.Tests/` → **844/844** - [x] Backend Api: `dotnet test tests/SIGCM2.Api.Tests/` → **254/254** (sequential per #29) - [x] Frontend: `cd src/web && npx vitest run` → **352/352** - [x] Smoke manual post-merge: crear rubro hijo bajo hoja (hoy OK; cuando Aviso exista, 409) ## Warnings resueltos (tracking) - **W1 — e2e 409 gap por RSA singleton sealed en TestWebApplicationFactory**: abierto como follow-up #36 (label `followup`). Prioridad antes de PRD-002. Cobertura actual compositiva (handler unit + ExceptionFilter unit + DI wiring test en `/arbol`). - **W2 — título tooltip divergente de spec**: resuelto en commit `a0a1874` — test aprieta match exacto del literal de impl ("El rubro contiene avisos asignados. Muévalos antes de agregar sub-rubros."). - **W3 — commit c3 duplicado en mensaje**: cosmetic, sin acción (requeriría rebase destructivo). - **W4 — `NullAvisoQueryRepository.CountAvisosBatchAsync` devuelve `{}` vs spec `{1:0,...}`**: design-intentional, todos los consumers usan `GetValueOrDefault(id, 0)`. Cerrado. ## Follow-ups - #36 — `[CAT-002 / Infra] TestWebApplicationFactory: desacoplar RSA singleton` (bloquea e2e override en PRD-002 y otros) ## Siguiente - Mergear este PR - PRD-002 reemplaza binding DI `IAvisoQueryRepository` → implementación real + activa `RubroEsRamaConHijosActivosException` en `CreateAvisoCommandHandler` - Considerar `RepeatableRead` o constraint DB para TOCTOU cuando exista `dbo.Aviso` ## Artifacts SDD (engram) - `sdd/cat-002-regla-oro-rama-vs-hoja/explore` - `sdd/cat-002-regla-oro-rama-vs-hoja/proposal` - `sdd/cat-002-regla-oro-rama-vs-hoja/spec` - `sdd/cat-002-regla-oro-rama-vs-hoja/design` - `sdd/cat-002-regla-oro-rama-vs-hoja/tasks` - `sdd/cat-002-regla-oro-rama-vs-hoja/apply-progress` - `sdd/cat-002-regla-oro-rama-vs-hoja/verify-report` - `sdd/cat-002-regla-oro-rama-vs-hoja/archive-report`

dmolinari added 8 commits 2026-04-19 11:41:45 +00:00

feat(domain): excepciones regla de oro rama/hoja (CAT-002) ddd28ea4d5

feat(application): IAvisoQueryRepository + NullAvisoQueryRepository (CAT-002) 673194e249

feat(application): RubroTreeBuilder + GetRubroTree con tieneAvisos (CAT-002) 9e50a929ae

feat(application): guard avisos en CreateRubroCommandHandler (CAT-002) 216983623a

feat(application): guard avisos en MoveRubroCommandHandler (CAT-002) c03aad8c5a

feat(application): RubroTreeBuilder + GetRubroTree con tieneAvisos (CAT-002) f861dfa826

feat(api): ExceptionFilter 409 para regla de oro + DTO delta (CAT-002) bb5dde6e24

feat(frontend): tieneAvisos en RubroTreeNode + disable btn subrubro (CAT-002) 4f25233bab

dmolinari referenced this pull request

2026-04-19 11:52:28 +00:00

[CAT-002 / Infra] TestWebApplicationFactory: desacoplar RSA singleton para permitir override scoped en e2e #36