Repository navigation
test: run controller-utils on Vitest - #10553
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning MetaMask internal reviewing guidelines:
|
68d12f9 to
675508f
Compare
b92fe81 to
175e025
Compare
675508f to
44ee6e0
Compare
175e025 to
186a8bd
Compare
44ee6e0 to
9f54633
Compare
186a8bd to
69b48d1
Compare
9f54633 to
562489c
Compare
289154f to
3cec491
Compare
562489c to
5cc065d
Compare
3cec491 to
eb6905d
Compare
4e2a256 to
b5e5992
Compare
eb6905d to
425f889
Compare
b5e5992 to
0e78d97
Compare
425f889 to
b5450af
Compare
0e78d97 to
02a0548
Compare
e837efa to
d77b580
Compare
86786bb to
25334b8
Compare
2dcc5b9 to
4fe325d
Compare
3126f77 to
f31c96c
Compare
4fe325d to
ae3585d
Compare
f31c96c to
e5aee1b
Compare
ae3585d to
4665ce9
Compare
4665ce9 to
83f3081
Compare
e5aee1b to
db2b678
Compare
db2b678 to
424497f
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 424497f. Configure here.
f478ce0 to
bb32e5f
Compare
bb32e5f to
380d4b6
Compare
380d4b6 to
e19d14b
Compare

Explanation
Moves
@metamask/controller-utilsto Vitest.References
Checklist
no-changeloglabel.Note
Low Risk
Test-only and tooling changes; production library code is unchanged aside from lockfile transitive deps for jsdom.
Overview
Migrates
@metamask/controller-utilsfrom Jest to Vitest, following the monorepo’s sharedvitest.config.packages.jspattern.Test scripts and devDependencies now use Vitest and
jsdominstead of Jest,jest-environment-jsdom,ts-jest, and the custom Jest jsdom environment (includingTextEncodershims). A newvitest.config.jssets the jsdom environment and coverage thresholds aligned with the old Jest setup.Unit tests import
describe/expect/it(andvifor mocks and fake timers) from Vitest;create-service-policy.test.tsis the bulk of the mechanicaljest→virewrites.tsconfig.jsondrops Jest types,test-runner-package-map.cjsno longer lists this package under Jest, andoxlint-suppressions.jsonis updated for removed Jest config paths and new Vitest lint suppressions.Reviewed by Cursor Bugbot for commit 4101c8a. Bugbot is set up for automated code reviews on this repo. Configure here.