Skip to content

fix(test): el smoke de SAIJ vuelve a pasar, y ahora tambien cubre legislacion - #42

Merged
leocagli merged 2 commits into
mainfrom
arreglo/saij-contrato-nuevo
Aug 30, 2026
Merged

leocagli merged 2 commits into
mainfrom
arreglo/saij-contrato-nuevo

Conversation

@leocagli

Copy link
Copy Markdown
Collaborator

SAIJ cambio el contrato de sus respuestas y el smoke test lo detecto, que es exactamente para lo que fue escrito. En main limpio venia 0 de 8, con 6 fallos y 2 saltados.

Las dos causas

1. Un nivel document nuevo en el medio. Lo que estaba en abs.content hoy esta en abs.document.content. El test buscaba abs.content.titulo, no lo encontraba, y caia al fallback abs.document, que antes era un string y hoy es un objeto. De ahi los seis 'titulo' no es string.

2. El filtro de legislacion ahora lleva tilde. Comprobado contra la API:

filtro resultados
Tipo de Documento/Legislacion 0
Tipo de Documento/Legislación 1

Por eso ese test se salteaba en silencio. Un SKIP es el peor de los tres estados en un repo legal: no falla, asi que nadie lo mira, y mientras tanto la busqueda de legislacion no estaba cubierta por nada.

El titulo se llama distinto en cada tipo

Ninguno coincide, verificado contra la API:

tipo campo
Jurisprudencia titulo
Doctrina titulo-doctrina
Legislacion titulo-norma

Ahora va por una funcion que los cubre a los tres.

Compatibilidad

Se aceptan las dos formas de respuesta, la vieja y la nueva, para que el test no se rompa si SAIJ vuelve atras.

Resultado

antes:  Total: 8  Correctos: 0  Fallos: 6  Saltados: 2   (exit 1)
ahora:  Total: 4  Correctos: 4  Fallos: 0  Saltados: 0   (exit 0)

Esto desbloquea el CI del repo: hasta ahora aprobar los workflows de cualquier PR abierto lo iba a poner en rojo por esta causa ajena a el.

🤖 Generated with Claude Code

leocagli and others added 2 commits August 30, 2026 14:17
…islacion

SAIJ cambio dos cosas y el smoke test las detecto, que es para lo que fue
escrito. Venia 0 de 8 con 6 fallos y 2 saltados.

Primera: las respuestas ahora vienen envueltas en un nivel `document`. Lo
que estaba en `abs.content` esta en `abs.document.content`. El test buscaba
`abs.content.titulo`, no lo encontraba, y caia al fallback `abs.document`,
que antes era un string y hoy es un objeto: de ahi los seis
"'titulo' no es string".

Segunda: el filtro de legislacion ahora lleva tilde. "Legislacion" devuelve
cero resultados y "Legislación" devuelve. Por eso ese test se salteaba en
silencio, que en un repo legal es el peor de los tres estados.

El titulo ademas se llama distinto en cada tipo y ninguno coincide:
`titulo` en jurisprudencia, `titulo-doctrina` en doctrina, `titulo-norma`
en legislacion. Va por una funcion que los cubre a los tres.

Las dos formas de respuesta se aceptan, la vieja y la nueva, para que el
test no se rompa si SAIJ vuelve atras.

Queda 4 de 4, sin saltados.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
El workflow pedia `cache: "npm"` en setup-node y el repo no tiene lockfile.
setup-node aborta el job al no encontrarlo, asi que moria en "Setup Node.js"
y el paso de tests quedaba skipped.

O sea que el CI de este repo nunca corrio, ni una vez, desde que se agrego
el 17 de agosto. Y el log decia "Smoke tests failed", que es peor que fallar:
no habia fallado ningun test porque no se ejecuto ninguno.

No hay nada que cachear. El smoke usa fetch pelado, sin dependencias.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@gitar-bot

gitar-bot Bot commented Aug 30, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

Fixes the SAIJ smoke test by adapting to API contract changes: handles the new nested document level in responses, corrects the legislation filter to include the accent (Legislación), and maps the correct title field for each document type (jurisprudencia, doctrina, legislación). Maintains backward compatibility with the previous response format. Test results improve from 0/8 passing to 4/4, unblocking CI workflows.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Important

Your trial ends in 2 days — upgrade now to keep code review, CI analysis, auto-apply, custom automations, and more.

Was this helpful? React with 👍 / 👎 | Gitar

@leocagli
leocagli merged commit 7fcb27c into main Aug 30, 2026
2 checks passed
@leocagli
leocagli deleted the arreglo/saij-contrato-nuevo branch August 30, 2026 17:46
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