Repository navigation
feat(productos): el ADMIN puede administrar la carta y el menu del dia - #31
Merged
Merged
Conversation
Dar de alta, editar y borrar platillos era exclusivo de DEV. Se abre a ADMIN, que es el dueno y quien cambia precios de verdad. Sin esto, la pantalla de Platillos que el frontend le acaba de dar responderia 403. De paso se cierra un hueco que ya existia: el ADMIN veia Platillos del Dia en su menu, pero actualizarDia y desactivarDia no lo aceptaban, asi que la pantalla cargaba y fallaba al activar un platillo. El test compara conjuntos exactos, no contiene: avisa igual si a alguien se le va el permiso de mas hacia un MESERO. Se comprobo que sabe fallar metiendole MESERO al borrado a proposito.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Va en pareja con RestFoodF
feat/carrito-todos-y-platillos-admin. Merge los dos juntos: si entra solo el frontend, el ADMIN ve la pantalla de Platillos y recibe 403 al guardar.Qué cambia
POST,PUTyDELETE /productospasan dehasRole('DEV')ahasAnyRole('ADMIN','DEV'). Dar de alta un platillo, corregir un precio y borrar de la carta era exclusivo de DEV; el dueño es quien lo hace en la vida real.De paso se cierra un hueco que ya estaba: el ADMIN veía Platillos del Día en su menú, pero
PATCH /productos/{id}/diayPUT /productos/desactivar-dia/{categoriaId}no lo aceptaban. La pantalla cargaba y fallaba al activar un platillo.Qué NO cambia
Nadie más gana permisos. MESERO y REPARTIDOR siguen igual, y el tope de platillos activos por categoría (el que cuida los 5 renglones del PDF) sigue intacto.
El test
PermisosProductosTestcompara conjuntos exactos, no "contiene". Avisa en las dos direcciones: si a alguien se le va el permiso de menos y la pantalla se rompe, y —más importante— si se le va de más hacia un MESERO y cualquiera desde una tablet puede borrar platillos.Se comprobó que sabe fallar: metiéndole
MESEROal borrado a propósito, falla conexpected: <[DEV, ADMIN]> but was: <[ADMIN, DEV, MESERO]>.Local: todo verde menos
contextLoads, que necesita el MySQL que el CI levanta.