chore!: remove usingPromise

Everyone should be using native promises by now, or should know how to stub natives

Author: mroderick

docs/release-source/release/examples/stubs-10-use-promise-library.test.js

@@ -77,14 +77,6 @@ If any expectation is not satisfied, an exception is thrown.
 
 Also restores the mocked methods.
 
-#### `mock.usingPromise(promiseLibrary);`
-
-Causes all expectations created from the mock to return promises using a specific
-Promise library instead of the global one when using `expectation.rejects` or
-`expectation.resolves`. Returns the mock object to allow chaining.
-
-_Since `sinon@6.2.0`_
-
 ### Expectations
 
 All the expectation methods return the expectation, meaning you can chain them.

docs/release-source/release/mocks.md

@@ -332,14 +332,6 @@ Fakes XHR and binds a server object to the sandbox such that it too is restored
 
 Access requests through `sandbox.requests` and server through `sandbox.server`
 
-#### `sandbox.usingPromise(promiseLibrary);`
-
-Causes all stubs and mocks created from the sandbox to return promises using a specific
-Promise library instead of the global one when using `stub.rejects` or
-`stub.resolves`. Returns the stub to allow chaining.
-
-_Since `sinon@2.0.0`_
-
 #### `sandbox.restore();`
 
 Restores all fakes created through sandbox.

docs/release-source/release/sandbox.md

@@ -12,7 +12,6 @@ examples:
   - stubs-7-call-fake
   - stubs-8-call-through
   - stubs-9-call-through-with-new
-  - stubs-10-use-promise-library
   - stubs-12-yield-to
   - stubs-14-add-behavior
   - stubs-15-replace-getter
@@ -232,7 +231,6 @@ Causes the stub to return a Promise which resolves to the provided value.
 
 When constructing the Promise, sinon uses the `Promise.resolve` method. You are
 responsible for providing a polyfill in environments which do not provide `Promise`.
-The Promise library can be overwritten using the `usingPromise` method.
 
 _Since `sinon@2.0.0`_
 
@@ -283,7 +281,6 @@ Causes the stub to return a Promise which rejects with an exception (`Error`).
 
 When constructing the Promise, sinon uses the `Promise.reject` method. You are
 responsible for providing a polyfill in environments which do not provide `Promise`.
-The Promise library can be overwritten using the `usingPromise` method.
 
 _Since `sinon@2.0.0`_
 
@@ -332,16 +329,6 @@ Like `callsArg`, but with arguments to pass to the callback.
 
 Like above but with an additional parameter to pass the `this` context.
 
-#### `stub.usingPromise(promiseLibrary);`
-
-Causes the stub to return promises using a specific Promise library instead of
-the global one when using `stub.rejects` or `stub.resolves`. Returns the stub
-to allow chaining.
-
-<div data-example-id="stubs-10-use-promise-library"></div>
-
-_Since `sinon@2.0.0`_
-
 #### `stub.yields([arg1, arg2, ...])`
 
 Similar to `callsArg`.

docs/release-source/release/stubs.md

@@ -90,10 +90,6 @@ const defaultBehaviors = {
         fake.callsThrough = false;
     },
 
-    usingPromise: function usingPromise(fake, promiseLibrary) {
-        fake.promiseLibrary = promiseLibrary;
-    },
-
     yields: function (fake) {
         fake.callArgAt = useLeftMostCallback;
         fake.callbackArguments = slice(arguments, 1);

lib/sinon/default-behaviors.js

@@ -5,7 +5,6 @@ const createProxy = require("./proxy");
 const nextTick = require("./util/core/next-tick");
 
 const slice = arrayProto.slice;
-let promiseLib = Promise;
 
 module.exports = fake;
 
@@ -119,7 +118,7 @@ fake.throws = function throws(value) {
 fake.resolves = function resolves(value) {
     // eslint-disable-next-line jsdoc/require-jsdoc
     function f() {
-        return promiseLib.resolve(value);
+        return Promise.resolve(value);
     }
 
     return wrapFunc(f);
@@ -147,29 +146,12 @@ fake.resolves = function resolves(value) {
 fake.rejects = function rejects(value) {
     // eslint-disable-next-line jsdoc/require-jsdoc
     function f() {
-        return promiseLib.reject(getError(value));
+        return Promise.reject(getError(value));
     }
 
     return wrapFunc(f);
 };
 
-/**
- * Causes `fake` to use a custom Promise implementation, instead of the native
- * Promise implementation.
- *
- * @example
- * const bluebird = require("bluebird");
- * sinon.fake.usingPromise(bluebird);
- *
- * @memberof fake
- * @param {*} promiseLibrary
- * @returns {Function}
- */
-fake.usingPromise = function usingPromise(promiseLibrary) {
-    promiseLib = promiseLibrary;
-    return fake;
-};
-
 /**
  * Returns a `fake` that calls the callback with the defined arguments.
  *

lib/sinon/fake.js

@@ -6,7 +6,6 @@ const proxyCallToString = require("./proxy-call").toString;
 const extend = require("./util/core/extend");
 const deepEqual = require("@sinonjs/samsam").deepEqual;
 const wrapMethod = require("./util/core/wrap-method");
-const usePromiseLibrary = require("./util/core/use-promise-library");
 
 const concat = arrayProto.concat;
 const filter = arrayProto.filter;
@@ -78,7 +77,6 @@ extend(mock, {
         const expectation = mockExpectation.create(method);
         expectation.wrappedMethod = this.object[method].wrappedMethod;
         push(this.expectations[method], expectation);
-        usePromiseLibrary(this.promiseLibrary, expectation);
 
         return expectation;
     },
@@ -119,12 +117,6 @@ extend(mock, {
         return true;
     },
 
-    usingPromise: function usingPromise(promiseLibrary) {
-        this.promiseLibrary = promiseLibrary;
-
-        return this;
-    },
-
     invokeMethod: function invokeMethod(method, thisValue, args) {
         /* if we cannot find any matching files we will explicitly call mockExpection#fail with error messages */
         /* eslint consistent-return: "off" */

lib/sinon/mock.js

@@ -16,7 +16,6 @@ const sinonFake = require("./fake");
 const valueToString = require("@sinonjs/commons").valueToString;
 const fakeServer = require("nise").fakeServer;
 const fakeXhr = require("nise").fakeXhr;
-const usePromiseLibrary = require("./util/core/use-promise-library");
 
 const DEFAULT_LEAK_THRESHOLD = 10000;
 
@@ -80,7 +79,6 @@ function Sandbox(opts = {}) {
     const sandbox = this;
     const assertOptions = opts.assertOptions || {};
     let fakeRestorers = [];
-    let promiseLib;
 
     let collection = [];
     let loggedLeakWarning = false;
@@ -117,8 +115,6 @@ function Sandbox(opts = {}) {
             addToCollection(method);
         });
 
-        usePromiseLibrary(promiseLib, ownMethods);
-
         return stubbed;
     };
 
@@ -177,7 +173,6 @@ function Sandbox(opts = {}) {
         const m = sinonMock.apply(null, arguments);
 
         addToCollection(m);
-        usePromiseLibrary(promiseLib, m);
 
         return m;
     };
@@ -430,16 +425,12 @@ function Sandbox(opts = {}) {
             forEach(ownMethods, function (method) {
                 addToCollection(method);
             });
-
-            usePromiseLibrary(promiseLib, ownMethods);
         } else if (Array.isArray(types)) {
             for (const accessorType of types) {
                 addToCollection(spy[accessorType]);
-                usePromiseLibrary(promiseLib, spy[accessorType]);
             }
         } else {
             addToCollection(spy);
-            usePromiseLibrary(promiseLib, spy);
         }
 
         return spy;
@@ -524,13 +515,6 @@ function Sandbox(opts = {}) {
         addToCollection(xhr);
         return xhr;
     };
-
-    sandbox.usingPromise = function usingPromise(promiseLibrary) {
-        promiseLib = promiseLibrary;
-        collection.promiseLibrary = promiseLibrary;
-
-        return sandbox;
-    };
 }
 
 Sandbox.prototype.match = match;

lib/sinon/sandbox.js

@@ -485,47 +485,4 @@ describe("fake", function () {
             assert.same(myFake.printf, proxy.printf);
         });
     });
-
-    describe(".usingPromise", function () {
-        before(requirePromiseSupport);
-
-        it("should exist and be a function", function () {
-            assert(fake.usingPromise);
-            assert.isFunction(fake.usingPromise);
-        });
-
-        it("should set the promise used by resolve", function () {
-            const promise = {
-                resolve: function (value) {
-                    return Promise.resolve(value);
-                },
-            };
-            const object = {};
-
-            const myFake = fake.usingPromise(promise).resolves(object);
-
-            return myFake().then(function (actual) {
-                assert.same(actual, object, "Same object resolved");
-            });
-        });
-
-        it("should set the promise used by reject", function () {
-            const promise = {
-                reject: function (err) {
-                    return Promise.reject(err);
-                },
-            };
-            const reason = new Error();
-
-            const myFake = fake.usingPromise(promise).rejects(reason);
-
-            return myFake()
-                .then(function () {
-                    referee.fail("this should not resolve");
-                })
-                .catch(function (actual) {
-                    assert.same(actual, reason, "Same object resolved");
-                });
-        });
-    });
 });