Erro de Teste Unitário com Java e Mockito - Resolvido

Pessoal, boa tarde.

Tudo bem.

Tenho uma classe de tipo Repository, para a comunicação JPA, a minha base e outra classe Service, para a realização das regras de negócios.

Entretanto, segue a classe responsável:

package br.com.projeto.minhasfinancas;


import br.com.projeto.minhasfinancas.executavel.Principal;
import br.com.projeto.minhasfinancas.model.entity.Usuario;
import br.com.projeto.minhasfinancas.model.repository.UsuarioRepository;
import br.com.projeto.minhasfinancas.service.UsuarioService;


import br.com.projeto.minhasfinancas.service.impl.UsuarioServiceImpl;
import org.checkerframework.common.reflection.qual.Invoke;
import org.junit.Assert;
import org.junit.jupiter.api.Assertions;
import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.extension.ExtendWith;

import org.mockito.InjectMocks;
import org.mockito.Mock;
import org.mockito.Mockito;
import org.mockito.Spy;
import org.mockito.junit.jupiter.MockitoExtension;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.SpringBootConfiguration;
import org.springframework.boot.test.context.SpringBootTest;
import org.springframework.boot.test.mock.mockito.MockBean;
import org.springframework.context.annotation.Configuration;
import org.springframework.context.annotation.Profile;
import org.springframework.dao.DataIntegrityViolationException;
import org.springframework.lang.NonNull;
import org.springframework.test.annotation.DirtiesContext;
import org.springframework.test.context.ActiveProfiles;
import org.springframework.test.context.junit.jupiter.SpringExtension;
import org.springframework.test.context.transaction.TestTransaction;
import org.springframework.transaction.annotation.Transactional;
import static org.assertj.core.api.Assertions.assertThat;

import java.util.*;

import static org.mockito.Mockito.*;


@SpringBootTest(classes = TestApplication.class)//, webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT)
@ActiveProfiles("dbproducao")
@ExtendWith({
        MockitoExtension.class
        })
public class UsuarioRepositoryTest {


    @Mock
    private UsuarioRepository usuarioRepository;

    @Autowired
    //@InjectMocks
    private UsuarioServiceImpl usuarioServiceImpl;

    private Usuario usuario = new Usuario();

    @BeforeEach
    public void inicializarDados(){
        usuario=Usuario.builder()
                .nome("Matheus Santos Andrade")
                .email("matheusbsi1992@gmail.com")
                .senha("123456")
                .build();

    }

    @Test
    public void aDeveVerificarInserirDados(){
        //cenario
        when(usuarioRepository.save(any(Usuario.class))).thenReturn(usuario);

        //acao
        Usuario salvarUsuario = usuarioServiceImpl.salvarUsuario(usuario);
        //assertThat(salvarUsuario.getEmail()).isNotNull();
        Assertions.assertEquals("matheusbsi1992@gmail.com",salvarUsuario.getEmail());

        //verificacao
        verify(usuarioRepository,times(1)).save(usuario);
        verifyNoMoreInteractions(usuarioRepository);
    }

    @Test
    public void aDeveVerificarQueExisteUmEmail(){
        //acao
        when(usuarioRepository.existsByEmail(usuario.getEmail())).thenReturn(Boolean.TRUE);

        boolean resultadoVerificado = usuarioServiceImpl.validarEmail(usuario.getEmail());

        //verificao
        Assertions.assertTrue(resultadoVerificado);

        verify(usuarioRepository,times(1)).existsByEmail(usuario.getEmail());
        verifyNoInteractions(usuarioRepository);

    }

}

Todavia, sempre que eu executo, o meu método responsável por inserção obtenho as devidas informações, logo abaixo.

Logo abaixo com a seguinte informação:

Wanted but not invoked:
usuarioRepository.save(
    Usuario(idusuario=123, nome=Matheus Santos Andrade, email=matheusbsi1992@gmail.com, senha=123456)
);
-> at br.com.projeto.minhasfinancas.UsuarioRepositoryTest.aDeveVerificarInserirDados(UsuarioRepositoryTest.java:81)
Actually, there were zero interactions with this mock.

