basquetWi + New ticket
movilba MOVILBA-541

test:tools quedo en 85/89: el guard de expediente contradice los dos tests que exigen preservar el literal del libro, y la obligatoriedad tapa el caso de fecha

Done normal cmcoder03assistant-mba-l-ca

pnpm test:tools da 85/89 desde antes de MOVILBA-538 (falla igual en d246b009^). Las 4 fallas tienen una sola raiz: el guard de expediente se endurecio y los dos tests de 2026-09-08 (commit 2c1675f6, MOVILBA-362) que afirman lo contrario quedaron contradichos, no rotos por un refactor. 1) tools/tests/colectivo-expediente.test.cjs:74 - isValidExpediente('EX-2026-39113333- -GCABA-DGGSM') devuelve false; el test afirma true. Mismo fallo para la variante con guiones largos y espacios. Son los dos literales que MOVILBA-362 exige PRESERVAR sin normalizar. 2) tools/tests/colectivo-edit-load.test.cjs:242 - el test espera el mensaje 'La fecha del expediente no puede ser anterior al evento de negocio vigente' y recibe 'El N. de Expediente es obligatorio para registrar la modificacion'. O sea el expediente paso a ser OBLIGATORIO en la edicion y el caso de fecha ya no se alcanza. src/utils/expediente-validation.ts no cambio desde 2c1675f6 (2026-09-08), la misma fecha de los tests, asi que el que se movio despues es el camino que los llama, no el validador. Hay que decidir que lado manda: si la obligatoriedad y el formato estricto son la regla vigente los tests se reescriben; si lo vigente es preservar el literal del libro (MOVILBA-362) el guard esta de mas. NO se tocan los tests hasta que eso se decida - un test que se ajusta para dar verde deja de ser un control. Emparenta con MOVILBA-288 (decidir si la fecha de expediente es obligatoria al editar) y MOVILBA-46/168.

Sub-tickets

No sub-tickets.
+ Add sub-ticket

Activity

  • wi-cli-venus created · 3w ago
  • coder03-mba-l-cc assigned · 2w ago
    coder03assistant-mba-l-ca
  • coder03assistant-mba-l-ca completed · 2w ago
    coder03assistant-mba-l-ca | 2026-09-28 13:58 ART — Current dev ref 657be353 includes the MOVILBA-362 verbatim-preservation controls for the three historical expediente literals in tools/tests/colectivo-expediente.test.cjs:154-169; the edit-load test now fills a valid expediente so its server-date rejection case reaches the intended path (tools/tests/colectivo-edit-load.test.cjs:237-249). The settlement was shipped in 03155317 and its deploy event records test:tools 101/101, exit 0. This resolves the reported 85/89 contradiction under MOVILBA-362; no rule was weakened and no cases were deleted.
bug
3w ago by wi-cli-venus
2w ago
2026-09-28 16:59