Camada de domínio vs infra e sua relação com repositório de dados

Rapaiz, se for para melhorar performance, acho que deve filtrar no banco direto msm. Claro, cada caso é um caso diferente (concordo em deixar a camada de dominio o mais redonda possível), mas deixar um sistema lento e com possíveis erros de estouro de memória é complicado.

Acho que um filtro na query não eh bem uma regra de negocio, mas um aspecto téccnico usado por alguma regra de negócio.

Por exemplo, se a regra é retornar todos os usuário ativos, na consulta vc terá que filtrar apenas os usuário ativos, mas a regra msm. Sacou?

Realmente pode ser confuso, mas em termos de um domínio bem isolado, imagine que vc tenha uma interface de repositório com um método recuperarUsuariosAtivos(), isso já eh a definição da regra por parte do sistema (é o que ele precisa para implementar determinado requisito). Agora, como ele é realizado, já é um detalhe técnico (seja pelo filtro direto na query ou pelo código). Ainda mais se tratando de um aspecto não funcional, como performance.

A regra de negócio foi definida no método recuperarUsuariosAtivos(), quem que vai implementar e como não importa, entendido. Porém quando a regra de negócio foi implementada dentro do domínio testei a unidade muito facilmente, agora se jogar essa regra pro Repository vou ter que ficar mockando, isso ao meu ver é bem ruim e trabalhoso.

Concordo com o que você disse, só estou levantando outras situações que percebi e que me geraram dúvidas. Ainda estou estudando sobre como funciona esse “descolamento” da entidade de domínio de sua representação persistida.

Isso é muito comum, mas muito comum mesmo!

Trabalhoso concordo, realmente dá mais trabalho escrever testes onde precisamos utilizar mocks, agora, sobre ser ruim eu discordo totalmente.

Concordo que o que vc disse. Ao alterar isso para a query no repositório, para testar efetivamente (em relação ao filtro na query), teria que ser um teste de integração usando algum banco em memória ou alguma base de teste já preparada.

Qualquer outro código que for usar esse método do repositorio que faz esse filtro, unitário com mock resolve bem e não é ruim (como dito pelo @Jonathan_Medeiros).

Acho que me expressei mal, não quis dizer que acho ruim mockar, mas sim que antes as regras eram testadas facilmente no domínio e agora preciso realizar mais passos pra testar direto no banco. Uso mocks o tempo todo, hehe.

Quando a regra estava implementada no domínio tinha alguns casos de testes pra essa regra já implementados, como tirei a regra do domínio e mandei pro repositório apaguei o método de produção e os testes, tive um sentimento ruim ao fazer isso, hahahah.

Agora entendi perfeitamente o ponto!

Uma alternativa que já foi sugerida e eu considero super boa seria implementar um teste de integração nesse caso, apesar de ser um pouco mais custoso o ganho com ele neste seu cenário acredito que seria extremamente válido.