Wanted but not invoked:
usuarioRepository.save(
    Usuario(idusuario=123, nome=Matheus Santos Andrade, email=matheusbsi1992@gmail.com, senha=123456)
);
-> at br.com.projeto.minhasfinancas.UsuarioRepositoryTest.aDeveVerificarInserirDados(UsuarioRepositoryTest.java:81)
Actually, there were zero interactions with this mock.

	at br.com.projeto.minhasfinancas.UsuarioRepositoryTest.aDeveVerificarInserirDados(UsuarioRepositoryTest.java:81)
	at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
	at java.base/java.lang.reflect.Method.invoke(Method.java:578)
	at org.junit.platform.commons.util.ReflectionUtils.invokeMethod(ReflectionUtils.java:727)
	at org.junit.jupiter.engine.execution.MethodInvocation.proceed(MethodInvocation.java:60)
	at org.junit.jupiter.engine.execution.InvocationInterceptorChain$ValidatingInvocation.proceed(InvocationInterceptorChain.java:131)
	at org.junit.jupiter.engine.extension.TimeoutExtension.intercept(TimeoutExtension.java:156)
	at org.junit.jupiter.engine.extension.TimeoutExtension.interceptTestableMethod(TimeoutExtension.java:147)
	at org.junit.jupiter.engine.extension.TimeoutExtension.interceptTestMethod(TimeoutExtension.java:86)
	at org.junit.jupiter.engine.execution.InterceptingExecutableInvoker$ReflectiveInterceptorCall.lambda$ofVoidMethod$0(InterceptingExecutableInvoker.java:103)
	at org.junit.jupiter.engine.execution.InterceptingExecutableInvoker.lambda$invoke$0(InterceptingExecutableInvoker.java:93)
	at org.junit.jupiter.engine.execution.InvocationInterceptorChain$InterceptedInvocation.proceed(InvocationInterceptorChain.java:106)
	at org.junit.jupiter.engine.execution.InvocationInterceptorChain.proceed(InvocationInterceptorChain.java:64)
	at org.junit.jupiter.engine.execution.InvocationInterceptorChain.chainAndInvoke(InvocationInterceptorChain.java:45)
	at org.junit.jupiter.engine.execution.InvocationInterceptorChain.invoke(InvocationInterceptorChain.java:37)
	at org.junit.jupiter.engine.execution.InterceptingExecutableInvoker.invoke(InterceptingExecutableInvoker.java:92)
	at org.junit.jupiter.engine.execution.InterceptingExecutableInvoker.invoke(InterceptingExecutableInvoker.java:86)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.lambda$invokeTestMethod$7(TestMethodTestDescriptor.java:217)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.invokeTestMethod(TestMethodTestDescriptor.java:213)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.execute(TestMethodTestDescriptor.java:138)
	at org.junit.jupiter.engine.descriptor.TestMethodTestDescriptor.execute(TestMethodTestDescriptor.java:68)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$6(NodeTestTask.java:151)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$8(NodeTestTask.java:141)
	at org.junit.platform.engine.support.hierarchical.Node.around(Node.java:137)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$9(NodeTestTask.java:139)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:138)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:95)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.invokeAll(SameThreadHierarchicalTestExecutorService.java:41)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$6(NodeTestTask.java:155)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$8(NodeTestTask.java:141)
	at org.junit.platform.engine.support.hierarchical.Node.around(Node.java:137)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$9(NodeTestTask.java:139)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:138)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:95)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.invokeAll(SameThreadHierarchicalTestExecutorService.java:41)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$6(NodeTestTask.java:155)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$8(NodeTestTask.java:141)
	at org.junit.platform.engine.support.hierarchical.Node.around(Node.java:137)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.lambda$executeRecursively$9(NodeTestTask.java:139)
	at org.junit.platform.engine.support.hierarchical.ThrowableCollector.execute(ThrowableCollector.java:73)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.executeRecursively(NodeTestTask.java:138)
	at org.junit.platform.engine.support.hierarchical.NodeTestTask.execute(NodeTestTask.java:95)
	at org.junit.platform.engine.support.hierarchical.SameThreadHierarchicalTestExecutorService.submit(SameThreadHierarchicalTestExecutorService.java:35)
	at org.junit.platform.engine.support.hierarchical.HierarchicalTestExecutor.execute(HierarchicalTestExecutor.java:57)
	at org.junit.platform.engine.support.hierarchical.HierarchicalTestEngine.execute(HierarchicalTestEngine.java:54)
	at org.junit.platform.launcher.core.EngineExecutionOrchestrator.execute(EngineExecutionOrchestrator.java:147)
	at org.junit.platform.launcher.core.EngineExecutionOrchestrator.execute(EngineExecutionOrchestrator.java:127)
	at org.junit.platform.launcher.core.EngineExecutionOrchestrator.execute(EngineExecutionOrchestrator.java:90)
	at org.junit.platform.launcher.core.EngineExecutionOrchestrator.lambda$execute$0(EngineExecutionOrchestrator.java:55)
	at org.junit.platform.launcher.core.EngineExecutionOrchestrator.withInterceptedStreams(EngineExecutionOrchestrator.java:102)
	at org.junit.platform.launcher.core.EngineExecutionOrchestrator.execute(EngineExecutionOrchestrator.java:54)
	at org.junit.platform.launcher.core.DefaultLauncher.execute(DefaultLauncher.java:114)
	at org.junit.platform.launcher.core.DefaultLauncher.execute(DefaultLauncher.java:86)
	at org.junit.platform.launcher.core.DefaultLauncherSession$DelegatingLauncher.execute(DefaultLauncherSession.java:86)
	at org.junit.platform.launcher.core.SessionPerRequestLauncher.execute(SessionPerRequestLauncher.java:53)
	at com.intellij.junit5.JUnit5IdeaTestRunner.startRunnerWithArgs(JUnit5IdeaTestRunner.java:57)
	at com.intellij.rt.junit.IdeaTestRunner$Repeater$1.execute(IdeaTestRunner.java:38)
	at com.intellij.rt.execution.junit.TestsRepeater.repeat(TestsRepeater.java:11)
	at com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:35)
	at com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:235)
	at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54)

