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: upgrade jsdom #8851

Merged
merged 3 commits into from
Aug 19, 2019
Merged

chore: upgrade jsdom #8851

merged 3 commits into from
Aug 19, 2019

Conversation

SimenB
Copy link
Member

@SimenB SimenB commented Aug 19, 2019

Summary

Finally 🙂

Test plan

Green CI

@@ -60,11 +60,11 @@ test('works with jsdom testEnvironmentOptions config JSON', () => {
'--config=' +
JSON.stringify({
testEnvironmentOptions: {
userAgent: 'Agent/007',
url: 'https://jestjs.io',
Copy link
Member Author

@SimenB SimenB Aug 19, 2019

Choose a reason for hiding this comment

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

cannot set userAgent this way anymore. See jsdom/jsdom#2279

@SimenB SimenB merged commit dabba86 into jestjs:next Aug 19, 2019
@SimenB SimenB deleted the up-jsdom branch August 19, 2019 11:58
SimenB added a commit that referenced this pull request Aug 19, 2019
thymikee pushed a commit to SimenB/jest that referenced this pull request Aug 19, 2019
SimenB added a commit that referenced this pull request Aug 21, 2019
@kentcdodds
Copy link
Contributor

HOOOORAYYYY!!!!!!!!!!!!!!!!!!!

@kentcdodds
Copy link
Contributor

I noticed the next release was published. Does that mean Jest 25 (with these changes) is coming soon?

@SimenB
Copy link
Member Author

SimenB commented Aug 22, 2019

We were planning on making quite a few changes for Jest 25: https://github.com/facebook/jest/milestone/9

However, I wonder if just dropping node 6 and upgrading jsdom is a nice little major release that should be an easy upgrade for people, and we can make 26 be more breaking. Just dropping Node 6 (meaning we no longer transpile async-await) and upgrading micromatch reduced test time by ~10% on mac and linux and ~20% on windows for Jest's own tests, so it'd be awesome to get into people's hands.

Thoughts?

@kentcdodds
Copy link
Contributor

As a maintainer of a testing library that relies on jsdom, having the latest version would be really nice :)

SimenB added a commit that referenced this pull request Aug 22, 2019
SimenB added a commit that referenced this pull request Aug 22, 2019
@SimenB SimenB mentioned this pull request Aug 22, 2019
@eweap
Copy link

eweap commented Aug 22, 2019

Yeah ! A major version for node 6 drop and jsdom upgrade would be great !

@SimenB
Copy link
Member Author

SimenB commented Aug 22, 2019

You can install jest@next now. We'll be going for a smaller 25 as I talked about above - will be writing a short blog post tomorrow and it'll probably be latest tomorrow or early next week (friday deploys are... sketchy)

@SimenB
Copy link
Member Author

SimenB commented Aug 29, 2019

25 stable is currently a bit delayed due to issues rolling it out at FB (something with how they fake promises is broken). Hopefully they'll figure something out and we can proceed 🙂

@ahnpnl
Copy link
Contributor

ahnpnl commented Sep 10, 2019

@thymikee does it mean jest-preset-angular can remove jest-environment-jsdom-thirteen after jest 25 rolls out ?

@thymikee
Copy link
Collaborator

@ahnpnl yes

@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants