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

chore(tests): advance the future time by 1000 yrs #3410

Merged
merged 1 commit into from
Jan 22, 2020

Conversation

johnjbarton
Copy link
Contributor

5 years ago someone set the test file time to 2020. And here we are.

@AppVeyorBot
Copy link

Build karma 2461 completed (commit fd783a8328 by @johnjbarton)

@karmarunnerbot
Copy link
Member

Build karma 63 completed (commit 57b69f4e70 by @johnjbarton)

@karmarunnerbot
Copy link
Member

Build karma 62 completed (commit fd783a8328 by @johnjbarton)

@@ -596,13 +596,13 @@ describe('FileList', () => {
emitter.on('file_list_modified', modified)

return list.refresh().then((files) => {
mockFs._touchFile('/some/b.js', '2020-01-01')
mockFs._touchFile('/some/b.js', '3020-01-01')
Copy link
Collaborator

Choose a reason for hiding this comment

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

this is advancing the future time by 1000 years. The commit message needs to be updated. Please also update the PR description.

5 years ago some one set the test file time to 2020. And here we are.
@johnjbarton johnjbarton changed the title chore(tests): advance the future time by 100 yrs chore(tests): advance the future time by 1000 yrs Jan 22, 2020
@karmarunnerbot
Copy link
Member

Build karma 65 completed (commit 684f2a95ae by @johnjbarton)

@johnjbarton johnjbarton merged commit 7ea120b into karma-runner:master Jan 22, 2020
@karmarunnerbot
Copy link
Member

🎉 This PR is included in version 5.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

4 participants