Posta o código do UsuarioService

Tente alterar isso:

de

 when(usuarioRepository.save(any(Usuario.class))).thenReturn(usuario);

por

 when(usuarioRepository.save(usuario)).thenReturn(usuario);

Você não passou a instância usuario para o mock, você passou any(Usuario.class), por isso o verify falha.

Você pode alterar o mock passando a instância ou alterar o verify passando any.

package br.com.projeto.minhasfinancas.service.impl;

import br.com.projeto.minhasfinancas.exceptions.RegraNegocioExecption;
import br.com.projeto.minhasfinancas.model.entity.Usuario;
import br.com.projeto.minhasfinancas.model.repository.UsuarioRepository;
import br.com.projeto.minhasfinancas.service.UsuarioService;
import lombok.extern.slf4j.Slf4j;
import org.springframework.beans.factory.annotation.Autowired;

import org.springframework.stereotype.Service;

import static org.springframework.util.Assert.notNull;


@Service
@Slf4j
public class UsuarioServiceImpl implements UsuarioService {


    private UsuarioRepository usuarioRepository;

    @Autowired
    public UsuarioServiceImpl (UsuarioRepository usuarioRepository){
        this.usuarioRepository = usuarioRepository;
    }

    @Override
    public Usuario autenticar(String email, String senha) {

        return null;
    }
    @Override
    public Usuario salvarUsuario(Usuario usuario) {
        validarEmail(usuario.getEmail());
        return usuarioRepository.save(usuario);
    }

