Issue: Refatoração repositório Ada Gitlab
Repositório: 2019.1-ADA

Data: 18/05/2019
Status: Fechado

Closed
8 tasks done
Refatoração repositório Ada Gitlab
#209
joaovitor3 opened this issue on May 18, 2019 · 12 comments
Comments
@joaovitor3
Contributor
joaovitor3 commented on May 18, 2019 • 
Descrição da Issue
Realizar refatoração do repositório Ada gitlab

Tasks:
 Refatorar configuração do banco de dados
 Refatorar classe de relatório em menores métodos
 Adicionar codeclimate ao repositório
 Aumentar cobertura de código
 Criação de classe pai com código que esteja duplicado
Critérios de aceitação
 Cobertura de testes acima de 60%
 Classe pai com métodos que estavam duplicados
 Codeclimate no repositório
@joaovitor3 joaovitor3 added the EPS label on May 18, 2019
@joaovitor3 joaovitor3 added this to the Sprint 9 milestone on May 18, 2019
@joaovitor3 joaovitor3 assigned joaovitor3 and ErickGiffoni on May 18, 2019
@BrunoOliveiraDantas BrunoOliveiraDantas self-assigned this on May 21, 2019
@joaovitor3
Contributor
Author
joaovitor3 commented on May 21, 2019 • 
codeclimate adicionado ao repositório, conforme a foto a seguir:

image

A classe pai com métodos comuns à outras classes, também foi criada.

image

@joaovitor3
Contributor
Author
joaovitor3 commented on May 25, 2019
As duplicações foram diminuídas, conforme a imagem a seguir:

image

@ateldyfilho ateldyfilho unassigned ErickGiffoni on May 25, 2019
@sudjoao
Collaborator
sudjoao commented on May 26, 2019 • 
Essa US foi alocada a princípio na Sprint 9, e tornou-se dívida técnica para a Sprint 10.
O pareamento responsável para a Sprint 9 foi @joaovitor3 e @BrunoOliveiraDantas .
O responsável para a Sprint 10 mudou para @sudjoao.

@sudjoao
Collaborator
sudjoao commented on May 26, 2019 • 
Para essa Sprint ficaram de dividas os testes unitários, e o aumento da cobertura dos mesmo para 60% ou mais.

@sudjoao sudjoao self-assigned this on May 26, 2019
@vitorandos
Collaborator
vitorandos commented on May 26, 2019
Por favor, @joaovitor3 , como uma tarefas do papel de DevOps, garanta que os guias de instalação existam e mantenham-se atualizados. Essa é uma tarefa que vem sendo negligenciada desde o início do projeto e que é muito importante pois facilita seu próprio papel, evitando que os demais integrantes necessitem de você para subir o sistema. Obrigado.

@ateldyfilho ateldyfilho modified the milestones: Sprint 9, Sprint 10 on May 26, 2019
@joaovitor3
Contributor
Author
joaovitor3 commented on May 26, 2019
@vitorandos
Se quiser que seguimos os papéis a risca, seguiremos.
Mas quando tiverem impedimentos apenas encaminharei as mensagens para o papel correto, ok?

@vitorandos
Collaborator
vitorandos commented on May 26, 2019
Perfeito!

@vitorandos vitorandos mentioned this issue on May 26, 2019
Garantir exclusão do banco após execução de testes #229
Closed
5 tasks
@vitorandos
Collaborator
vitorandos commented on May 28, 2019
Para essa nova Sprint há um novo critério de aceitação que é a cobertura de testes acima de 80%. Idealmente, acima de 90%.

@sudjoao
Collaborator
sudjoao commented on May 29, 2019
A maioria dos testes já foram refatorados durante a semana, atualmente, como se pode se ver na imagem, a cobertura de testes está em 74%, os principais pontos que estão faltando testar são a parte do User e Webhook, que planejo terminar ainda hoje, o foco é conseguir uma cobertura de teste acima de 90%, tentando alcançar o 100%.
Segue a imagem citada:
Captura de tela de 2019-05-29 20-45-47

@sudjoao
Collaborator
sudjoao commented on May 31, 2019
Atualmente a cobertura de testes está em 91%, quase todos os testes que eram possíveis fazer atualmente foram feitos.
Segue a imagem com mais informações:
Captura de tela de 2019-05-31 23-51-04

@sudjoao
Collaborator
sudjoao commented on May 31, 2019
Além disso foi realizado algumas alterações na parte do report, antes existia apenas uma requisição, e como era uma requisição grande o tempo decorrido para ela rodar era muito grande, pensando nisso agora existem 5 requisições para pegar todos os dados necessários para o report.

sudjoao added a commit that referenced this issue on May 31, 2019
@sudjoao
#209 Change some actions to get gitlab data
904552f
@sudjoao sudjoao mentioned this issue on Jun 1, 2019
#209 Refatoração repositório Ada Gitlab #241
 Merged
@sudjoao
Collaborator
sudjoao commented on Jun 1, 2019 • 
A criação do pull request foi realizada, apesar da refatoração a parte de gerar o relatório ainda está levando um tempo consideravelmente alto para ser feito, isso será algo para ser levado e discutido durante a Sprint review.

@ateldyfilho ateldyfilho closed this as completed on Jun 1, 2019
joaovitor3 added a commit that referenced this issue on Jun 3, 2019
@joaovitor3
Merge pull request #241 from fga-eps-mds/RefatorarGitlab …

Contexto de uso: Não entrega do escopo planejado
Categoria: Pendência