[Sugestão] Validação da classe ValidacaoTutorComLimiteDeAdocoes antes da otimização do código

Assim como a classe ValidacaoTutorComAdocaoEmAndamento, a classe ValidacaoTutorComLimiteDeAdocoes apresenta um código diferente do código testado na primeira resolução dos desafios. O código presente no projeto para download está desta forma:

@Component
public class ValidacaoTutorComLimiteDeAdocoes implements ValidacaoSolicitacaoAdocao {

    @Autowired
    private AdocaoRepository adocaoRepository;

    @Autowired
    private TutorRepository tutorRepository;

    public void validar(SolicitacaoAdocaoDto dto) {
        List<Adocao> adocoes = adocaoRepository.findAll();
        Tutor tutor = tutorRepository.getReferenceById(dto.idTutor());
        for (Adocao a : adocoes) {
            int contador = 0;
            if (a.getTutor() == tutor && a.getStatus() == StatusAdocao.APROVADO) {
                contador = contador + 1;
            }
            if (contador == 5) {
                throw new ValidacaoException("Tutor chegou ao limite máximo de 5 adoções!");
            }
        }
    }

}

De inicio podemos reparar em uma correção necessária para que a validação funcione, sendo ela remover a iniciação da variável contador de dentro do laço for e adiciona-la antes do inicio do mesmo, caso contrário ela será sempre zerada para cada elemento percorrido na lista e sua incrementação não passará de 1:

@Component
public class ValidacaoTutorComLimiteDeAdocoes implements ValidacaoSolicitacaoAdocao {

    @Autowired
    private AdocaoRepository adocaoRepository;

    @Autowired
    private TutorRepository tutorRepository;

    public void validar(SolicitacaoAdocaoDto dto) {
        List<Adocao> adocoes = adocaoRepository.findAll();
        Tutor tutor = tutorRepository.getReferenceById(dto.idTutor());
        int contador = 0;
        for (Adocao a : adocoes) {
            if (a.getTutor() == tutor && a.getStatus() == StatusAdocao.APROVADO) {
                contador = contador + 1;
            }
            if (contador == 5) {
                throw new ValidacaoException("Tutor chegou ao limite máximo de 5 adoções!");
            }
        }
    }

}

Por fim, os testes desenvolvidos para cobrir os casos possíveis dessa classe de validação foram implementados da seguinte maneira:

@ExtendWith(MockitoExtension.class)
class ValidacaoTutorComLimiteDeAdocoesTest {

    @InjectMocks
    private ValidacaoTutorComLimiteDeAdocoes validacaoTutorComLimiteDeAdocoes;
    @Mock
    private AdocaoRepository adocaoRepository;
    @Mock
    private TutorRepository tutorRepository;
    @Mock
    private SolicitacaoAdocaoDto dto;
    @Mock
    private Tutor tutor;
    @Mock
    private Pet pet;
    @Spy
    private List<Adocao> listaDeAdocoes = new ArrayList<>();
    @Spy
    private Adocao adocao;


    @Test
    void deveriaPermitirAdocaoPorTutorComLimite() {

        adocao = new Adocao(tutor, pet, "Motivo qualquer");
        adocao.marcarComoAprovada();
        listaDeAdocoes.add(adocao);
        BDDMockito.given(adocaoRepository.findAll()).willReturn(listaDeAdocoes);
        BDDMockito.given(tutorRepository.getReferenceById(dto.idTutor())).willReturn(tutor);

        Assertions.assertDoesNotThrow(() -> validacaoTutorComLimiteDeAdocoes.validar(dto));
    }

    @Test
    void deveriaNaoPermitirAdocaoPorTutorSemLimite() {

        adocao = new Adocao(tutor, pet, "Motivo qualquer");
        adocao.marcarComoAprovada();
        for(int i = 0; i < 5; i++)
            listaDeAdocoes.add(adocao);

        BDDMockito.given(adocaoRepository.findAll()).willReturn(listaDeAdocoes);
        BDDMockito.given(tutorRepository.getReferenceById(dto.idTutor())).willReturn(tutor);

        Assertions.assertThrows(ValidacaoException.class, () -> validacaoTutorComLimiteDeAdocoes.validar(dto));
    }

}

Gostaria de um feedback sobre minha implementação dos testes e também disponibilizar o código para os demais alunos com dificuldade nessa etapa.