JMockit的连续期望

Question

我最近遇到了下面说明的问题，但找不到相应的解释。 我正在测试的代码看起来还可以，但是测试失败。 我花了一段时间才了解出了什么问题，尤其是因为我的测试代码并不像下面的代码那么简单。

我正在与大家共享它，希望找到一些解释，并得出结论，它是JMockit错误还是所需的行为，因此在实施我们的测试时需要注意一些事项。

代码中的注释将显示问题所在。

预先感谢您的输入。

package my.tst.pkg;

import mockit.Expectations;
import mockit.Mocked;
import org.testng.annotations.Test;

import java.util.Collection;

import static java.util.Arrays.asList;
import static org.testng.AssertJUnit.assertFalse;

public class ConsecutiveExpectationsTest {
    class KeyHolder{
        private String key;
        String getKey() {
            return key;
        }
    }
    class ClassUnderTest {
        private Collection<KeyHolder> keyHolders;

        ClassUnderTest(Collection<KeyHolder> keyHolders) {
            this.keyHolders = keyHolders;
        }
        boolean isValid() {
            // At least one holder with no key means invalid
            return !keyHolders.stream().filter(kh -> kh.getKey() == null).findFirst().isPresent();
        }
    }

    @Mocked
    KeyHolder keyHolder;

    @Test
    public void shouldBeInvalidIfNullKey() throws Exception {
        new Expectations() {{
            // This expectations fail the test
            keyHolder.getKey(); returns("KEY", null);
            // However if casting the null to a String the test will pass
            // Is this something we should always do?
            // keyHolder.getKey(); returns("KEY", (String) null);
        }};
        assertFalse(new ClassUnderTest(asList(keyHolder, keyHolder)).isValid());
    }
}

Answer 1

这是Java语言使用不当的情况，（恰当的）Java IDE应该向用户报告该语言。

IntelliJ，其中一个，报告了“将'null'参数混淆为var-arg方法”警告。

这是令人困惑的/模棱两可的，因为returns(...)方法的第二个参数是一个Object... remainingValues varargs参数（在字节码中只是一个数组），并且由于Java编译器建立数组参数的方式打个电话。 如果仅传递null ，则编译器会将其解释为(Object[]) null ， 而不是您期望的那样解释为new Object[] {null} 。 将其强制转换为String消除了歧义，从而迫使编译器生成一个包含一个null元素的数组。

就是说，JMockit可以将null Object[]参数解释为等效于一个元素的数组。 此更改可能会在将来的版本中出现。 目前，应该使用强制转换为特定元素类型（对于使用IntelliJ的用户，无论如何都要避免代码检查警告）。