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

Bump jest from 27.5.1 to 28.1.3 in /frontend #2211

Merged
merged 3 commits into from
Aug 4, 2022

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jul 18, 2022

Bumps jest from 27.5.1 to 28.1.3.

Release notes

Sourced from jest's releases.

v28.1.3

Features

  • [jest-leak-detector] Use native FinalizationRegistry when it exists to get rid of external C dependency (#12973)

Fixes

  • [jest-changed-files] Fix a lock-up after repeated invocations (#12757)
  • [@jest/expect-utils] Fix deep equality of ImmutableJS OrderedSets (#12977)
  • [jest-mock] Add index signature support for spyOn types (#13013, #13020)
  • [jest-snapshot] Fix indentation of awaited inline snapshots (#12986)

Chore & Maintenance

  • [*] Replace internal usage of pretty-format/ConvertAnsi with jest-serializer-ansi-escapes (#12935, #13004)
  • [docs] Update spyOn docs (#13000)

New Contributors

Full Changelog: jestjs/jest@v28.1.2...v28.1.3

v28.1.2

Fixes

  • [jest-runtime] Avoid star type import from @jest/globals (#12949)

Chore & Maintenance

  • [docs] Mention that jest-codemods now supports Sinon (#12898)

New Contributors

Full Changelog: jestjs/jest@v28.1.1...v28.1.2

v28.1.1

Features

  • [jest] Expose Config type (#12848)
  • [@jest/reporters] Improve GitHubActionsReporters annotation format (#12826)
  • [@jest/types] Infer argument types passed to test and describe callback functions from each tables (#12885, #12905)

... (truncated)

Changelog

Sourced from jest's changelog.

28.1.3

Features

  • [jest-leak-detector] Use native FinalizationRegistry when it exists to get rid of external C dependency (#12973)

Fixes

  • [jest-changed-files] Fix a lock-up after repeated invocations (#12757)
  • [@jest/expect-utils] Fix deep equality of ImmutableJS OrderedSets (#12977)
  • [jest-mock] Add index signature support for spyOn types (#13013, #13020)
  • [jest-snapshot] Fix indentation of awaited inline snapshots (#12986)

Chore & Maintenance

  • [*] Replace internal usage of pretty-format/ConvertAnsi with jest-serializer-ansi-escapes (#12935, #13004)
  • [docs] Update spyOn docs (#13000)

28.1.2

Fixes

  • [jest-runtime] Avoid star type import from @jest/globals (#12949)

Chore & Maintenance

  • [docs] Mention that jest-codemods now supports Sinon (#12898)

28.1.1

Features

  • [jest] Expose Config type (#12848)
  • [@jest/reporters] Improve GitHubActionsReporters annotation format (#12826)
  • [@jest/types] Infer argument types passed to test and describe callback functions from each tables (#12885, #12905)

Fixes

  • [@jest/expect-utils] Fix deep equality of ImmutableJS OrderedMaps (#12763)
  • [jest-docblock] Handle multiline comments in parseWithComments (#12845)
  • [jest-mock] Improve spyOn error messages (#12901)
  • [jest-runtime] Correctly report V8 coverage with resetModules: true (#12912)
  • [jest-worker] Make JestWorkerFarm helper type to include methods of worker module that take more than one argument (#12839)

Chore & Maintenance

  • [docs] Updated docs to indicate that jest-environment-jsdom is a separate package #12828
  • [docs] Document the comments used by coverage providers #12835
  • [docs] Use docusaurus-remark-plugin-tab-blocks to format tabs with code examples (#12859)
  • [jest-haste-map] Bump walker version (#12324)

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Jul 18, 2022
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/frontend/jest-28.1.3 branch 2 times, most recently from 0d67e5d to 72ef34c Compare July 18, 2022 10:02
@Vinnl
Copy link
Collaborator

Vinnl commented Jul 19, 2022

Carrying over from #2141 (comment):

I investigated the failing test here for a bit, and it looks like the <form>'s submit event does not get triggered when pressing the <button type="submit"> (though a click event on the button does get triggered). There's a new version of the user-event library available that it's recommended to upgrade to, but that didn't fix it right away. Pausing that work here for a bit in case it's a bug that gets fixed soon.

(Oh, and also, jest-environment-jsdom needs to be added as an explicit dependency now.)

Edit: upgrade to the new version in #1978, which also had some breaking changes to incorporate. Hopefully that will also help applying this update.

Edit: after some debugging it looks like the waitlist form's onSubmit event handler gets removed? (Add console.log("FORM:", submitButton.form.onSubmit); to the test to see.) This might be intentional; IIRC React 18 handles events at the root or something? So we might need to upgrade to React 18 first (which depends on some other dependencies also becoming compatible with it.)

Edit: got some more info: testing-library/user-event#401 seems to suggest needing an up-to-date version of @testing-library/react, which in turn says that it requires React 18. So presumably that is indeed the issue; hopefully soon all our deps will be compatible with React 18. It might be worth it trying to figure out which ones are blocking it.

Edit: one other thing I tried is pressing {Enter} while in the input field, but that doesn't trigger the submit either.

I can manually trigger it with this ugly piece of code:

    const propsProp = Object.keys(submitButton.form).find(key => key.startsWith("__reactProps"));
    submitButton.form[propsProp].onSubmit({ preventDefault: jest.fn() });

If I add a console.log to the submit handler, I can then at least see that getting printed, and with that, the test succeeds.

Edit: OK, and in terms of libraries blocking the upgrade, that mostly appear to be:

@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/frontend/jest-28.1.3 branch 3 times, most recently from 6e1054a to 9132e13 Compare July 25, 2022 13:10
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/frontend/jest-28.1.3 branch 3 times, most recently from b3efb89 to e23f0f7 Compare August 1, 2022 07:44
Bumps [jest](https://github.com/facebook/jest/tree/HEAD/packages/jest) from 27.5.1 to 28.1.3.
- [Release notes](https://github.com/facebook/jest/releases)
- [Changelog](https://github.com/facebook/jest/blob/main/CHANGELOG.md)
- [Commits](https://github.com/facebook/jest/commits/v28.1.3/packages/jest)

---
updated-dependencies:
- dependency-name: jest
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot force-pushed the dependabot/npm_and_yarn/frontend/jest-28.1.3 branch from e23f0f7 to bdd740c Compare August 1, 2022 07:47
Vinnl added 2 commits August 3, 2022 14:58
I haven't been able to find the cause of the failures, but since
it's just two tests of a relatively straightforward feature, I'd
like to no longer block the upgrade on it. For more info, see
#2211 (comment)
@Vinnl
Copy link
Collaborator

Vinnl commented Aug 3, 2022

I added two commits, one to migrate, and one to disable the remaining two tests. I'd like to get this upgrade over with, the tests aren't super critical, and unfortunately I haven't been able to figure out why they started failing. I've asked for someone else's review before I merge this.

Copy link
Collaborator

@Vinnl Vinnl left a comment

Choose a reason for hiding this comment

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

I got one thumbs-up and a no-comment, so...

@dependabot merge

@dependabot dependabot bot merged commit f24ee56 into main Aug 4, 2022
@dependabot dependabot bot deleted the dependabot/npm_and_yarn/frontend/jest-28.1.3 branch August 4, 2022 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant