Skip to content

Enhanced retain.js tests by fake timers #444

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

Merged
merged 4 commits into from
Feb 22, 2020

Conversation

gnought
Copy link
Collaborator

@gnought gnought commented Feb 22, 2020

No description provided.

Verified

This commit was signed with the committer’s verified signature.
gnought Gnought

Verified

This commit was signed with the committer’s verified signature.
gnought Gnought

Verified

This commit was signed with the committer’s verified signature.
gnought Gnought

Verified

This commit was signed with the committer’s verified signature.
gnought Gnought
@gnought
Copy link
Collaborator Author

gnought commented Feb 22, 2020

@mcollina looks good?

Copy link
Collaborator

@mcollina mcollina left a comment

Choose a reason for hiding this comment

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

lgtm

@gnought gnought merged commit e4b78df into moscajs:master Feb 22, 2020
@gnought gnought deleted the feature/test_retain_fake_timers branch February 22, 2020 15:32
@coveralls
Copy link

coveralls commented Nov 26, 2024

Pull Request Test Coverage Report for Build 8b42d25394f9c6a0c904a28ed1d5e64ed1ef8543-PR-444

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 99.72%

Totals Coverage Status
Change from base Build 37b04b7778d0ebcd410ffda33787f92be975c1da: 0.0%
Covered Lines: 762
Relevant Lines: 762

💛 - Coveralls

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants