Revert PR #1807 to fix #1814

[fatso83]

Purpose (TL;DR)

closes #1814 by reverting changes introduced in #1807

Background

#1807 started using lolex 2.7 (up from 2.4.2). The regular tests pass, but as the cloud tests running against our suite of browsers only run on merge to master it only became apparent that it broke Safari 9 after the fact. We might try again once sinonjs/fake-timers#190 has been closed.

How to verify - mandatory

1. export SAUCE_USERNAME=sinonjs
2. export SAUCE_ACCESS_KEY=<secret>
3. npm run test-cloud

[fatso83]

Well, that fixed the Safari issues, but we still have some issues with IE 11! @mroderick

  1) fake
       .resolves
         should return a function that resolves to the argument:
     ReferenceError: 'Promise' is undefined
      at f (lib/sinon/fake.js:82)
      at fakeInstance (lib/sinon/fake.js:46)
      at invoke (lib/sinon/spy.js:209)
      at proxy (lib/sinon/spy.js:95)
      at Anonymous function (test/fake-test.js:190)

  2) fake
       .rejects
         should return a function that rejects to the argument:
     ReferenceError: 'Promise' is undefined
      at f (lib/sinon/fake.js:90)
      at fakeInstance (lib/sinon/fake.js:46)
      at invoke (lib/sinon/spy.js:209)
      at proxy (lib/sinon/spy.js:95)
      at Anonymous function (test/fake-test.js:203)

  3) fake
       .rejects
         should return the same error type as it is passed:
     ReferenceError: 'Promise' is undefined
      at f (lib/sinon/fake.js:90)
      at fakeInstance (lib/sinon/fake.js:46)
      at invoke (lib/sinon/spy.js:209)
      at proxy (lib/sinon/spy.js:95)
      at Anonymous function (test/fake-test.js:214)

  4) fake
       .rejects
         when passed a String
           should reject with an Error:
     ReferenceError: 'Promise' is undefined
      at f (lib/sinon/fake.js:90)
      at fakeInstance (lib/sinon/fake.js:46)
      at invoke (lib/sinon/spy.js:209)
      at proxy (lib/sinon/spy.js:95)
      at Anonymous function (test/fake-test.js:224)

[benjamingr]

Looks like either a promise polyfill or a guard around APIs using promises in environments that don't ship them natively is missing @fatso83