search

ymoreiratiti / consultaNFE

Retorna os dados de uma NFE (Nota Fiscal Eletrônica), baseado no QRCode
Apache License 2.0
51 stars 24 forks source link

Testes unitários necessitam de mock específico #10

Open adrianteixeira opened 3 years ago

adrianteixeira commented 3 years ago

Dando uma olhada nos testes adicionados recentemente vi que eles acessam o serviço externo para a consulta.. Isso fere a prática dos testes unitários testarem um pedaço minimo da solução sem dependencia externa (banco, servicos, etc..)

Seria interessante alterar a nomenclatura atual para Testes de Integração ou alterar a solução de testes para modelos mockados onde já temos os dados de origem dentro do proprio build

ymoreiratiti commented 3 years ago

Podemos alterar para Testes de Integração.

O Ideal é fazer testes com o serviço real da NFE, visto que eles podem alterar o layout das páginas sem aviso prévio.