fix(2525): ensure non empty message when error of type string is passed, but no message (#2544)

Co-authored-by: Morgan Roderick <[email protected]>

Author: peanutenthusiast

lib/sinon/default-behaviors.js

@@ -15,7 +15,9 @@ function throwsException(fake, error, message) {
         fake.exceptionCreator = error;
     } else if (typeof error === "string") {
         fake.exceptionCreator = function () {
-            const newException = new Error(message || "");
+            const newException = new Error(
+                message || `Sinon-provided ${error}`
+            );
             newException.name = error;
             return newException;
         };

test/proxy-call-test.js

@@ -1134,7 +1134,7 @@ describe("sinonSpy.call", function () {
 
             assert.equals(
                 object.doIt.getCall(0).toString().replace(/ at.*/g, ""),
-                "doIt() !TypeError"
+                "doIt() !TypeError(Sinon-provided TypeError)"
             );
         });
 

test/stub-test.js

@@ -940,6 +940,21 @@ describe("stub", function () {
             assert.contains(stub.firstCall.toString(), "not implemented");
         });
 
+        it("creates a non empty error message when error is a string and no message is passed", function () {
+            const stub = createStub();
+
+            stub.withArgs(1).throws("TypeError");
+
+            assert.exception(
+                function () {
+                    stub(1);
+                },
+                {
+                    message: "Sinon-provided TypeError",
+                }
+            );
+        });
+
         describe("lazy instantiation of exceptions", function () {
             let errorSpy;
             beforeEach(function () {