Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: share the same user storage mock instance in tests #28119

Merged
merged 3 commits into from
Oct 28, 2024

Conversation

mathieuartu
Copy link
Contributor

@mathieuartu mathieuartu commented Oct 28, 2024

Description

This PR ensures we share the same UserStorageMockttpController instance for every user storage endpoint.

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Start E2E tests
  2. Make sure they pass

Screenshots/Recordings

Before

After

Pre-merge author checklist

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.

@mathieuartu mathieuartu added the team-notifications Notifications team label Oct 28, 2024
Copy link
Contributor

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@metamaskbot
Copy link
Collaborator

Builds ready [aaee978]
Page Load Metrics (2016 ± 50 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint1875223520249747
domContentLoaded1847218119819646
load18672244201610550
domInteractive23110592411
backgroundConnect9110332813
firstReactRender582681204723
getState56419199
initialActions00000
loadScripts1313163514568440
setupStore1169292311
uiStartup20862560225813264
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

Copy link
Contributor

@matteoscurati matteoscurati left a comment

Choose a reason for hiding this comment

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

LGTM

@mathieuartu mathieuartu marked this pull request as ready for review October 28, 2024 12:05
@mathieuartu mathieuartu requested a review from a team as a code owner October 28, 2024 12:05
@mathieuartu mathieuartu added this pull request to the merge queue Oct 28, 2024
@metamaskbot
Copy link
Collaborator

Builds ready [7db7aed]
Page Load Metrics (2161 ± 152 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint183329292172315151
domContentLoaded178729192136315151
load183229272161316152
domInteractive23107542211
backgroundConnect96524157
firstReactRender84159108168
getState666302412
initialActions01000
loadScripts132623581626291140
setupStore1192342613
uiStartup204231842403332160
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

Merged via the queue into develop with commit 4be7b4a Oct 28, 2024
76 checks passed
@mathieuartu mathieuartu deleted the feat/share_user_storage_mock_instance branch October 28, 2024 12:41
@github-actions github-actions bot locked and limited conversation to collaborators Oct 28, 2024
@metamaskbot metamaskbot added the release-12.7.0 Issue or pull request that will be included in release 12.7.0 label Oct 28, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
release-12.7.0 Issue or pull request that will be included in release 12.7.0 team-notifications Notifications team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants