java - 这是对该方法进行单元测试的好方法吗？

Question

我需要使用 Junit/Mockito 测试此方法。 实际上，我无法使用 Mockito 正确编写测试，因此我使用自己的输入而不是模拟扫描仪。我不知道我是否做对了。 (有人知道如何用mockito制作它吗？)

这是我要测试的方法:

public RentingACar rentACar(Scanner input) {
    RentingACar rentingACar = new RentingACar();

    System.out.print("Brand: ");
    rentingACar.setBrand(input.next());
    System.out.print("Name: ");
    rentingACar.setName(input.next());
    System.out.print("Surname: ");
    rentingACar.setSurname(input.next());
    System.out.print("Rent Date: ");
    rentingACar.setRentDate(input.next());
    System.out.print("Client number: ");
    rentingACar.setClientNumber(input.nextInt());

    return rentingACar;
}

这是我的测试:

@Test
void rentACar() {
    Scanner scanner = new Scanner("Mazda\nPeter\nParker\n20.02\n1234");
    ClientDataGetter clientDataGetter = new ClientDataGetter();

    RentingACar rentingACar = clientDataGetter.rentACar(scanner);

    assertNotNull(rentingACar);

    assertEquals("Mazda", rentingACar.getBrand());
    assertEquals("Peter", rentingACar.getName());
    assertEquals("Parker", rentingACar.getSurname());
    assertEquals("20.02", rentingACar.getRentDate());
    assertEquals(1234, rentingACar.getClientNumber());
}

这有什么意义吗？

Answer 1

当你不需要Mockito时为什么要使用它，天哪，使用Mockito编写它比不使用它会遇到更多问题？ Mockito 完全是最后的选择(例如模拟数据库或外部服务)，即使如此，您是否应该使用它还是值得怀疑的(而不是仅仅编写自己的 FakeDatabase 或 FakeExternalService )。这里绝对不需要，模拟的 Scanner 有何帮助？

我想说你的方法很好，尽管你正在测试的代码还有很多不足之处。由于这个问题是关于测试的，所以我不会谈论代码本身。

您仅测试了所谓的 happy-path，这意味着输出完美且一切正常的情况。

您还需要做的是测试边缘情况(当输入处于最大/最小值时，例如数字中的 L.MIN_VALUE/0/Long.MAX_VALUE、字符串中的 0 长度等)，例如使用空/一个字母名称，客户编号以0开头。

毕竟您需要测试输入是否完全错误:没有 \n 符号，没有日期应有的位置等，这里有很多可能性。您可能会发现您的代码不够，您必须更改它(也许添加一些输入验证)。这就是为什么最好从测试开始(测试驱动开发)，您可以在编写代码之前看到您必须进行验证，并且您将为此做好准备。这是一个简单的情况，但向现有代码添加重要功能并不总是那么容易，如果该代码已被某人使用，甚至可能存在风险。我见过很多情况，有人需要进行这样的修改，但如果不进行大规模重构，这是根本不可能的，所以这就是他们所做的(请永远不要这样做):

if(mySpecialCase) {
  // all the logic
} else {
  // all the logic, almost identical to the logic above
}

编辑:回复您的评论 - 为什么制定 if 条件是不好的做法？

我在这里不仅仅讨论 if 条件。我正在谈论两个 if 分支中潜在的代码重复，它们很可能具有非常相似的代码。想象一下，您需要进行另一项更改，您必须向两个分支添加另一个 if，这会变得一团糟。

If 条件也不应该过度使用，特别是当它涉及到许多不同的分支时。仅通过查看代码很难调试、测试和了解发生了什么。让某人(或您)通过查看和分析代码就知道代码的作用非常重要。解决这个问题通常并不容易，需要使用一些design patterns ，例如 decorator ，有时甚至是简单的 switch可以提供很多帮助。