Breaking change in 10.0.1: config.target is no longer supported #2352

kevinoid · 2021-04-08T14:00:29Z

Describe the bug
With [email protected], if options contains target, sinon.useFakeTimers throws a TypeError. This did not occur with [email protected].

To Reproduce

Create index.js with the following content:

const sinon = require('sinon');
sinon.useFakeTimers({ target: { Date } });

Run node index.js after npm install [email protected] and after npm install [email protected]. After installing [email protected] the following error is printed:

/path/to/node_modules/@sinonjs/fake-timers/src/fake-timers-src.js:1299
            throw new TypeError(
            ^

TypeError: config.target is no longer supported. Use `withGlobal(target)` instead.
    at Object.install (/path/to/node_modules/@sinonjs/fake-timers/src/fake-timers-src.js:1299:19)
    at createClock (/path/to/node_modules/sinon/lib/sinon/util/fake-timers.js:12:31)
    at exports.useFakeTimers (/path/to/node_modules/sinon/lib/sinon/util/fake-timers.js:54:16)
    at Sandbox.useFakeTimers (/path/to/node_modules/sinon/lib/sinon/sandbox.js:396:46)
    at Object.<anonymous> (/path/to/index.js:2:7)
    at Module._compile (internal/modules/cjs/loader.js:999:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1027:10)
    at Module.load (internal/modules/cjs/loader.js:863:32)
    at Function.Module._load (internal/modules/cjs/loader.js:708:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:60:12)

Expected behavior
No exception would occur.

Additional context
The problem was introduced by f838d78 which bumped @sinonjs/fake-timers from ^6.0.1 to ^7.0.4 which includes sinonjs/fake-timers#318.

The text was updated successfully, but these errors were encountered:

regseb · 2021-04-08T16:04:50Z

I have a problem with [email protected] and typescript which is surely related to this issue.

To reproduce, create index.js with the following content:

const sinon = require("sinon");

And execute:

npm install [email protected]
npm install @types/[email protected]
npm install [email protected]
npx tsc index.js --declaration --allowJs --emitDeclarationOnly

The result is:

node_modules/@types/sinon/index.d.ts:778:36 - error TS2694: Namespace '"/tmp/testcase/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'TimerId'.

778     type SinonTimerId = FakeTimers.TimerId;
                                       ~~~~~~~

node_modules/@types/sinon/index.d.ts:780:39 - error TS2694: Namespace '"/tmp/testcase/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'InstalledMethods'.

780     type SinonFakeTimers = FakeTimers.InstalledMethods &
                                          ~~~~~~~~~~~~~~~~

node_modules/@types/sinon/index.d.ts:781:20 - error TS2694: Namespace '"/tmp/testcase/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'NodeClock'.

781         FakeTimers.NodeClock &
                       ~~~~~~~~~

node_modules/@types/sinon/index.d.ts:782:20 - error TS2694: Namespace '"/tmp/testcase/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'BrowserClock'.

782         FakeTimers.BrowserClock & {
                       ~~~~~~~~~~~~


Found 4 errors.

fatso83 · 2021-04-08T19:43:18Z

@regseb ~~we don't maintain those typescript bindings, but ship our own~~. Ah, sorry, I saw the problem is the fake-timer definitions which we DO maintain. This change needs to be reverted.

freelerobot · 2021-04-08T20:47:52Z

Hi thanks for fixing. what's the ETA on this?

We'll pin our version, if it's going to take a few days.

fatso83 · 2021-04-09T09:07:06Z

@nicoleczhu pin your version. no one is paid to do this, so it will be when someone does the work to fix and test this manually. that being said, I think this has a high priority, so it will be looked at soon by someone in @sinonjs/sinon-core

bcoe · 2021-04-09T15:37:44Z

@fatso83 I empathize with the no one is paid to do this argument, as someone who's sunk countless hours into handling CVEs for injection bugs logged against yargs.

But just wanted to make sure you knew about this feature on npm:

npm dist-tag add [email protected] latest

☝️ this would get folks unblocked who are broken by 10.0.1, but doesn't require a hard unpublish of 10.0.1.

https://github.com/sinonjs/fake-timers appears to have added types which are incompatible with the current implementation in DefinitelyTyped. Refs: sinonjs/sinon#2353 Refs: sinonjs/sinon#2352

sinonjs/sinon#2352

* fix: temporarily pin sinon at 10.0.0 until this is fixed: sinonjs/sinon#2352 * chore: let npm rearrange the imports for some devDependencies

mroderick · 2021-04-10T17:18:43Z

But just wanted to make sure you knew about this feature on npm:

npm dist-tag add [email protected] latest

☝️ this would get folks unblocked who are broken by 10.0.1, but doesn't require a hard unpublish of 10.0.1.

Top tip! Thank you @bcoe

mroderick · 2021-04-11T13:52:58Z

Thank you for the very accurate bug report @kevinoid 👍

Your analysis is correct: the updated version of @sinonjs/fake-timers does indeed introduce a breaking change.

In the npm registry the 10.0.1 package is deprecated and 10.0.0 is marked as latest, so this should not affect any more users (hat tip @bcoe).

Once #2351 is merged and #2353 is resolved, I'll release a new major version detailing the breaking change in the changelog

bcoe · 2021-04-12T15:35:07Z

@mroderick thank you for getting us unblocked 👏

fatso83 · 2021-04-20T07:44:53Z

Once #2351 is merged and #2353 is resolved, I'll release a new major version detailing the breaking change in the changelog

@mroderick are we there? I am not sure what is needed to get #2353 resolved. The NPM fix seemed like it resolved the issue and #2351 prevents it, or ...?

fatso83 · 2021-05-24T19:40:46Z

I just retried @regseb's example on the current Sinon code, and I got 0 errors, so it seems like this should be good ... Releasing Sinon 11.

fatso83 · 2021-05-24T21:17:10Z

As Sinon 11 is out, the breaking change comes with a major version and should be good. Mentioned in the changelog.

perrin4869 · 2021-05-25T04:51:10Z

My typescript project is failing to compile with sinon@11 with the error mentioned above

fatso83 · 2021-05-25T05:20:45Z

@perrin4869 Make sure you update all the dependencies. Do you still have that error if you clean rm -r node_modules && npm install? I did try out regseb's example locally (using npm link) before publishing, and did not see any issue, and AFAIK the error is caused by the fake-timers dependency, so maybe it's stuck on an earlier revision?

fatso83 · 2021-05-25T05:48:57Z

@perrin4869 If you still have problems, could you please file a new issue and make some reproducible test case for us? I do not have anything to reproduce. The only bit I have works fine.

perrin4869 · 2021-05-25T06:14:06Z

hm... Will try to dig into it later, maybe dependabot did a derp, but looking at the package-lock.json it seems fake-timers updated correctly

fatso83 · 2021-05-25T07:51:43Z

@perrin4869 Remember you also need to update your types. Using the example by @regseb:

$ npm i [email protected]
+ [email protected]
updated 6 packages and audited 20 packages in 0.786s

2 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

$ npx tsc index.js --declaration --allowJs --emitDeclarationOnly
node_modules/@types/sinon/index.d.ts:778:36 - error TS2694: Namespace '"/home/carlerik/dev/nimble/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'TimerId'.

778     type SinonTimerId = FakeTimers.TimerId;
                                       ~~~~~~~

node_modules/@types/sinon/index.d.ts:780:39 - error TS2694: Namespace '"/home/carlerik/dev/nimble/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'InstalledMethods'.

780     type SinonFakeTimers = FakeTimers.InstalledMethods &
                                          ~~~~~~~~~~~~~~~~

node_modules/@types/sinon/index.d.ts:781:20 - error TS2694: Namespace '"/home/carlerik/dev/nimble/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'NodeClock'.

781         FakeTimers.NodeClock &
                       ~~~~~~~~~

node_modules/@types/sinon/index.d.ts:782:20 - error TS2694: Namespace '"/home/carlerik/dev/nimble/node_modules/@sinonjs/fake-timers/types/fake-timers-src"' has no exported member 'BrowserClock'.

782         FakeTimers.BrowserClock & {
                       ~~~~~~~~~~~~


Found 4 errors.


carlerik at FAT-WORKSTATION in ~/dev/nimble (develop)
$ npm i @types/sinon@latest
+ @types/[email protected]
removed 1 package, updated 1 package and audited 19 packages in 0.455s

2 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities


carlerik at FAT-WORKSTATION in ~/dev/nimble (develop)
$ npx tsc index.js --declaration --allowJs --emitDeclarationOnly 

# no errors!

perrin4869 · 2021-05-25T14:20:22Z

Ah didn't notice @types/sinon was installed, it wasn't listed in my package.json file. Yeah updating it fixed it, thanks!

thebrianchen mentioned this issue Apr 8, 2021

fix: use BigInt when calculating nanos in Timestamp.fromMillis() googleapis/nodejs-firestore#1468

Merged

freelerobot mentioned this issue Apr 8, 2021

Synthesis failed for nodejs-logging googleapis/nodejs-logging#1033

Closed

feywind mentioned this issue Apr 8, 2021

fix: added support for Opentelemetry 0.18 googleapis/nodejs-pubsub#1234

Merged

4 tasks

This was referenced Apr 8, 2021

Synthesis failed for nodejs-storage googleapis/nodejs-storage#1439

Closed

Synthesis failed for gcs-resumable-upload googleapis/gcs-resumable-upload#411

Closed

bcoe mentioned this issue Apr 9, 2021

fix(sinon): use sinonjs__fake-timers rather than alias DefinitelyTyped/DefinitelyTyped#52294

Closed

7 tasks

feywind added a commit to feywind/nodejs-pubsub that referenced this issue Apr 9, 2021

fix: temporarily pin sinon at 10.0.0 until this is fixed:

be2aeb0

sinonjs/sinon#2352

feywind mentioned this issue Apr 9, 2021

fix: temporarily pin sinon at 10.0.0 googleapis/nodejs-pubsub#1252

Merged

feywind added a commit to googleapis/nodejs-pubsub that referenced this issue Apr 9, 2021

fix: temporarily pin sinon at 10.0.0 (#1252)

0922164

* fix: temporarily pin sinon at 10.0.0 until this is fixed: sinonjs/sinon#2352 * chore: let npm rearrange the imports for some devDependencies

bcoe mentioned this issue Apr 11, 2021

Sinon 10 uses fake-timers 7 and nise 5, nise 5 uses fake-timers 6. So with one package we get two defferent majors of subpackage. #2353

Closed

fatso83 closed this as completed May 24, 2021

azatoth mentioned this issue Nov 5, 2021

Dependabot pull requests need to be applied ttarnowski/ts-sinon#252

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Breaking change in 10.0.1: config.target is no longer supported #2352

Breaking change in 10.0.1: config.target is no longer supported #2352

kevinoid commented Apr 8, 2021

regseb commented Apr 8, 2021 •

edited

Loading

fatso83 commented Apr 8, 2021 •

edited

Loading

freelerobot commented Apr 8, 2021

fatso83 commented Apr 9, 2021 •

edited

Loading

bcoe commented Apr 9, 2021

mroderick commented Apr 10, 2021

mroderick commented Apr 11, 2021

bcoe commented Apr 12, 2021

fatso83 commented Apr 20, 2021

fatso83 commented May 24, 2021

fatso83 commented May 24, 2021

perrin4869 commented May 25, 2021 •

edited

Loading

fatso83 commented May 25, 2021

fatso83 commented May 25, 2021

perrin4869 commented May 25, 2021

fatso83 commented May 25, 2021

perrin4869 commented May 25, 2021

Breaking change in 10.0.1: config.target is no longer supported #2352

Breaking change in 10.0.1: config.target is no longer supported #2352

Comments

kevinoid commented Apr 8, 2021

regseb commented Apr 8, 2021 • edited Loading

fatso83 commented Apr 8, 2021 • edited Loading

freelerobot commented Apr 8, 2021

fatso83 commented Apr 9, 2021 • edited Loading

bcoe commented Apr 9, 2021

mroderick commented Apr 10, 2021

mroderick commented Apr 11, 2021

bcoe commented Apr 12, 2021

fatso83 commented Apr 20, 2021

fatso83 commented May 24, 2021

fatso83 commented May 24, 2021

perrin4869 commented May 25, 2021 • edited Loading

fatso83 commented May 25, 2021

fatso83 commented May 25, 2021

perrin4869 commented May 25, 2021

fatso83 commented May 25, 2021

perrin4869 commented May 25, 2021

regseb commented Apr 8, 2021 •

edited

Loading

fatso83 commented Apr 8, 2021 •

edited

Loading

fatso83 commented Apr 9, 2021 •

edited

Loading

perrin4869 commented May 25, 2021 •

edited

Loading