私はコードをリファクタリングしようとしています。理解のために、コードはactionX
を実行することです。継承をテストする方法、または私の設計が間違っていますか?
古いコードは、PENDING
状態にある注文のためだけに機能します。この状態でactiveOrders
に対してactionX
が実行されます。 actionX
を実行する前に、fetchA
,fetchB
およびfetchC
のような小さなステップがあります。このコードには、追加情報があるmetaData
オブジェクトがあります。このmetaData
を使用して、すべてfetch*
が実行されます。 fetchA
とfetchB
は互いに依存しますが、fetchC
は他とは独立しています。
今、新しいステータスが導入されました。これはSHIPPED
と呼ばれています。この場合、actionY
を実行する必要があります。このステータスとPENDING
の唯一の共通部分はfetchC
です。
私にとってこれはFactoryPattern
を適用するのに最適な候補者でした。だから、基本クラス(fetchC
が共通なのでインターフェイスではありません)を作成し、そこから2人の子供を作成しました。 ActionYHandler
およびActionXHandler
。工場にはorderType
が入り、ActionYHandler
またはActionXHandler
が返されます。
各ハンドラには、特定のアクションを処理するためのメソッドhandle (MetaData metaData)
があります。しかし、fetchC
は、スーパークラスの一部として実装されました。つまり、ActionHandler
です。コードの冗長性が少なくなるようにします。各ハンドラは、fetchC
によって返されたデータを利用する必要があります。したがって、親レベルでfetchC
を実装すると、親メソッドを呼び出すことができます。
Mockito
を使用してユニットテストケースを作成するまで、すべてが完璧に機能します。問題が発生したときに私はfetchC
actionHandlerXMock
オブジェクトを使って模擬しようとしました。私が得たエラーは、この記事の最後に記載されています。
私はグーグルで始まり、stumbled upon this postというエラーが発生し、私のデザインが間違っていると思うようになった。ここに質問があります。
fetchC
は親クラスの一部である必要があります。はいの場合、どのようにテストすることができます- 私が考えることができる他のオプションは
factory
にfetchC
メソッドを実装し、から取得したデータを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)