Feature/phpunit #28
Feature/phpunit #28
Conversation
lpirola
force-pushed
the
feature/phpunit
branch
5 times, most recently
from
59a13e0
to
b98f3f8
Compare
lpirola
force-pushed
the
feature/phpunit
branch
5 times, most recently
from
391db43
to
01a2204
Compare
lpirola
force-pushed
the
feature/phpunit
branch
3 times, most recently
from
f2616c3
to
09f9787
Compare
| name: Testing | ||
|
|
||
| on: | ||
| pull_request: |
There was a problem hiding this comment.
@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: '**'
| @@ -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.
@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.
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