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: Update jest packages to v29 #14221

Merged
merged 5 commits into from Aug 31, 2022
Merged

chore: Update jest packages to v29 #14221

merged 5 commits into from Aug 31, 2022

Conversation

gu-stav
Copy link
Contributor

@gu-stav gu-stav commented Aug 25, 2022

What does it do?

This updates jest related dependencies to version 29. A previous attempt to update jest@28 didn't work, because we were blocked on a babel-issue that is resolved in v29.

Jest updated the snapshot format slightly, which is why some of them changed: https://jestjs.io/blog/2022/08/25/jest-29

Why is it needed?

Update dependencies.

How to test it?

Jest tests itself.

Related issue(s)/PR(s)

@gu-stav gu-stav added source: tooling Source is GitHub tooling/tests/ect pr: fix This PR is fixing a bug labels Aug 25, 2022
@gu-stav gu-stav added this to the 4.3.6 milestone Aug 25, 2022
@codecov
Copy link

codecov bot commented Aug 25, 2022

Codecov Report

Merging #14221 (ac2060a) into main (5555d54) will increase coverage by 2.11%.
The diff coverage is n/a.

❗ Current head ac2060a differs from pull request most recent head 51caa5f. Consider uploading reports for the commit 51caa5f to get more accurate results

@@            Coverage Diff             @@
##             main   #14221      +/-   ##
==========================================
+ Coverage   55.44%   57.55%   +2.11%     
==========================================
  Files        1274     1262      -12     
  Lines       31859    30689    -1170     
  Branches     5750     5747       -3     
==========================================
  Hits        17664    17664              
+ Misses      12372    11205    -1167     
+ Partials     1823     1820       -3     
Flag Coverage Δ
front 61.11% <ø> (+3.15%) ⬆️
unit 49.35% <ø> (+0.03%) ⬆️

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

Impacted Files Coverage Δ
packages/core/admin/admin/src/app.js 0.00% <0.00%> (ø)
packages/core/admin/admin/src/index.js 0.00% <0.00%> (ø)
packages/core/email/admin/src/index.js 0.00% <0.00%> (ø)
packages/core/upload/admin/src/index.js 0.00% <0.00%> (ø)
packages/plugins/i18n/admin/src/index.js 0.00% <0.00%> (ø)
packages/plugins/i18n/admin/src/schemas.js 0.00% <0.00%> (ø)
packages/plugins/i18n/admin/src/permissions.js 0.00% <0.00%> (ø)
packages/core/upload/admin/src/pages/App/index.js 0.00% <0.00%> (ø)
...ckages/plugins/i18n/admin/src/middlewares/index.js 0.00% <0.00%> (ø)
...kages/core/content-type-builder/admin/src/index.js 0.00% <0.00%> (ø)
... and 195 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@alexandrebodin
Copy link
Member

If I remember correctly we add a pbl with v28. Was it fixed with v29?

@gu-stav
Copy link
Contributor Author

gu-stav commented Aug 25, 2022

@alexandrebodin yes I've mentioned the problem and the fix in the PR description. Everything seems to work now.

@gu-stav gu-stav changed the title chore: Update jest-* to v29 chore: Update jest packages to v29 Aug 25, 2022
alexandrebodin
alexandrebodin previously approved these changes Aug 25, 2022
Copy link
Member

@alexandrebodin alexandrebodin left a comment

Choose a reason for hiding this comment

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

Haven't done added testing but the changes look good

alexandrebodin
alexandrebodin previously approved these changes Aug 26, 2022
alexandrebodin
alexandrebodin previously approved these changes Aug 29, 2022
@gu-stav gu-stav merged commit 7f0ddf0 into main Aug 31, 2022
@gu-stav gu-stav deleted the chore/jest-29 branch August 31, 2022 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: fix This PR is fixing a bug source: tooling Source is GitHub tooling/tests/ect
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants