Насмешка над пустым методом в Spring Framework (Mockito) - PullRequest
0 голосов
/ 20 июня 2019

Я пишу интеграционные тесты для весеннего веб-приложения, и я достиг шага, когда мне нужно смоделировать вызовы сервисных методов, которые имеют тип возврата void. Я провел некоторое исследование некоторых способов сделать это, но ни один из них не кажется правильным.

Что я хочу сделать, это:

  • Когда метод rec () вызывается на recipeService, он должен сохранить рецепт

Ниже я приведу код, а также два основных способа, которые я уже пробовал. Если кто-то может помочь, это было бы здорово!

Метод, который нуждается в издевательстве

@RequestMapping(path = "/recipes/add", method = RequestMethod.POST)
public String persistRecipe(@Valid Recipe recipe, BindingResult result, @RequestParam("image") MultipartFile photo, RedirectAttributes redirectAttributes) {
    if (result.hasErrors()) {
        redirectAttributes.addFlashAttribute("recipe", recipe);
        redirectAttributes.addFlashAttribute("flash",
                new FlashMessage("I think you missed something. Try again!", FlashMessage.Status.FAILURE));
        return "redirect:/recipes/add";
    }

    User user = getUser();
    recipe.setOwner(user);
    user.addFavorite(recipe);
    recipeService.save(recipe, photo);
    userService.save(user);
    redirectAttributes.addFlashAttribute("flash", new FlashMessage("The recipe has successfully been created", FlashMessage.Status.SUCCESS));

    return "redirect:/recipes";
}

Служба, требующая вызова (метод сохранения)

@Service
public class RecipeServiceImpl implements RecipeService {

private final RecipeRepository recipes;

@Autowired
public RecipeServiceImpl(RecipeRepository recipes) {
    this.recipes = recipes;
}

@Override
public void save(Recipe recipe, byte[] photo) {
    recipe.setPhoto(photo);
    recipes.save(recipe);
}

@Override
public void save(Recipe recipe, MultipartFile photo) {
    try {
        recipe.setPhoto(photo.getBytes());
        recipes.save(recipe);
    } catch (IOException ex) {
        ex.printStackTrace();
    }
}

@Override
public Recipe findById(Long id) {
    Optional<Recipe> recipe = recipes.findById(id);
    if (recipe.isPresent()) {
        return recipe.get();
    }

    // TODO:drt - Create new exception to handle this
    throw new RuntimeException();
}

@Override
public Recipe findByName(String name) {
    return null;
}

@Override
public List<Recipe> findAll() {
    return (List<Recipe>) recipes.findAll();
}

@Override
public void deleteById(Long id) {
    recipes.deleteById(id);
}

} * * тысяча двадцать-один

Попытка 1

@Test
@WithMockUser(value = "daniel")
public void createNewRecipeRedirects() throws Exception {
    User user = userBuilder();
    Recipe recipe = recipeBuilder(1L);
    recipe.setOwner(user);
    user.addFavorite(recipe);
    MockMultipartFile photo = new MockMultipartFile("image", "food.jpeg",
                    "image/png", "test image".getBytes());

    when(userService.findByUsername("daniel")).thenReturn(user);

    doAnswer(new Answer<Void>() {

        @Override
        public Void answer(InvocationOnMock invocation) throws Throwable {
            Object[] arguments = invocation.getArguments();
            if (arguments != null && arguments.length > 1 && arguments[0] != null && arguments[1] != null) {

                Recipe recipe1 = (Recipe) arguments[0];
                MultipartFile file = (MultipartFile) arguments[1];
                recipe1.setPhoto(file.getBytes());

            }
            return null;
        }
    }).when(recipeService).save(any(Recipe.class), any(MultipartFile.class));

    mockMvc.perform(post("/recipes/add"))
            .andExpect(redirectedUrl("/recipes"))
            .andExpect(flash().attributeExists("flash"));


}

Попытка 2

@Test
@WithMockUser(value = "daniel")
public void createNewRecipeRedirects() throws Exception {
    List<Recipe> recipes = recipeListBuilder();
    List<User> users = new ArrayList<>();
    User user = userBuilder();
    Recipe recipe = recipeBuilder(1L);
    recipe.setOwner(user);
    user.addFavorite(recipe);
    MockMultipartFile photo = new MockMultipartFile("image", "food.jpeg",
                    "image/png", "test image".getBytes());

    when(userService.findByUsername("daniel")).thenReturn(user);

    doAnswer(answer -> {
        recipe.setPhoto(photo.getBytes());
        recipes.add(recipe);
        return true;
    }).when(recipeService).save(any(Recipe.class), any(MultipartFile.class));

    doAnswer(answer -> {
        users.add(user);
        return true;
    }).when(userService).save(any(User.class));

    mockMvc.perform(post("/recipes/add"))
            .andExpect(redirectedUrl("/recipes"))
            .andExpect(flash().attributeExists("flash"));

    assertEquals(3, recipes.size());
    assertEquals(1, users.size());
}

Полный тестовый код до настоящего времени

