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 dependencies to latest #335

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

KaliedaRik
Copy link

What does this change?

This PR updates the Jest tool to the latest versions.

The primary aim of this update is to get rid of some Dependabot security alerts, and to lower technical debt going forward.

How to test

  • Run yarn test locally - all tests should pass, as before.
  • Run the PR in the CODE environment to see if the new system brakes any CI steps.
  • Test in CODE - everything should operate as before.

@KaliedaRik KaliedaRik requested review from twrichards and a team as code owners December 31, 2024 13:49
@KaliedaRik KaliedaRik marked this pull request as draft December 31, 2024 14:06
@KaliedaRik
Copy link
Author

Converting this PR to draft as we have failing tests.

Two test suites (in .ts files) pass with no issue:

Screenshot 2024-12-31 at 14 04 12 Screenshot 2024-12-31 at 14 04 30

The third fails, complaining that the jest.config.js hasn't been set up properly to handle Typescript and/or ES Modules. I don't believe these reports, given that two other test suites work fine:

Screenshot 2024-12-31 at 14 03 47

The failure appears to be in a node_modules file - maybe forcing Jest to ignore the node_modules folder will fix this? (Jest claims it ignores that folder out-of-the-box ... but I'm not convinced this is entirely true)

Screenshot 2024-12-31 at 14 05 49

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

Successfully merging this pull request may close these issues.

1 participant