java - 关于 JUnit 测试的字段的访问权限

Question

假设我有一个类以链表的形式实现了一个栈:

public class Stack {
    private StackNode base;

    public void push(Object item) {
        if (base == null) {base = new StackNode(item);}
        // remaining implementation not shown
    }
}

和一个 JUnit 测试类:

public class TestStack {
    @Test
    public void testPush() {
        Stack st = new Stack();
        st.push(new Integer(3)); 

        assertEquals((Integer)st.base.getContents(), new Integer(3)); //Error!
        // other test cases not shown
    }
}

我的问题是:解决测试类对 base 的访问权限问题的最佳实践是什么？

我知道这个question其中讨论了制作访问器，我知道我也可以将我的 Stack 和 TestStack 代码放入一个包中，并制作 base 字段包 -私有(private)的。

我想知道最好的行动方案是什么。我知道我不应该在 push 的单元测试中使用 pop，反之亦然，因为这会耦合那些单元测试。

我认为，如果我不希望客户端代码访问该字段，那么创建(公共(public))访问器是一种不好的做法，为该字段(或该字段本身)包创建访问器也是正确的做法-私有(private)？

另一个信息question讨论了私有(private)方法的选项，但这是关于公共(public)方法的，为了测试它我需要访问一个私有(private)字段，所以“如果你需要访问私有(private)方法/类，你做错了”的许多陈述并不' 似乎适用于此。

Answer 1

我们应该编写使用公共(public) API 的测试。但这并不总是可能的。在这种情况下，我会编写一个测试，同时使用两种方法，push 和 pop，因为这两种方法构成了一个功能堆栈:

public class Stack {
    private StackNode base;

    public void push(Object item) {
        ...
    }

    // I know this wasn't part of your code
    public Object pop() {
        ...
    }
}

public class TestStack {
    @Test
    public void testOnePush() {
        // GIVEN
        Stack st = new Stack();

        // WHEN
        st.push(new Integer(3)); 
        Object popped = st.pop();

        assertEquals(popped, new Integer(3));
    }
}

如果无法进行这种测试，则可以直接或通过 getter 公开 base。在这种情况下，我更愿意写一个警告注释，即它仅用于测试:

public class Stack {
    StackNode base; // package-private for testing reasons
}

也可以用方法名称警告其他开发人员:

public class Stack {
    private StackNode base;

    /** For unit tests only!!! */
    public StackNode getBaseForTests() {
         return base;
    }
}