Skip to content

test: Add manual scenario for upgrade testing#26317

Merged
HowardBraham merged 2 commits intodevelopfrom
upgrade-testing
Aug 16, 2024
Merged

test: Add manual scenario for upgrade testing#26317
HowardBraham merged 2 commits intodevelopfrom
upgrade-testing

Conversation

@chloeYue
Copy link
Copy Markdown
Contributor

@chloeYue chloeYue commented Aug 2, 2024

Description

This PR introduces a manual testing scenario aimed at ensuring the seamless upgrade of MetaMask extension from previous branch to the release branch. The primary goal is to validate that the upgrade process preserves user data, maintains essential functionality, handles popup modals related to upgrade news correctly, and does not introduce any disruptions to the user experience.

Open in GitHub Codespaces

Related issues

Fixes: #26085

Manual testing steps

Follow testing steps in this manual scenario

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.

@chloeYue chloeYue added area-documentation Issues relating to documentation, in the codebase and off. team-extension-platform Extension Platform team area-qa Relating to QA work (Quality Assurance) labels Aug 2, 2024
@chloeYue chloeYue self-assigned this Aug 2, 2024
@chloeYue chloeYue requested a review from a team as a code owner August 2, 2024 14:37
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Aug 2, 2024

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.

@chloeYue chloeYue marked this pull request as draft August 2, 2024 14:37
@chloeYue chloeYue marked this pull request as ready for review August 2, 2024 14:57
@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [7bdb3ed]
Page Load Metrics (496 ± 350 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint713751428641
domContentLoaded10161433617
load411866496728350
domInteractive10161433617
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.04%. Comparing base (69e57d8) to head (84eb1bb).
Report is 4 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #26317      +/-   ##
===========================================
+ Coverage    70.04%   70.04%   +0.01%     
===========================================
  Files         1411     1411              
  Lines        50022    50025       +3     
  Branches     13809    13812       +3     
===========================================
+ Hits         35033    35038       +5     
+ Misses       14989    14987       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

sleepytanya
sleepytanya previously approved these changes Aug 2, 2024
@hjetpoluru hjetpoluru self-requested a review August 2, 2024 16:25
hjetpoluru
hjetpoluru previously approved these changes Aug 2, 2024
Copy link
Copy Markdown
Contributor

@hjetpoluru hjetpoluru left a comment

Choose a reason for hiding this comment

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

LGTM

@chloeYue chloeYue dismissed stale reviews from hjetpoluru and sleepytanya via 84eb1bb August 2, 2024 16:46
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Aug 2, 2024

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [84eb1bb]
Page Load Metrics (320 ± 277 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint722561143919
domContentLoaded10182323718
load411776320576277
domInteractive10182323718
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@hjetpoluru hjetpoluru self-requested a review August 13, 2024 14:16
@HowardBraham HowardBraham merged commit 555eb57 into develop Aug 16, 2024
@HowardBraham HowardBraham deleted the upgrade-testing branch August 16, 2024 16:05
@github-actions github-actions bot locked and limited conversation to collaborators Aug 16, 2024
@metamaskbot metamaskbot added the release-12.5.0 Issue or pull request that will be included in release 12.5.0 label Aug 16, 2024
@gauthierpetetin gauthierpetetin added release-12.4.0 Issue or pull request that will be included in release 12.4.0 and removed release-12.5.0 Issue or pull request that will be included in release 12.5.0 labels Sep 11, 2024
@hjetpoluru hjetpoluru added the e2e-test End to end test for the MetaMask extension label Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-documentation Issues relating to documentation, in the codebase and off. area-qa Relating to QA work (Quality Assurance) e2e-test End to end test for the MetaMask extension release-12.4.0 Issue or pull request that will be included in release 12.4.0 team-extension-platform Extension Platform team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Add manual scenario for extension upgrade testing

6 participants