Skip to content

fix(DEF-019): bloquear pedidos de un carrito vacio - #46

Merged
Ypz22 merged 2 commits into
mainfrom
fix/DEF-019-carrito-vacio
Jul 19, 2026
Merged

Ypz22 merged 2 commits into
mainfrom
fix/DEF-019-carrito-vacio

Conversation

@Ypz22

@Ypz22 Ypz22 commented Jul 19, 2026

Copy link
Copy Markdown
Collaborator

create_order usaba cart['total'] sin verificar que el carrito tuviera productos, permitiendo generar un pedido a partir de un carrito vacio (la creacion de carritos ya rechazaba productos=[] desde el fix de CP-24, pero un carrito vacio podia existir igual, ej. insertado directo en Mongo). Ahora create_order responde 400 si cart.productos esta vacio.

Cierra CP-23 (pedido de carrito vacio bloqueado).

Close #32

Ypz22 added 2 commits July 18, 2026 23:49
create_order usaba cart['total'] sin verificar que el carrito tuviera
productos, permitiendo generar un pedido a partir de un carrito vacio
(la creacion de carritos ya rechazaba productos=[] desde el fix de
CP-24, pero un carrito vacio podia existir igual, ej. insertado
directo en Mongo). Ahora create_order responde 400 si cart.productos
esta vacio.

Cierra CP-23 (pedido de carrito vacio bloqueado).
El default de FLASK_API en middleware.ts y astro.config.mjs cambio de
localhost a 127.0.0.1, pero las aserciones del test seguian esperando
localhost:5000, rompiendo test:coverage en CI.
@sonarqubecloud

Copy link
Copy Markdown

@Ypz22
Ypz22 merged commit 25a6139 into main Jul 19, 2026
2 checks passed
@Ypz22
Ypz22 deleted the fix/DEF-019-carrito-vacio branch July 19, 2026 04:59
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.

DEF-019 · Se permite guardar/comprar con carrito vacío

1 participant