Простая история парадокса

public class User {
public static User NULL_USER = new User(-1, "Not Available", "", new Date());
private final long id;
private final String name;
private final String email;
private final Date createDate;
public User(long id, String name, String email, Date createDate) {this.id = id;this.name = name;this.email = email;this.createDate = createDate;}
public long getId() {return id;}
public String getName() {return name;}
public String getEmail() {return email;}
public Date getCreateDate() {return createDate;}
}

@Controller
public class UserController {
@Autowired
private UserService userService;
@RequestMapping("/find1")
public String findUser(@RequestParam("user") String name, Model model) {
User user = userService.findUser(name);
  model.addAttribute("user", user);return "user";}
}

public interface UserService {
public abstract User findUser(String name);}

@Service
public class UserServiceImpl implements UserService {
@Autowired
private UserDao userDao;
/**
  * @see com.captaindebug.cargocult.ntier.UserService#findUser(java.lang.String)
  */@Override
public User findUser(String name) {return userDao.findUser(name);}
}

public interface UserDao {
public abstract User findUser(String name);}

@Repository
public class UserDaoImpl implements UserDao {
private static final String FIND_USER_BY_NAME = "SELECT id, name,email,createdDate FROM Users WHERE name=?";
@Autowire
d
private JdbcTemplate jdbcTemplate;
/**
  * @see com.captaindebug.cargocult.ntier.UserDao#findUser(java.lang.String)
  */
@Override
public User findUser(String name) {
User user;try {FindUserMapper rowMapper = new FindUserMapper();
  user = jdbcTemplate.queryForObject(FIND_USER_BY_NAME, rowMapper, name);} catch (EmptyResultDataAccessException e) {user = User.NULL_USER;}return user;}
}

Если вы посмотрите на этот код, как это ни парадоксально, он выглядит хорошо; на самом деле это похоже на классический пример учебника о том, как написать N-уровневое приложение MVC. Контроллер передает ответственность за сортировку бизнес-правил на сервисный уровень, а сервисный уровень получает данные из БД с использованием объекта доступа к данным, который, в свою очередь, использует вспомогательный класс RowMapper <> для извлечения объекта User. Когда у контроллера есть объект User, он внедряет его в модель, готовую для отображения. Эта модель ясна и расширяема; мы изолируем базу данных от сервиса и сервис от контроллера с помощью интерфейсов и тестируем все, используя как JUnit с Mockito, так и интеграционные тесты. Это должно быть последнее слово в кодировке MVC учебника, или это? Давайте посмотрим на код. 

Во-первых, есть ненужное использование интерфейсов. Некоторые утверждают, что легко переключать реализации баз данных, но кто когда-либо это делает?
¹  плюс, современные инструменты макетирования могут создавать свои прокси, используя определения классов, поэтому, если ваш дизайн не требует нескольких реализаций одного и того же интерфейса, использование интерфейсов не имеет смысла. 

Далее, есть UserServiceImpl, который является классическим примером 
ленивого анти-паттерна класса , потому что он ничего не делает, кроме бессмысленного делегирования объекту доступа к данным. Аналогично, контроллер также довольно ленив, поскольку делегирует его ленивому UserServiceImpl, прежде чем добавлять полученный класс User в модель: фактически все эти классы являются примерами анти-паттерна lazy class. 

Написав некоторые ленивые классы, они теперь без необходимости тестируются до смерти, даже класс UserServiceImpl без событий. Стоит писать тесты только для классов, которые действительно выполняют некоторую логику. 

public class UserControllerTest {
private static final String NAME = "Woody Allen";
private UserController instance;
@Mock
private Model model;
@Mock
private UserService userService;
@Before
public void setUp() throws Exception {
MockitoAnnotations.initMocks(this);

  instance = new UserController();
  ReflectionTestUtils.setField(instance, "userService", userService);}
@Test
public void testFindUser_valid_user() {
User expected = new User(0L, NAME, "[email protected]", new Date());
  when(userService.findUser(NAME)).thenReturn(expected);

  String result = instance.findUser(NAME, model);
  assertEquals("user", result);

  verify(model).addAttribute("user", expected);}
@Test
public void testFindUser_null_user() {
when(userService.findUser(null)).thenReturn(User.NULL_USER);

  String result = instance.findUser(null, model);
  assertEquals("user", result);

  verify(model).addAttribute("user", User.NULL_USER);}
@Test
public void testFindUser_empty_user() {
when(userService.findUser("")).thenReturn(User.NULL_USER);

  String result = instance.findUser("", model);
  assertEquals("user", result);

  verify(model).addAttribute("user", User.NULL_USER);}

}

