测试以假阳性通过

Question

我的测试出现误报。 我正在寻找密钥“1234”，我知道它不存在，但测试仍然通过。

我有一个 MAP 方法（reportUIDsAndTitles），它存储一堆 UID 和标题的键和值。 我的测试循环遍历这些键，如果发现它断言标题与键匹配。 但是在下面的代码中，如果没有找到密钥，测试仍然通过。

我该如何解决？ 我已经验证我的 MAP 方法有效并且循环有效，但不确定如何设置失败。

@Test
public void apiTitleOfReport() {
    Map<String, String> pairs;
    pairs = reportUIDsAndTitles();

    for (Map.Entry pairEntry : pairs.entrySet()) {
        if (pairEntry.getKey().equals("1234")) {
            String keyValue = (String) pairEntry.getValue();
            assertEquals(keyValue, "Name of title I am looking for", "Report Title not found.");
        }}}

******* 这是我更新后的代码，如建议的那样，在 IF 块之前缺少 Assert。 现在，如果密钥不存在，则失败。 但如果它确实存在，代码会继续并检查标题是否正确。 谢谢大家的建议。

@Test
public void apiTitleOfReport() {
    Map<String, String> pairs;
    pairs = reportUIDsAndTitles();
    assertTrue(pairs.containsKey("1234"));
    

for (Map.Entry pairEntry : pairs.entrySet()) {
        if (pairEntry.getKey().equals("1234")) {
            String key = (String) pairEntry.getValue();
            assertEquals(key, "Name of title I am looking for", "Report Title not found.");
        }}}

Answer 1

您遍历pairs映射中的每个条目，然后：

1. 如果条目的键是"1234" ，则检查该值（旁注：您正在调用值keyValue ？您需要更好的命名技巧，伙伴：P）是否等于某个特定的东西。 如果是，那太好了。 如果没有，则失败。
2. 就是这样 - 如果条目的键不完全是"1234" ，我们继续循环。 只有键为"1324"任何条目才会导致任何实际代码运行。
3. 就是这样。 没有#3。

因此，正如您所说，没有键 1234 的条目，因此您的代码什么也不做。

如何解决？ 你的问题不清楚意图。 一种方法可能是：

@Test
public void ensure1234IsNotInTheMap() {
    assertFalse(map.containsKey("1234"));
}

也许：

@Test
public void ensure1234IsMappedToFOOBAR() {
    assertEquals("FOOBAR", map.get("1234"));
    // if 1234 isn't in the map at all, the map.get op
    // returns null, and thus your test will properly fail,
    // as null is not equal to "FOOBAR".
}