Skip to content

Add/ci#33

Open
smyleface18 wants to merge 23 commits intoTrycatch-tv:mainfrom
smyleface18:add/ci
Open

Add/ci#33
smyleface18 wants to merge 23 commits intoTrycatch-tv:mainfrom
smyleface18:add/ci

Conversation

@smyleface18
Copy link
Contributor

  1. 🔧 Es la primera vez que configuro un flujo de CI, así que puede que no esté 100% perfecto, pero lo he probado y funciona correctamente con los tests

  2. 📦 En la PR anterior comenté que, como en el archivo main.go se importa directamente el paquete docs, es necesario generar primero la documentación Swagger.
    Por eso, agregué la instalación de swag y su ejecución (swag init -g src/cmd/main.go -o docs) antes de compilar o correr tests.

Generación de Swagger

3.🔐 El archivo .env.test lo construyo dentro del pipeline usando variables de entorno configuradas en GitHub Actions.
Estas variables son prácticamente las mismas que se usan en local.
Configuración de environment

4.🕒 Hago que la ejecución espere 2 segundos antes de correr los tests porque el contenedor Docker del entorno de pruebas a veces no está listo inmediatamente.
Sin este sleep, algunos tests fallan porque la DB aún no está lista para aceptar conexiones.

Espera de contenedor
  1. evidencia de que funciona en mi github
image

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

Comments