@RunWith(SpringRunner.class)
@SpringBootTest(classes = Application.class)
@WebAppConfiguration
public class RecipeControllerTests {

private MockMvc mockMvc;

@Mock
private RecipeService recipeService;

@Mock
private UserService userService;

@Mock
private IngredientService ingredientService;

@Autowired
WebApplicationContext wac;

@InjectMocks
private RecipeController recipeController;

@Before
public void setUp() throws Exception {
    MockitoAnnotations.initMocks(this);
    mockMvc = MockMvcBuilders.webAppContextSetup(wac).apply(springSecurity()).build();
}

/**
 * Tests for index pages / & /recipes
 */
@Test
@WithUserDetails(value = "daniel")
public void indexPageLoads() throws Exception {
    List<Recipe> recipes = recipeListBuilder();
    List<Ingredient> ingredients = ingredientsListBuilder();

    when(recipeService.findAll()).thenReturn(recipes);
    when(ingredientService.findAll()).thenReturn(ingredients);
    when(userService.findByUsername("daniel")).thenReturn(userBuilder());

    mockMvc.perform(get("/recipes"))
            .andExpect(model().attributeExists("recipes", "ingredients", "favs"))
            .andExpect(status().isOk());
}

/**
 * Tests for page /recipes/add
 */
@Test
@WithMockUser
public void addRecipePageLoads() throws Exception {
    mockMvc.perform(get("/recipes/add"))
            .andExpect(model().attributeExists("task", "buttonAction", "action", "photo", "recipe"))
            .andExpect(status().isOk());
}

@Test
@WithUserDetails("daniel")
public void createNewRecipeRedirects() throws Exception {
    User user = userBuilder();
    Recipe recipe = recipeBuilder(1L);
    recipe.setOwner(user);
    user.addFavorite(recipe);
    MultipartFile photo = new MockMultipartFile("image", "food.jpeg",
            "image/jpeg", "dummy content file".getBytes());

    when(userService.findByUsername("daniel")).thenReturn(user);
    verify(recipeService, times(1)).save(recipe, photo);
    verify(userService, times(1)).save(user);


    mockMvc.perform(post("/recipes/add"))
            .andExpect(redirectedUrl("/recipes"))
            .andExpect(flash().attributeExists("flash"));


}


private User userBuilder() {
    User user = new User();
    user.setFavorites(recipeListBuilder());
    user.setId(1L);
    user.setRoles(new String[]{"ROLE_USER", "ROLE_ADMIN"});
    user.setUsername("daniel");
    user.setPassword("password");

    return user;
}

private List<Recipe> recipeListBuilder() {
    List<Recipe> recipes =  new ArrayList<>();
    recipes.add(recipeBuilder(1L));
    recipes.add(recipeBuilder(2L));

    return recipes;
}

private List<Ingredient> ingredientsListBuilder() {
    List<Ingredient> ingredients = new ArrayList<>();
    ingredients.add(ingredientBuilder());

    return ingredients;
}

private Ingredient ingredientBuilder() {
    Ingredient ingredient = new Ingredient();
    ingredient.setCondition("good");
    ingredient.setName("test ing");
    ingredient.setQuantity(1);
    ingredient.setId(1L);

    return ingredient;
}

private Recipe recipeBuilder(Long id) {
    Recipe recipe = new Recipe();
    recipe.setName("Test recipe");
    recipe.setDescription("Test Description");
    recipe.setId(id);
    recipe.setCategory(Category.ALL_CATEGORIES);
    recipe.setCookTime(10);
    recipe.setPrepTime(10);
    recipe.addIngredient(ingredientBuilder());

    return recipe;
}
}

Ответы [ 3 ]

0 голосов
/ 20 июня 2019

Если вы издеваетесь над методом сохранения, я бы использовал один из ‘do ...‘ Docs Конечно, это говорит о том, что у вашего метода есть побочный эффект.

Если вы хотите, чтобы метод вызывался, вы можете использовать ‘verify‘, как указано в других ответах.

Вообще говоря, насмешка позволяет вам заменить некоторую совместную работу / функциональность версией, которая находится под контролем тестов, где проверка позволяет проверить что-то произошло (или не произошло)

0 голосов
/ 21 июня 2019

try Mockito.doNothing(): по сути, Mockito ничего не делает, когда вызывается метод в фиктивном объекте:

Mockito.doNothing().when(recipeService).save(any(Recipe.class), any(MultipartFile.class));
0 голосов
/ 20 июня 2019

Если у вас есть какая-то логика, которую вы хотите выполнить модульное тестирование, и эта логика вызывает методы другого компонента, который вы хотите смоделировать, и некоторые из этих методов возвращают void - типичный способ проверки вашей логики - проверить, что ваша логика на самом деле вызывал void методы издевательства над объектом. Вы можете добиться этого, используя Mockito::verify:

 Mockito.verify(recipeService, Mockito.times(1)).save(any(Recipe.class), any(MultipartFile.class));

Таким образом, вы проверяете, что логика метода persistRecipe() фактически вызвала нужный метод для вашего фиктивного объекта.

...