Inserção de Pontos na Carteira com Design Patterns#15
Inserção de Pontos na Carteira com Design Patterns#15maria-luiza-duda wants to merge 1 commit intofructuoso:mainfrom
Conversation
fructuoso
left a comment
fructuoso
left a comment
There was a problem hiding this comment.
Muito obrigado pelo seu PR, seguem alguns comentários, espero que contrubuam para você continue a estudar e aprender cada vez mais.
A classe src/Infra.Repository.Detran.Tests/DetranPontosCarteiraFactory.cs foi criada mas ela não testa absolutamente nada, seria legal você implementar ao menos um teste.
Senti falta de você registrando o Factory como eu fiz no código https://github.com/fructuoso/DesignPatternSamples/blob/main/src/Infra.Repository.Detran.Tests/DependencyInjectionFixture.cs#L35-L40 sem o registro o Factory não vai saber qual tipo concreto deve ser implementado, lembrando que esse ajuste se faz necessário tanto no DependencyInjectionFixture quanto no Startup.
| .AddTransient<DetranRJVerificadorDebitosRepository>() | ||
| .AddTransient<DetranRSVerificadorDebitosRepository>() | ||
| .AddSingleton<IDetranVerificadorDebitosFactory, DetranVerificadorDebitosFactory>(); | ||
| .AddSingleton<IDetranVerificadorDebitosFactory, DetranVerificadorDebitosFactory, IDetranPontosCarteiraFactory, DetranPontosCarteiraFactory>(); |
There was a problem hiding this comment.
O ideal é criar uma nova linha.
2 participants
Inserção de um novo serviço, Pontos na Carteira, com o padrão Decorator de Design de Projetos.