Adiciona testes unitários para classes utilitárias #44
Conversation
joaomarcosribeiroc
changed the title
There was a problem hiding this comment.
Fiz uma sugestão para legibilidade. Contudo, não irei aprovar o PR devido faltar fazer os testes para os outros Utils: Generator e Participant.
Nestes outros testes, será necessário o Mockito, caso não conheça, me chama no pv que te passo um overview dele. Basicamente, ele irá mocar os parametros passados.
|
|
||
| String resultingFileName = CertificateUtil.createFileName(certificate); | ||
|
|
||
| Assertions.assertEquals(resultingFileName, new StringBuilder(fileName).append(".").append(fileExtension).toString()); |
There was a problem hiding this comment.
Poderia melhorar a legibilidade do código aqui, usando string no expected. Geralmente em testes, o codigo inverte, geralmente é melhor manter o texto para deixar mais legivel do que tornar dinamico, pois se alguem mudar algo, vai quebrar e não passará em branco.
Por exemplo:
Assertions.assertEquals(resultingFileName, "filename.fileExtension");
Desta forma, o dev bate o olho e sabe qual o retorno esperado.
There was a problem hiding this comment.
Certo. Estarei fazendo essas melhorias.
joaomarcosribeiroc
force-pushed
the
issue_42
branch
from
974eee7 to
9b1fdae
Compare
…t será removido após correção do erro e finalização do teste para manter a limpeza do histórico
alexferreiradev
force-pushed
the
issue_42
branch
from
9b1fdae to
fef5200
Compare
|
Como não teve commits novos, irei fechar ele para não confundir pessoal do gojava. |
Adiciona testes unitários para a classe utilitária CertificateUtil.
Para testar, utilize o comando:
mvn test -Dtest=CertificateUtilTestResolve o issue #42.