…ations + Mobile Core UX (#28297) ## **Description**  Part of the analytics cleanup workstream migrating from the legacy `useMetrics` hook to the new `useAnalytics` hook. This PR covers ~3 files in the Notifications and Mobile Core UX areas: - `app/components/UI/Notification/List/index.tsx` — replaces `useMetrics` with `useAnalytics` - `app/components/UI/MarketInsights/Views/MarketInsightsView/MarketInsightsView.tsx` — fixes `MetaMetricsEvents` import to source from `core/Analytics` directly - Tests updated to use `createMockUseAnalyticsHook` and `jest.mocked()` pattern instead of manual mock factories ## **Changelog**  CHANGELOG entry: null ## **Related issues** Fixes: #26818 ## **Manual testing steps** N/A ## **Screenshots/Recordings** ### **Before** N/A ### **After** N/A ## **Pre-merge author checklist** - [x] I've followed [MetaMask Contributor Docs](https://github.com/MetaMask/contributor-docs) and [MetaMask Mobile Coding Standards](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/CODING_GUIDELINES.md). - [x] I've completed the PR template to the best of my ability - [x] I've included tests if applicable - [x] I've documented my code using [JSDoc](https://jsdoc.app/) format if applicable - [x] I've applied the right labels on the PR (see [labeling guidelines](https://github.com/MetaMask/metamask-mobile/blob/main/.github/guidelines/LABELING_GUIDELINES.md)). Not required for external contributors. ## **Pre-merge reviewer checklist** - [ ] I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed). - [ ] I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.  --- > [!NOTE] > **Low Risk** > Low risk refactor limited to analytics plumbing and test mocks; primary risk is missing/incorrect event tracking due to hook or import changes. > > **Overview** > Refactors Notifications analytics to use the new `useAnalytics` hook (instead of legacy `useMetrics`) when marking notifications read and emitting `MetaMetricsEvents.NOTIFICATION_CLICKED`. > > Updates related tests to mock `useAnalytics` via `createMockUseAnalyticsHook`, and removes now-unused metrics mocking from `MainNotificationToggle.hooks.test.tsx`. Also aligns `MarketInsightsView` to import `MetaMetricsEvents` from `core/Analytics`. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit a7c19a6. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup>

Uh oh!

PR D4: migrate useMetrics to useAnalytics in Notifications + Mobile Core UX (~3 files) #26818

Description

Metadata

Metadata

Assignees

Labels

Type

Fields

Projects

Milestone

Relationships

Development

Issue actions