-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: automated integration testing process #105
Conversation
api/routes/router.go
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e se for mudar para lowercase. temos alterar a validação aqui também
!= development
Pr do splash era para estar listado nos commits dessa branch ? |
api/routes/routes.go
Outdated
UserController *controllers.UserController | ||
} | ||
|
||
func InitRoutes(controllers Controllers, c *chi.Mux) { | ||
|
||
c.Route("/api", func(r chi.Router) { | ||
r.Route("/pets", func(r chi.Router) { | ||
r.Get("/{id}", controllers.FindPetController.FindPet) | ||
r.Get("/{id}", controllers.PetController.FindPet) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
modifica para petID para deixar na mesma logica
infra/db/pet_repository.go
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Galera, vamos usar o sqlx como padrao?
https://github.com/jmoiron/sqlx
Ja usamos em Usuario
- add services to automate the integration testing process; - remove everything related to production because it's not needed; - make minor fixes and improvements.
- fix typos; - full support for environment parametrization in docker compose.
- and minor improvements.
f861e2e
to
4411d39
Compare
Preciso:
|
Feature
Services and a dedicated workflow for integration testing using docker compose.
Bugfix
Description
N/A
Cause
N/A
Solution
N/A
Changelog
N/AVisual evidences 🖼️
Checklist
Additional info
N/A