@@ -109,7 +109,7 @@ Screw.Stub = (function() { } if (private.oldVal.constructor === Function) { - return private.oldVal(); + return private.oldVal.apply(obj, arguments); } else { return private.oldVal; } lib/screw.stub.js @@ -102,6 +102,17 @@ Screw.Unit(function() { Screw.Stub.reset(); expect(obj.pizza()).to(equal, expected); }); + + it("should call original method with arguments if stub is not matched", function() { + var originalMethodCalled = false; + obj = { methodToStub: function(a) { + originalMethodCalled = true; + expect(a).to(equal, 'foo'); } }; + Screw.Stub.shouldReceive(obj, 'methodToStub').withArguments('bar'); + obj.methodToStub('foo'); + obj.methodToStub('bar'); + expect(originalMethodCalled).to(be_true); + }); }); }); }); spec/stub_spec.js 65473aec8e67658889a3a9548b376078d924a989 Mat Schaffer mat.schaffer@gmail.com http://github.com/tobowers/screw-unit/commit/ad2ff70b8b3f24e62d05404c912bf5ad2d00fbd9 ad2ff70b8b3f24e62d05404c912bf5ad2d00fbd9 2009-06-30T15:49:29-07:00 2009-06-30T15:49:29-07:00 shouldReceive is set up to call the original function if the withArguments doesn't match. But it calls it without arguments. This patch fixes it so it includes arguments in the call. 91fc44129bc237e4259cec90bb84a4df9fed30ee Mat Schaffer mat.schaffer@gmail.com