-
Notifications
You must be signed in to change notification settings - Fork 75
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
Creating first few integration tests #571
Creating first few integration tests #571
Conversation
Salve amigão @henriqueholtz. Desculpe a demora. Parece que tem alguns conflitos nessa PR. Pode dar uma olhada? |
Opa, ainda está como draft (não está finalizado), to fazendo aos poucos haha |
|
salve @henriqueholtz já podemos fazer o code review e merge? |
Opa, sim, está pronto p/ review... |
/// <summary> | ||
/// Only should be used for integration tests | ||
/// </summary> | ||
public static bool IgnoreMigrations = false; |
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.
Pode falar um pouco desse cara? Em que contexto ele fica true?
Amigão @henriqueholtz outra pergunta. Em que momento esse teste vai rodar? |
Book
:[GET] api/book/AvailableBooks_All
Meetup
:[GET] api/Meetup?page={page}&pageSize={pageSize}&upcoming={upcoming}
(7x)[GET] api/Meetup/search?criteria={criteria}
(3x)Category
:[GET] api/category
All these new tests will be already executed on Github Action.