Sinon: withArgsはsinon.mockでは正しく機能しません

例示的な問題レポートをありがとう。 私はモック機能を使用していません。脳が痛くなるので、他の誰かがこれを見る必要がありますが、あなたの例はそれをはるかに簡単にするはずです。

自分でコードを自由に見てください-多くの場合、これを修正する最も簡単な方法です:-)

コードを見てみました。
まず第一に、このテストは機能します：

class C {
  static foo(arg1, arg2) {return 'foo';}
}

it('test withArgs mock', () => {
  let CMock = sinon.mock(C);

  CMock.expects('foo').withArgs('a', 'b').returns(1);
  CMock.expects('foo').withArgs('c', 'd').returns(2);

  expect(C.foo('a', 'b')).toEqual(1);
  expect(C.foo('c', 'd')).toEqual(2);

  CMock.restore();
});

これが機能する理由と元の問題のテストが機能しない理由は2つあります。

• 2つのexpectsを作成します。
• expectsによって返されるメソッドの代わりに、静的メソッドを呼び出します。

つまり、これも機能しません。

it('test withArgs mock', () => {
  let CMock = sinon.mock(C);

  CMock.expects('foo').withArgs('a', 'b').returns(1);
  let fooStub = CMock.expects('foo').withArgs('c', 'd').returns(2);

  expect(fooStub('a', 'b')).toEqual(1); // Error: foo received wrong arguments ["a", "b"], expected ["c", "d"]
  expect(fooStub('c', 'd')).toEqual(2);

  CMock.restore();
});

なぜそれが起こるのですか？

expectsが呼び出されるたびに、次のことが起こります。

• 新しい期待値（たとえば、 minCalls/maxCalls拡張されたスタブ- once()を呼び出すと変更される変数）が作成されますが、それは「スコープ付きスタブ」であり、そうではありません。他のexpects接続している。 expectsから返されたオブジェクトです。
• 期待のリストに追加されます。

これにより、これらのシナリオで異なる動作が発生します。

• expectsによって返されるメソッドを呼び出すと、そのexpectsのみに制限されます。
• スタブではなくメソッドを直接呼び出すと、すべてのexpectsが検索され、適切なものが見つかります。 複数ある場合は、最初に呼び出されます。

これは正しい動作ですか？ 誰かがメソッドでexpectsを呼び出すと、それはstubまったく同じように動作する（追加のminCalls/maxCalls ）というのがより直感的に思えます。 これにより、次のようになります。

• stubとmock-expectation間でより多くのコードを共有できます。
• 上記のシナリオの動作は同じになります。

さて、 minCalls/maxCallsどうなりますか？

現在expectationは1つしかないため、モックを使用して呼び出し数の検証を行う3つの代替方法を提案します。

• once() 、 twice()などの連結を許可します。

  let fooStub = CMock.expects('foo').withArgs('a', 'b').returns(1).twice();
  fooStub.withArgs('c', 'd').returns(2).once(); // Does not affect the previous expectation.

ただし、少し混乱するかもしれません。

• once() 、 twice()などを連結しないでください。

  CMock.expects('foo').withArgs('a', 'b').returns(1).twice(); // `twice()` returns `null`, so that one has to call `expects()` again to set expectations for other arguments. 
  CMock.expects('foo').withArgs('c', 'd').returns(2).once();

より明確ですが、 foo複数回スタブすることはできないため、それを回避する方法が必要になります。 また、これはまだ可能です：

  CMock.expects('foo').withArgs('a', 'b').returns(1).withArgs('c', 'd').returns(2).twice(); // I guess that this should only affect the last `withArgs`?

• 呼び出し回数を記憶するSpyAPIを使用します。

この問題は、最近のアクティビティがないため、自動的に古いものとしてマークされています。 それ以上のアクティビティが発生しない場合は閉じられます。 貢献していただきありがとうございます。