2017-02-10 1 views
0

Вот мой метод испытания:JUnit Утверждающие нуль - с помощью шлейфов и издевается

@Test 
public void shouldReturnUser_whenPassedUser() { 
    // given 
    when(userStub.getUsername()).thenReturn(defaultName); 
    when(userStub.getPassword()).thenReturn(defaultPassword); 
    when(userStub.getEmail()).thenReturn(defaultEmail); 

    // when 
    User savedUser = userServiceBean.saveUser(userStub); 

    // then 
    assertNotNull(savedUser); 
    verify(userRepository, times(1)).save(any(User.class)); 
} 

Вот мои настройки:

@Spy 
@InjectMocks 
private UserServiceBean userServiceBean; 

@Mock 
private UserRepository userRepository; 

@Mock 
private Principal principal; 

@Mock 
private PasswordEncoder passwordEncoder; 

@Mock 
private User userStub; 

private String defaultName = "user"; 
private String defaultPassword = "password"; 
private String defaultEmail = "[email protected]"; 

@Before 
public void init() { 
    MockitoAnnotations.initMocks(this); 
} 

А вот мой метод userService.saveUser:

public User saveUser(User user) { 
    User newUser = new User(user.getUsername(), user.getEmail(), passwordEncoder.encode(user.getPassword())); 
    return userRepository.save(newUser); 
} 

И мои результаты тестирования:

java.lang.AssertionError 
at org.junit.Assert.fail(Assert.java:86) 
at org.junit.Assert.assertTrue(Assert.java:41) 
at org.junit.Assert.assertNotNull(Assert.java:712) 
at org.junit.Assert.assertNotNull(Assert.java:722) 
at com.doublemc.services.UserServiceBeanTest.shouldReturnUser_whenPassedUser(UserServiceBeanTest.java:93) 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
at java.lang.reflect.Method.invoke(Method.java:498) 
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) 
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) 
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) 
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17) 
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26) 
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325) 
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78) 
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57) 
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290) 
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71) 
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288) 
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58) 
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268) 
at org.junit.runners.ParentRunner.run(ParentRunner.java:363) 
at org.junit.runner.JUnitCore.run(JUnitCore.java:137) 
at com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:68) 
at com.intellij.rt.execution.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:51) 
at com.intellij.rt.execution.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:237) 
at com.intellij.rt.execution.junit.JUnitStarter.main(JUnitStarter.java:70) 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
at java.lang.reflect.Method.invoke(Method.java:498) 
at com.intellij.rt.execution.application.AppMain.main(AppMain.java:147) 

Почему мой метод возвращает значение null? Я использую stubUser, и я ему точно говорю, что возвращать каждому получателю (как я делаю в моем методе saveUser), и он все равно возвращает null? Как мне изменить этот тест на работу?

+0

Try для отладки кода, с этим вы увидите, что именно не вводился (это то, что я подозреваю), а затем добавить этот результат здесь, это поможет. –

+1

КПП. лучше использовать реальные объекты вместо макетов, где это возможно. Это гарантирует, что вы не пропустите слишком много от реальности. Поэтому я предлагаю создать реальный объект 'User' вместо макета со всеми его издеваемыми методами. –

ответ

3

Вы забыли определить макет поведения для макета userRepository. Таким образом, Mockito использует значение по умолчанию null для save.

Определить поведение, как это:

when(userRepository.save(any(User.class))).thenReturn(new User("saved name", "saved email", "saved password")); 
+0

Хороший улов. Добавление некоторого примера того, что он должен сделать, сделает этот ответ лучше, когда (userRepository) .save (..... '+1 в любом случае. –

+0

Что вы подразумеваете под« define mock behavior »? Извините, я очень новичок в Mockito. – doublemc

+0

@doublemc Mocks - это именно то, что они могут быть использованы вместо «реальных» экземпляров классов, но они ничего не сделают, если вы не скажете им. Итак, в этом примере мы рассказываем наш макет userRepository, чтобы вернуть конкретный экземпляр пользователя, когда мы вызываем метод «save». Это поведение, которое мы называем в насмешках. Другими словами, «когда я вызываю этот метод с этими аргументами, верните этот объект». – ayahuasca

Смежные вопросы