Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 390 implicit dependencies on faking interval #391

Conversation

fatso83
Copy link
Contributor

@fatso83 fatso83 commented Jun 18, 2021

Purpose (TL;DR) - mandatory

Remove implicit dependency on *Interval functions in the shouldAdvanceTime feature.

Background (Problem in detail) - optional

See #390

@codecov
Copy link

codecov bot commented Jun 18, 2021

Codecov Report

Merging #391 (830d73e) into master (cc30595) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #391      +/-   ##
==========================================
+ Coverage   93.84%   93.85%   +0.01%     
==========================================
  Files           1        1              
  Lines         585      586       +1     
==========================================
+ Hits          549      550       +1     
  Misses         36       36              
Flag Coverage Δ
unit 93.85% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/fake-timers-src.js 93.85% <100.00%> (+0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc30595...830d73e. Read the comment docs.

@fatso83 fatso83 force-pushed the issue-390-implicit-dependencies-on-faking-interval branch from dfc764e to 31b1127 Compare June 18, 2021 16:12
@fatso83
Copy link
Contributor Author

fatso83 commented Jun 21, 2021

Not heard anything. Merging.

@fatso83 fatso83 merged commit 604f090 into sinonjs:master Jun 21, 2021
@fatso83 fatso83 deleted the issue-390-implicit-dependencies-on-faking-interval branch June 21, 2021 09:13
fatso83 referenced this pull request Jun 21, 2021
#102)

lolex can now attach itself to the system timers and automatically advance
mocked time at a certain 'real' interval

- changed uninstall signature to carry over configuration flags in cases where this might be necessary
- updated shouldAdvanceTime functionality to support new install method signature
- moved time delta parameter to config
* Updated Readme.md and History.md
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.

None yet

1 participant