プライベートメソッドを公開して単体テストを行うのは良い考えでしょうか? 質問する

注:
この回答は元々この質問に対して投稿されたものです単体テストだけが、ゲッターを介してプライベートインスタンス変数を公開する正当な理由になるでしょうか?これはこれに統合されたため、ここで提示されたユースケースに少し特化している可能性があります。

一般的に言えば、私は通常、テストを容易にするために「本番」コードをリファクタリングすることに賛成です。しかし、ここではそれが良い判断ではないと思います。優れた単体テストは (通常) クラスの実装の詳細ではなく、目に見える動作のみを考慮に入れます。内部スタックをテストに公開する代わりに、first()またはを呼び出した後にクラスが期待どおりの順序でページを返すかどうかをテストできますlast()。

たとえば、次の疑似コードを考えてみます。

public class NavigationTest {
    private Navigation nav;

    @Before
    public void setUp() {
        // Set up nav so the order is page1->page2->page3 and
        // we've moved back to page2
        nav = ...;
    }

    @Test
    public void testFirst() {
        nav.first();

        assertEquals("page1", nav.getPage());

        nav.next();
        assertEquals("page2", nav.getPage());

        nav.next();
        assertEquals("page3", nav.getPage());
    }

    @Test
    public void testLast() {
        nav.last();

        assertEquals("page3", nav.getPage());

        nav.previous();
        assertEquals("page2", nav.getPage());

        nav.previous();
        assertEquals("page1", nav.getPage());
    }
}