search

npedidos / rest-api

NPedidos es una aplicación que permite la gestión simple de pedidos de menús de comidas, por fechas.
MIT License
8 stars 16 forks source link

Crear representación del fichero properties #1

Closed nmarulo closed 1 year ago

nmarulo commented 1 year ago

Crear una clase llama AppProperties en el paquete red.softn.npedidos.configuration que represente las propiedades que empiezan por app del fichero application.properties.

Luego reemplazar las propiedades usadas en las clases WebSecurityConfig y AppConfig.

juanguirp commented 1 year ago

¡Hola @nmarulo! ¿Puedo colaborarte con este ajuste? Yo podría añadir una clase de configuración con el nombre que indicas (AppProperties.java) e inyectarla en las 2 clases que mencionas (como haces uso de una etiqueta de Lombok para inyectar por el constructor, aplicaría lo mismo para esta nueva clase). ¡Saludos!

nmarulo commented 1 year ago

Hola @juanguirp, todo tuyo. 👍

juanguirp commented 1 year ago

¡Gracias @nmarulo ! 🤟🏻

He aplicado los cambios que se necesitan para resolver el issue. Tu servicio fue ejecutado sin problemas y los valores de las propiedades fueron cargadas en el nuevo bean (estos logs no fueron agregados en el PR):

imagen

Adicional a esto, he añadido unos comentarios en el application.properties para agrupar las propiedades por tipo y facilitar la búsqueda visual de las que son creadas por ti. Dejame saber si te sirve este cambio o prefieres que lo deje como estaba previamente:

imagen
nmarulo commented 1 year ago

Lo he visto brevemente y se ve correcto. 👏 En cuanto tenga un momento lo reviso bien.

Y sobre los comentarios del fichero application.properties, me parece genial, muchas gracias. 😄

juanguirp commented 1 year ago

Hola! Acabo de mandar un ajuste en las unitarias 😆 Me hicieron falta un par de @Import para utilizar la nueva configuración 😅 Los test unitarios pasan correctamente:

imagen