Fix regression for issue sinonjs#1526 regarding onFirstCall().throws()

noisecapella · Aug 14, 2017 · 7b53434 · 7b53434
1 parent 350717d
commit 7b53434
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 0 deletions.
diff --git a/lib/sinon/behavior.js b/lib/sinon/behavior.js
@@ -115,6 +115,7 @@ var proto = {
     isPresent: function isPresent() {
         return (typeof this.callArgAt === "number" ||
                 this.exception ||
+                this.exceptionCreator ||
                 typeof this.returnArgAt === "number" ||
                 this.returnThis ||
                 this.resolveThis ||

diff --git a/test/stub-test.js b/test/stub-test.js
@@ -137,6 +137,19 @@ describe("stub", function () {
                 stub();
             });
         });
+
+        it("throws only on the first call", function () {
+            var stub = createStub.create();
+            stub.returns("no exception");
+            stub.onFirstCall().throws();
+
+            assert.exception(function () {
+                stub();
+            });
+
+            // on the second call there is no exception
+            assert.same(stub(), "no exception");
+        });
     });
 
     describe(".resolves", function () {