    @Override
    public boolean validarEmail(String email) {
        try{
            notNull(email,"Email é obrigatório!!!");

            return usuarioRepository.existsByEmail(email);
        }catch (Exception ex){
            throw new RegraNegocioExecption(String.format("Já existe um usuário cadastrado com este =%s ",email," "));
        }






    }
}

 @Test
    public void aDeveVerificarInserirDados(){
        //cenario
        //ADD -->>> lenient()
        lenient().when(usuarioRepository.save(usuario)).thenReturn(usuario);

        //acao
        Usuario salvarUsuario = usuarioServiceImpl.salvarUsuario(usuario);
        //assertThat(salvarUsuario.getEmail()).isNotNull();
        Assertions.assertEquals("matheusbsi1992@gmail.com",salvarUsuario.getEmail());

        //verificacao
        //verify(usuarioRepository,times(1)).save(usuario);
        verifyNoMoreInteractions(usuarioRepository);
    }

Com a verificação não passa. Mas conforme a indicação do lenient, entao eu consigo realizar o meu devido teste de unidade.

Se não me engano, usar o lenient torna o mock “tolerante”, que de certa forma, enfraquece o teste.

Ao pensar, que ainda está realizando um devido assertion, mas de forma a analisar, o processo. E então mesmo sem o lenient , o código não passa ou falha. O ruim é que ele não faz em tese uma devida verificação, mas sim só analisa que o código, em tese foi inserido e logo após verifica que aquele determinado valor retornou algo e que não se “repete” mais.

Tenta modificar seu teste para algo assim:

@ExtendWith(MockitoExtension.class)
class UsuarioServiceTest {

    @Mock
    private UsuarioRepository usuarioRepository;

    @InjectMocks
    private UsuarioServiceImpl subject;

    @BeforeEach
    void setup(){
        Mockito.reset(usuarioRepository);
    }

    @Test
    void aDeveVerificarInserirDados(){
        final Usuario expected = Usuario.builder()
                .nome("Matheus Santos Andrade")
                .email("matheusbsi1992@gmail.com")
                .senha("123456")
                .build();

        when(usuarioRepository.existsByEmail(anyString())).thenReturn(false);
        when(usuarioRepository.save(any(Usuario.class))).thenReturn(expected);

        final Usuario actual = subject.salvarUsuario(expected);
        // Para que a asserção funcione como esperado é necessário que sua classe Usuario tenha implementado os métodos equals/hashCode com as propriedades de comparação corretas
        Assertions.assertEquals(usuario, actual);

        verify(usuarioRepository).existsByEmail("matheusbsi1992@gmail.com");
        verify(usuarioRepository).save(
            ArgumentMatchers.argThat(user -> {
                Assertions.assertEquals(user.getName(), "Matheus Santos Andrade");
                Assertions.assertEquals(user.getEmail(), "matheusbsi1992@gmail.com");
                Assertions.assertEquals(user.getSenha(), "123456");
                return true;
            })
        );
    }

}

OK.

Então, o equals e hashcode já estão implementados através do lombok.

Irei analisar para verificar o devido processo.

A questão se injetar o Service, é que o devido processo não chega a pesquisar o devido item e não chega também a inserir.

E se chegar a verificar se o Repository é chamado, então o processo não finaliza.

O exemplo que mostrei é de um teste unitário, por isso os comportamentos externos são tratados como mock, como é o caso do repository.

Agora, se a ideia é fazer um teste de integração a abordagem muda completamente, desde o setup da classe até a própria escrita dos cenários.

Exato. Se eu retirar os valores de assinatura da injeção isto indica, que será algo relacionado, para a integração.

Então, eu deve realizar a estrutura de unidade, mesmo. Eu francamente, me passei, só pensando na parte de inserção dos dados, e não em “mockar” eles.

Para este caso, eu teria que alterar a minha estrutura, para receber propriamente uma lista de objetos e não uma classe, para verificar se existe ou não aquele determinado item.

Então, eu sugiro encerrar este devido processo aqui em aberto, para sucesso. Isto já resolve e de quaisquer formar, irei realizar um pequeno estudo, em Udemy , com o uso de Mockito e Junit 4. Embora, já exista a questão de:

https://www.udemy.com/course/testes-unitarios-em-java/ ou

https://www.udemy.com/course/testes-unitarios-junit5/

Eu agradeço e muito.