dmolinari/SIG-CM2.0
1
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases Wiki Activity

[PRC-001] Alinear FluentValidation PricePerUnit con opt-in billing (permitir 0) #57

New Issue
Closed
opened 2026-04-21 15:59:02 +00:00 by dmolinari · 1 comment
dmolinari commented 2026-04-21 15:59:02 +00:00
Owner
Copy Link

W4 del verify-report-v2: el DB check constraint pasó de > 0 a >= 0 (CK_Price_NonNegative) para permitir seed global en 0. Verificar que el validator de FluentValidation en CreateChargeableCharConfigCommandValidator permita PricePerUnit >= 0 (o documentar explícitamente la diferencia: DB permite 0, API rechaza 0 si el negocio lo quiere así).

W4 del verify-report-v2: el DB check constraint pasó de `> 0` a `>= 0` (CK_Price_NonNegative) para permitir seed global en 0. Verificar que el validator de FluentValidation en `CreateChargeableCharConfigCommandValidator` permita `PricePerUnit >= 0` (o documentar explícitamente la diferencia: DB permite 0, API rechaza 0 si el negocio lo quiere así).
dmolinari commented 2026-04-21 16:31:35 +00:00
Author
Owner
Copy Link

Resuelto por PR #60 (commit 3e7c4bf).

Validator ahora permite PricePerUnit >= 0, consistente con CK_ChargeableCharConfig_Price_NonNegative de V023.

Cambio: CreateChargeableCharConfigCommandValidator.PricePerUnit: GreaterThan(0m) → GreaterThanOrEqualTo(0m). Mensaje actualizado: "PricePerUnit debe ser >= 0. Usá 0 para desactivar el cobro de este símbolo (opt-in billing)."

Tests: PricePerUnit_Zero_Passes_OptInBilling (antes era _FailsValidation), negativos siguen rechazados. E2E Post_InvalidPrice_Returns400ValidationFailure ahora usa -1 en lugar de 0.

Resuelto por PR #60 (commit `3e7c4bf`). Validator ahora permite `PricePerUnit >= 0`, consistente con `CK_ChargeableCharConfig_Price_NonNegative` de V023. **Cambio**: `CreateChargeableCharConfigCommandValidator.PricePerUnit`: `GreaterThan(0m)` → `GreaterThanOrEqualTo(0m)`. Mensaje actualizado: "PricePerUnit debe ser >= 0. Usá 0 para desactivar el cobro de este símbolo (opt-in billing)." **Tests**: `PricePerUnit_Zero_Passes_OptInBilling` (antes era `_FailsValidation`), negativos siguen rechazados. E2E `Post_InvalidPrice_Returns400ValidationFailure` ahora usa `-1` en lugar de `0`.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
followup
Deuda técnica o mejora detectada en un UDT, a retomar en otro futuro. Revisar al empezar cada UDT.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
dmolinari
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: dmolinari/SIG-CM2.0#57
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?