void方法的单元测试应该是什么样的？

Question

我写了一个程序来模拟商店里的篮子。用户可以添加和删除篮子中的物品。现在，我想知道，我（void）方法的单元测试应该是什么样子。我正在使用JUnit4。

我试图更改我的addItem方法以返回布尔类型，而不是什么都不返回。有了它，我就可以编写简单的Assert了。assertTrue（篮子。添加…） 。但我不知道，void方法的单元测试应该如何编写。

public boolean addItem(Item item) {
    if (orderedItems.containsKey(item)) {
        Integer currItemCount = orderedItems.get(item);
        orderedItems.replace(item, currItemCount, currItemCount + 1);
        return true;
    } else {
        orderedItems.put(item, 1);
        return true;
    }
}

public void addItem(Item item) {
    if (orderedItems.containsKey(item)) {
        Integer currItemCount = orderedItems.get(item);
        orderedItems.replace(item, currItemCount, currItemCount + 1);
    } else {
        orderedItems.put(item, 1);
    }
}




Answer 1

请注意，布尔值在第一个版本中是无效的，因为它总是返回true。此外，添加布尔值实际上不会帮助您对代码进行单元测试，因为这太肤浅了。所以你应该坚持你最初的想法（没有布尔值）。

关于您的问题：是的，addItem（）方法不会返回任何内容，但是如果您可以在篮子中添加项目，您很可能也想阅读它们
测试add（）最简单的方法是依赖get（）/find（）方法<反之亦然。

你可以写这样的东西来测试篮子中没有的项目：

@Test
public void addItem_given_item_not_present_then_getAll_retrieves_it() {
   basket.addItem(new Item("foo"));
   Map<Item, Integer> items = basket.getAll();
   // assert items contains 1 element with 1 occurrence and with the foo name
}

当然，您应该编写其他测试场景。