Skip to content

Move Jest config from package.json to jest.config.js - #5680

Merged
mstange merged 2 commits into
firefox-devtools:mainfrom
mstange:external-jest-config
Nov 17, 2025
Merged

Move Jest config from package.json to jest.config.js#5680
mstange merged 2 commits into
firefox-devtools:mainfrom
mstange:external-jest-config

Conversation

@mstange

Copy link
Copy Markdown
Contributor

Having this config outside of package.json is consistent with the rest of our configurations. It also lets us add comments.

And it will make it easier to add other test environments, for example if we want to test how code runs in a pure node environment rather than in the browser-like environment.

@mstange
mstange requested a review from canova as a code ownerNovember 15, 2025 17:15
@codecov

codecovBot commented Nov 15, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.62%. Comparing base (abd8ccf) to head (548d48e).
⚠️ Report is 48 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #5680 +/- ##
=======================================
Coverage 85.62% 85.62% =======================================
Files 312 312 Lines 30875 30879 +4 Branches 8507 8413 -94 =======================================
+ Hits 26436 26440 +4 
Misses 4009 4009 Partials 430 430 

☔ 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.

@mstange
mstangeforce-pushed the external-jest-config branch from a2a7930 to 548d48eCompareNovember 15, 2025 17:22

@canovacanova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

@mstange
mstange enabled auto-merge November 17, 2025 18:48
@mstange
mstange merged commit 611c19f into firefox-devtools:mainNov 17, 2025
12 of 13 checks passed
@canovacanova mentioned this pull request Dec 18, 2025
canova added a commit that referenced this pull request Dec 18, 2025
Changes:
[Markus Stange] Use a longer test timeout when debugging with VS code.
(#5679)
[Markus Stange] Move Jest config from package.json to jest.config.js
(#5680)
[Markus Stange] Make binary profile format parsing use Uint8Array
instead of ArrayBuffer (#5678)
[Markus Stange] Use workbox-cli to generate the service worker (#5681)
[Nazım Can Altınova] Migrate from Appveyor to GitHub Actions Windows
runners (#5660)
[Nazım Can Altınova] Remove some unused dependencies (#5696)
[Nazım Can Altınova] Update the document links and sections (#5705)
[Nazım Can Altınova] Clear selected and expanded call node paths on
browser back button if it removes transforms (#5701)
[Nazım Can Altınova] Properly type the Map and Set objects (#5623)
[Valentin Gosu] Add priorityHeader field to network requests (#5707)
[Nazım Can Altınova] Redirect unpublished url loads to the homepage
similar to from-file (#5712)
[Florian Quèze/Nazım Can Altınova] Add an importer for the text format
taken as input by flamegraph.pl. (#5359)
[Florian Quèze] Improve the import of profiles generated from clang
-ftime-trace=file.json (#5714)
[Markus Stange] Move React stuff out of marker schema logic module.
(#5720)
And thanks to our localizers:
en-CA: chutten
en-CA: Paul
es-CL: ravmn
fr: Théo Chevalier
fur: Fabio Tomat
ru: berry
tr: Selim Şumlu
zh-CN: Olvcpr423
zh-CN: wxie
Sign up for freeto 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.

2 participants

@mstange@canova