Skip to content

chore(gas-fee-controller): replace Sinon with Jest fake timers#7968

Merged
cryptodev-2s merged 1 commit intomainfrom
chore/gas-fee-controller-replace-sinon-with-jest
Feb 18, 2026
Merged

chore(gas-fee-controller): replace Sinon with Jest fake timers#7968
cryptodev-2s merged 1 commit intomainfrom
chore/gas-fee-controller-replace-sinon-with-jest

Conversation

@cryptodev-2s
Copy link
Contributor

@cryptodev-2s cryptodev-2s commented Feb 17, 2026

Explanation

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Low Risk
Test-only timing/mocking refactor plus dependency removal; production code paths are unchanged, with risk mainly limited to potential flaky/incorrect timer advancement in the updated tests.

Overview
Updates GasFeeController.test.ts to stop using sinon.useFakeTimers()/clock.tickAsync() and instead rely on Jest fake timers (jest.useFakeTimers, jest.advanceTimersByTimeAsync, jest.advanceTimersToNextTimerAsync) for polling and NetworkController provider initialization flows.

Removes the sinon devDependency (and lockfile entry) and drops the related ESLint suppression that was only needed for the Sinon-based import pattern.

Written by Cursor Bugbot for commit 993f7b6. This will update automatically on new commits. Configure here.

@cryptodev-2s cryptodev-2s requested review from a team as code owners February 17, 2026 22:36
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

@cryptodev-2s cryptodev-2s force-pushed the chore/gas-fee-controller-replace-sinon-with-jest branch 4 times, most recently from 0a4771e to 2afe7e1 Compare February 17, 2026 23:48
@cryptodev-2s cryptodev-2s force-pushed the chore/gas-fee-controller-replace-sinon-with-jest branch from 2afe7e1 to 993f7b6 Compare February 17, 2026 23:49
@cryptodev-2s cryptodev-2s added this pull request to the merge queue Feb 18, 2026
Merged via the queue into main with commit d2a1ec0 Feb 18, 2026
737 of 904 checks passed
@cryptodev-2s cryptodev-2s deleted the chore/gas-fee-controller-replace-sinon-with-jest branch February 18, 2026 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants