-
Notifications
You must be signed in to change notification settings - Fork 2
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
Develop #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- mapeamento correto de métodos HTTP - uso de logger - nomenclatura de variáveis - tratamento de valores nulos - organização de imports - melhorias de legibilidade
- templates HTML refletem modificações - depedências em serviços refletem mudanças
- logger de erros - variáveis locais ao invés de atributos
- polimorfismo no uso de generics - agrega controle de roles em declaração única de url - sobrecarga de métodos na classe de UsuarioSistema
- nomenclatura de variáveis - construtor vazio - retorno de método sem variável auxiliar
- nomenclatura de variáveis - uso de logger - retira possibilidade de certos exploits em xml - convenções de código
- nomenclatura de variáveis
- nomenclatura de variáveis
- nomenclatura de variáveis
Correções gerais sugeridas pelo Sonar
…-nao-exibidos Ajustando consulta de caixas para retornar caixas fechados na tela de caixas
…-data-abertura-caixa Corrigindo filtro por data de abertura de caixa. Closes ddszb#8
…s-produto Ajustando Status de Produto ficando fixo no "ATIVO".
Adicionando os testes unitarios das classes de serviço Ajuste, AjusteProduto
Finalizando os testes de unidade das classes que faltavam.
…fone-salvar-pessoa Ajustes no campo de telefone no cadastro de pessoa. Closes ddszb#4
correção das mudanças feitas para execução dos testes. Utilizando singleton e a anotação SpringBootTest
Testes unidade
Correção usuário em Testes unidade
Test/integracao
Aumentando a cobertura dos testes de unidade em pelo menos 80%
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merge o branch develop de ddszb/pdv para o master