-
Notifications
You must be signed in to change notification settings - Fork 6
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
Feature/phpunit #28
Feature/phpunit #28
Conversation
59a13e0
to
b98f3f8
Compare
391db43
to
01a2204
Compare
f2616c3
to
09f9787
Compare
name: Testing | ||
|
||
on: | ||
pull_request: |
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.
@lpirola analisando especificamente o momento de executar os testes fico pensando se faz sentido executar em todos os commits, dessa forma conseguimos identificar o problema de forma mais rápida. Vou mandar aqui alguns pontos positivos e negativos baseado das duas abordagens:
- Execução dos Testes em Cada Commit
Vantagens:
- Detecta erros e regressões rapidamente, proporcionando feedback imediato ao desenvolvedor.
- Garante que o código na branch esteja sempre em um estado funcional.
Desvantagens:
- Pode ser demorado se o conjunto de testes for grande, afetando a produtividade.
- Pode ser excessivo em projetos menores ou com commits muito frequentes.
- Execução dos Testes no Merge Request (Pull Request) para a develop
Vantagens:
- Permite revisão de código com confiança de que as alterações não quebram funcionalidades existentes.
- Centraliza a validação do código em momentos cruciais, como a integração de novas funcionalidades.
Desvantagens:
- O feedback não é tão imediato quanto nos commits.
- Problemas podem ser detectados tardiamente, após várias alterações já terem sido feitas.
Para adicionar em todos os commits podemos usar:
on:
push:
# only trigger on branches, not on tags
branches: '**'
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.
feito!
@@ -80,7 +80,7 @@ protected function _requireAuthentication() { | |||
if($app->request->isAjax() || $app->request->getHeaderLine('Content-Type') === 'application/json'){ | |||
$app->view->controller->errorJson(\MapasCulturais\i::__('Esta ação requer autenticação'), 401); | |||
}else{ | |||
$app->redirect($app->controller('auth')->createUrl(''), 302); | |||
$app->redirect($app->controller('auth')->createUrl(''), 401); |
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.
@lpirola uma possibilidade também seria usar a própria contante da classe Response do Http Foundation do symfony.
Exemplo de uso:
use Symfony\Component\HttpFoundation\Response;
$app->redirect($app->controller('auth')->createUrl(''), Response::HTTP_UNAUTHORIZED);
Mais detalhes podem ser acessados aqui: https://symfony.com/doc/current/components/http_foundation.html#response
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.
Preferi manter assim para não misturar os frameworks. Podemos refletir melhor sobre se vamos incluir o Symfony na pasta do Slim.
Atualiza instituições disponíveis para campo de dados bancários
Descrição
Adicionar capacidade de testar de forma unitária o código php
Validação
Disparar gatilho de pipeline
Issues relacionadas