メソッド内のメソッドをテストする最善の方法は何ですか？ （JUnit）

Question

私はJavaクラスAを持っています。Aのコンストラクタは、いくつかのメソッドm1、m2を呼び出します。

class A{ 

    public A(){ 
     m1(); 
     m2(); 
    ...... } 

    public void m1(){...}; 
    public void m2(){...}; 
} 

これらのメソッドのテストを書くにはどうすればよいですか？私は一般的にそれらを書いています

class TestClass{ 

static A a = null; 

public TestClass(){ 
    a = new A(); 
} 

@Test 
public testm1(){ 
    A.m1(); 
    //Some logic 
} 

しかし、これらの関数はコンストラクタ自体で呼び出されます。メソッドを2回実行してチェックしているように見えますが、メソッドが正しく記述されていないと、コンストラクター自体が失敗します。それをテストする正しい方法は何ですか？

Answer 1

まず、あなたのインターフェイスは奇妙に見えます。

他のパブリックメソッドを呼び出す1つのパブリックメソッドを持つことにはあまり意味がありません。だから、実際のの問題は、あなたが記述したものとまったく同じです。つまり、コンストラクタがそのm1とm2メソッドを呼び出すということです。

コンストラクタはこれらのメソッドを呼び出すか、理想的にはプライベートである必要があります。またはそれらは公開されていますが、それを呼び出すのはコンストラクタの責任ではありません。

その意味では、答えは一歩前進してデザインのこの側面を注意深く見ていることです。可能であれば、それを変更してください！

いずれにしても：あなたの現在のデザインを維持するとき。他の方法はありませんが、そのようなものを複数回テストすることです。。あなたは、あなたのコンストラクタが（m1（）、m2（）、m3（）のどれかを呼び出しているかどうかに関わらず、あなたのコンストラクタが動作しているかどうかをテストする必要があります。クラスの既存のインスタンス上のすべてのパブリックメソッドを呼び出すことも確実に行う必要があります。

最後に：「存在する」ためにメソッドを呼び出さないでください。各メソッドは明確な目的（別名契約）を持っている必要があります。それらの契約が確実に守られるようにテストケースを作成します。