Skip to content

Conversation

pauljung14
Copy link
Contributor

@pauljung14 pauljung14 commented Sep 24, 2025

JIRA Ticket(s) if any

Description

Bump jest to v28. Resolve security vulnerabilities in downstream form-data dependency.

@pauljung14 pauljung14 requested a review from mprew97 as a code owner September 24, 2025 20:04
@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.49%. Comparing base (e3bc4e7) to head (3635aa5).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #532   +/-   ##
=======================================
  Coverage   74.49%   74.49%           
=======================================
  Files          57       57           
  Lines        2741     2741           
  Branches      803      820   +17     
=======================================
  Hits         2042     2042           
  Misses        698      698           
  Partials        1        1           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

moduleNameMapper: pathsToModuleNameMapper(config.compilerOptions.paths, {
prefix: '<rootDir>/'
}),
setupFiles: ['fake-indexeddb/auto', 'jest-localstorage-mock'],
Copy link
Contributor Author

Choose a reason for hiding this comment

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

mock localstorage doesn't come bundled with jest anymore

mockGlobalProperty('crypto', webcrypto);

const store = {};
mockGlobalProperty('localStorage', {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

localStorage doesn't come bundled with jest anymore (but it's needed for generating UUA mocks)

Object.defineProperty(global, property, { writable: true, value });
};

mockGlobalProperty('crypto', webcrypto);
Copy link
Contributor Author

Choose a reason for hiding this comment

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

crypto.getRandomValues don't come bundled with jest anymore (but it's needed for generating uuid mocks)

@pauljung14 pauljung14 changed the title [MOB-12122] Bump jest to v28 [MOB-12122] Bump jest to v28 to resolve security vulnerabilities Sep 25, 2025
@pauljung14 pauljung14 merged commit 1483ef5 into main Sep 29, 2025
1 check passed
@pauljung14 pauljung14 deleted the MOB-12122 branch September 29, 2025 19:32
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.

3 participants