メソッドを使用してモックを使用する場合

Question

APIを使用してWebアプリケーションのテストを作成し、コントローラメソッドをテストします。コントローラは、私が嘲笑したいサービスメソッドを使います。 テストコード：

UserService.save（ユーザ）がNULLであった場合、それはいくつかのユニークなロング値にユーザのIDプロパティを設定呼ば

@Mock 
private UserService userService; 
@Mock 
private PasswordEncoder passwordEncoder; 
@InjectMocks 
private UserApi userApi; 

    MockitoAnnotations.initMocks(this); 
    mockMvc = MockMvcBuilders.standaloneSetup(userApi).build(); 

@Test 
public void addUser() throws Exception { 
    Role role = new Role(1L, "ROLE_USER"); 
    User user = new User (null, "Test user", "Test password", true, role); 
    when(roleService.findByName(role.getName())).thenReturn(role); 
    when(passwordEncoder.encode(user.getPassword())).thenReturn("Encoded password"); 
    when(userService.save(user)).thenReturn(1L); 


    mockMvc.perform(MockMvcRequestBuilders.post("/user").content(user.toJson())).andDo(print()) 
      .andExpect(status().isOk()) 
      .andExpect(content().contentType("application/json")) 
      .andExpect(content().string(user.toJson())); 

} 

。私はUserService.save（user）のMockを実際のsave（）と同じように、保存されたユーザのidを変更するように設定できますか？ PasswordEncoderと同じですが、PasswordEncoder.encode（string）を呼び出すと、文字列の値がエンコードされた値に変更されます。

Answer 1

を作成するとき、すなわち、単にユーザIDを設定します。したがって、私はそれをお勧めしません。あなたの単体テストがSpringコントローラをテストしていて、UserServiceまたはPasswordEncoderをテストしておらず、SpringコントローラがユーザのIDが変更されたかどうか、またはパスワードがエンコードされているかどうかを確認する責任はありません。 userServiceとpasswordEncoderの両方が呼び出されたことを確認できますが、コントローラーのテストが行​​われる限りです。

ではなく、例えば、テストデータベースに、実際の授業で、エンドツーエンドのテストを実行し、あなたが本当にUserServiceとPasswordEncoderを嘲笑されていない何をしたいのかのように聞こえるが、。新しいユーザーをコントローラーからPOSTしてから、テストデータベースからフェッチし、得られた結果が正しいかどうかを確認することができます。このシナリオでは、Springコントローラをユニットテストするのではなく、実際のチェーン全体をテストします。

when(userService.save(user)).thenAnswer(new Answer<Long>() { 
    @Override 
    public Long answer(InvocationOnMock invocation) { 
     user.setId(1L); 
     return 1L; 
    } 
}); 

メソッドはモックに呼び出されると、このカスタム答えが呼び出されます：あなたはthenAnswer代わりのthenReturnを使用してカスタムの答えを設定することにより、求めているものを行うことができ、言っ

その結果は嘲笑されたメソッドによって返されます。ここでは、返す前にユーザーのIDも設定します。これはまだまだ恐ろしいことですが、適切な統合テストを行うことが非常に望ましいでしょう。

Answer 2

ユニットテストで機能/統合テストではないと仮定すると、UserServiceとUserを嘲笑しているなら、あなたはその動作を気にしないので、あなたはそれを "嘲笑"しています。つまり、以前に設定したUserまたはUserのモックを返すように伝えることができます。

あなたがこのテスト香りで、おそらくあなたはあまりテストされることを意味したくてUserオブジェクト