public class UserServiceTest {
private static final String NAME = "Annie Hall";
private UserService instance;
@Mock
private UserDao userDao;
@Before
public void setUp() throws Exception {
MockitoAnnotations.initMocks(this);

  instance = new UserServiceImpl();

  ReflectionTestUtils.setField(instance, "userDao", userDao);}
@Test
public void testFindUser_valid_user() {
User expected = new User(0L, NAME, "[email protected]", new Date());
  when(userDao.findUser(NAME)).thenReturn(expected);

  User result = instance.findUser(NAME);
  assertEquals(expected, result);}
@Test
public void testFindUser_null_user() {
when(userDao.findUser(null)).thenReturn(User.NULL_USER);

  User result = instance.findUser(null);
  assertEquals(User.NULL_USER, result);}
@Test
public void testFindUser_empty_user() {
when(userDao.findUser("")).thenReturn(User.NULL_USER);

  User result = instance.findUser("");
  assertEquals(User.NULL_USER, result);}
}

public class UserDaoTest {
private static final String NAME = "Woody Allen";
private UserDao instance;
@Mock
private JdbcTemplate jdbcTemplate;
@Before
public void setUp() throws Exception {
MockitoAnnotations.initMocks(this);

  instance = new UserDaoImpl();
  ReflectionTestUtils.setField(instance, "jdbcTemplate", jdbcTemplate);}
@SuppressWarnings({ "unchecked", "rawtypes" })
@Test
public void testFindUser_valid_user() {
User expected = new User(0L, NAME, "[email protected]", new Date());
  when(jdbcTemplate.queryForObject(anyString(), (RowMapper) anyObject(), eq(NAME))).thenReturn(expected);

  User result = instance.findUser(NAME);
  assertEquals(expected, result);}
@SuppressWarnings({ "unchecked", "rawtypes" })
@Test
public void testFindUser_null_user() {
when(jdbcTemplate.queryForObject(anyString(), (RowMapper) anyObject(), isNull())).thenReturn(User.NULL_USER);

  User result = instance.findUser(null);
  assertEquals(User.NULL_USER, result);}
@SuppressWarnings({ "unchecked", "rawtypes" })
@Test
public void testFindUser_empty_user() {
when(jdbcTemplate.queryForObject(anyString(), (RowMapper) anyObject(), eq(""))).thenReturn(User.NULL_USER);

  User result = instance.findUser("");
  assertEquals(User.NULL_USER, result);}

}

@RunWith(SpringJUnit4ClassRunner.class)@WebAppConfiguration
@ContextConfiguration({ "file:src/main/webapp/WEB-INF/spring/appServlet/servlet-context.xml","file:src/test/resources/test-datasource.xml" })public class UserControllerIntTest {
@Autowired
private WebApplicationContext wac;
private MockMvc mockMvc;
/**
  * @throws java.lang.Exception
  */
@Before
public void setUp() throws Exception {
mockMvc = MockMvcBuilders.webAppContextSetup(wac).build();}
@Test
public void testFindUser_happy_flow() throws Exception {
ResultActions resultActions = mockMvc.perform(get("/find1").accept(MediaType.ALL).param("user","Tom"));
  resultActions.andExpect(status().isOk());
  resultActions.andExpect(view().name("user"));
  resultActions.andExpect(model().attributeExists("user"));
  resultActions.andDo(print());

  MvcResult result = resultActions.andReturn();
  ModelAndView modelAndView = result.getModelAndView();
  Map<String, Object> model = modelAndView.getModel();

  User user = (User) model.get("user");
  assertEquals("Tom", user.getName());
  assertEquals("[email protected]", user.getEmail());}

}

@RunWith(SpringJUnit4ClassRunner.class)
@WebAppConfiguration
@ContextConfiguration({ "file:src/main/webapp/WEB-INF/spring/appServlet/servlet-context.xml",
    "file:src/test/resources/test-datasource.xml" })
public class UserControllerIntTest {

  @Autowired
  private WebApplicationContext wac;

  private MockMvc mockMvc;

  /**
   * @throws java.lang.Exception
   */
  @Before
  public void setUp() throws Exception {

    mockMvc = MockMvcBuilders.webAppContextSetup(wac).build();
  }

  @Test
  public void testFindUser_happy_flow() throws Exception {

    ResultActions resultActions = mockMvc.perform(get("/find1").accept(MediaType.ALL).param("user", "Tom"));
    resultActions.andExpect(status().isOk());
    resultActions.andExpect(view().name("user"));
    resultActions.andExpect(model().attributeExists("user"));
    resultActions.andDo(print());

    MvcResult result = resultActions.andReturn();
    ModelAndView modelAndView = result.getModelAndView();
    Map<String, Object> model = modelAndView.getModel();

    User user = (User) model.get("user");
    assertEquals("Tom", user.getName());
    assertEquals("[email protected]", user.getEmail());
  }

}