[cryptodev-2s]

Explanation

Use Vitest's module-promise overload for the remaining partial mock factories in keyring-controller, phishing-controller, and user-operation-controller. Partial factories receive the inferred importOriginal helper, removing the repeated module path and explicit typeof import(...) generic.

References

• Follow-up to test: run utils on Vitest #10871.
• Vitest's typed module mock API.

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've communicated my changes to consumers by updating changelogs for packages I've changed
• I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

---

Note

Low Risk
Test-only mock setup refactors with no production code or runtime behavior changes.

Overview
This PR refactors partial vi.mock factories in controller test files to Vitest’s typed module-promise overload: vi.mock(import('…'), async (importOriginal) => ({ ...(await importOriginal()), … })).

Affected tests: uuid v4 stub in keyring-controller, and @metamask/controller-utils partial mocks (safelyExecuteWithTimeout or query) in phishing-controller and user-operation-controller. Behavior is unchanged—still spreads the real module and overrides specific exports—but drops duplicate module path strings and explicit typeof import(...) generics in favor of inferred importOriginal. Follow-up to the same pattern used in #10871.

^{Reviewed by Cursor Bugbot for commit a5b1baa. Bugbot is set up for automated code reviews on this repo. Configure here.}