2017-09-22 16 views
-1

私はコードをリファクタリングしようとしています。理解のために、コードはactionXを実行することです。継承をテストする方法、または私の設計が間違っていますか?

古いコードは、PENDING状態にある注文のためだけに機能します。この状態でactiveOrdersに対してactionXが実行されます。 actionXを実行する前に、fetchA,fetchBおよびfetchCのような小さなステップがあります。このコードには、追加情報があるmetaDataオブジェクトがあります。このmetaDataを使用して、すべてfetch*が実行されます。 fetchAfetchBは互いに依存しますが、fetchCは他とは独立しています。

今、新しいステータスが導入されました。これはSHIPPEDと呼ばれています。この場合、actionYを実行する必要があります。このステータスとPENDINGの唯一の共通部分はfetchCです。

私にとってこれはFactoryPatternを適用するのに最適な候補者でした。だから、基本クラス(fetchCが共通なのでインターフェイスではありません)を作成し、そこから2人の子供を作成しました。 ActionYHandlerおよびActionXHandler。工場にはorderTypeが入り、ActionYHandlerまたはActionXHandlerが返されます。

各ハンドラには、特定のアクションを処理するためのメソッドhandle (MetaData metaData)があります。しかし、fetchCは、スーパークラスの一部として実装されました。つまり、ActionHandlerです。コードの冗長性が少なくなるようにします。各ハンドラは、fetchCによって返されたデータを利用する必要があります。したがって、親レベルでfetchCを実装すると、親メソッドを呼び出すことができます。

Mockitoを使用してユニットテストケースを作成するまで、すべてが完璧に機能します。問題が発生したときに私はfetchCactionHandlerXMockオブジェクトを使って模擬しようとしました。私が得たエラーは、この記事の最後に記載されています。

私はグーグルで始まり、stumbled upon this postというエラーが発生し、私のデザインが間違っていると思うようになった。ここに質問があります。

  1. fetchCは親クラスの一部である必要があります。はいの場合、どのようにテストすることができます
  2. 私が考えることができる他のオプションはfactoryfetchCメソッドを実装し、から取得したデータをhandle (MetaData metaData, FetchCResult fetchCResult)に渡します。テストの観点からは、このデータを簡単に嘲笑することができます。しかし、それは工場の目的に反するものです。今、工場はここでいくつかのビジネスロジックを認識している必要があります。

ERRORトレース

org.mockito.exceptions.misusing.MissingMethodInvocationException: 
when() requires an argument which has to be 'a method call on a mock'. 
For example: 
    when(mock.getArticles()).thenReturn(articles); 

Also, this error might show up because: 
1. you stub either of: final/private/equals()/hashCode() methods. 
    Those methods *cannot* be stubbed/verified. 
    Mocking methods declared on non-public parent classes is not supported. 
2. inside when() you don't call method on mock but on some other object. 
at <my_class_name> 
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method) 
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62) 
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) 
at java.lang.reflect.Method.invoke(Method.java:498) 
at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) 
at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) 
at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) 
at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17) 
at org.junit.runners.ParentRunner.runLeaf(ParentRunner.java:325) 
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:78) 
at org.junit.runners.BlockJUnit4ClassRunner.runChild(BlockJUnit4ClassRunner.java:57) 
at org.junit.runners.ParentRunner$3.run(ParentRunner.java:290) 
at org.junit.runners.ParentRunner$1.schedule(ParentRunner.java:71) 
at org.junit.runners.ParentRunner.runChildren(ParentRunner.java:288) 
at org.junit.runners.ParentRunner.access$000(ParentRunner.java:58) 
at org.junit.runners.ParentRunner$2.evaluate(ParentRunner.java:268) 
at org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:26) 
at org.junit.runners.ParentRunner.run(ParentRunner.java:363) 
at org.mockito.internal.runners.JUnit45AndHigherRunnerImpl.run(Unknown Source) 
at org.mockito.runners.MockitoJUnitRunner.run(Unknown Source) 
at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.run(JUnit4TestReference.java:50) 
at org.eclipse.jdt.internal.junit.runner.TestExecution.run(TestExecution.java:38) 
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:459) 
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:675) 
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:382) 
at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:192) 

答えて

1

ユニットテストは、行動を確認します。

継承は、あなたが(明示的)をテストしていないwhch 実装の詳細です。

関連する問題