Multichain API E2E Test: wallet_notify

[ffmcgee725]

-- test: multichain test dapp e2e test for wallet_notify

Description

Related issues

Fixes:

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). 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.

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@metamask/[email protected]	None	0	1.69 MB	metamaskbot

🚮 Removed packages: npm/@metamask/[email protected]

View full report↗︎

[adonesky1]

Could we rename the test file to wallet_notify.spec.ts ?

[metamaskbot]

Builds ready [38c9ead]

• builds: chrome, firefox
• builds (flask): chrome, firefox
• builds (MMI): chrome
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7
  • common: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 12, 13, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (1721 ± 75 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	409	2065	1655	331	159
		domContentLoaded	1493	1995	1706	156	75
		load	1504	2006	1721	156	75
		domInteractive	23	84	45	22	10
		backgroundConnect	9	48	19	11	5
		firstReactRender	17	101	46	28	13
		getState	4	60	18	21	10
		initialActions	0	1	0	0	0
		loadScripts	1053	1536	1254	148	71
		setupStore	6	82	19	23	11
		uiStartup	1692	2575	2027	217	104

[jiexi]

I wonder if we can use css to hide the extra text rather than adding and removing it from the DOM

[ffmcgee725]

@adonesky1 I change file name on this one so Jiexi doesn't need to re-approve 👍🏾