Skip to content

fix: test vault decryption chrome#24830

Merged
seaona merged 3 commits intodevelopfrom
flaky-fix-vault-decryption
May 28, 2024
Merged

fix: test vault decryption chrome#24830
seaona merged 3 commits intodevelopfrom
flaky-fix-vault-decryption

Conversation

@seaona
Copy link
Copy Markdown
Member

@seaona seaona commented May 28, 2024

Description

There is a current ci failure on the vault decryption chrome test where we are trying to disable all the setting options using the toggles. However, one of the toggles has a confirmation popup that is displayed whenever we try to disable it. The current code doesn't take into account the confirmation popup so the test fails at this point.

The fix is to disable this specific setting (Profile Sync) individually, like it's done with another setting which also has a specific confirm popup, and then perform the rest of toggle offs.

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. You can remove the filter for running the vault decryption in this branch (or just check the job I ran below)

Screenshots/Recordings

This is the current failure point, where we are toggling off all the list of items but the Profile Sync has a confirmation pop-up, which makes the test fail at this point.

vault-decrypt-error.mp4

image

Pre-merge author checklist

  • I’ve followed MetaMask 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.

@github-actions
Copy link
Copy Markdown
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.

@seaona seaona added the team-extension-platform Extension Platform team label May 28, 2024
@codecov
Copy link
Copy Markdown

codecov bot commented May 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.67%. Comparing base (7eb2f2b) to head (ed81dc6).
Report is 7 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #24830      +/-   ##
===========================================
- Coverage    65.67%   65.67%   -0.00%     
===========================================
  Files         1360     1360              
  Lines        54105    54105              
  Branches     14066    14066              
===========================================
- Hits         35533    35532       -1     
- Misses       18572    18573       +1     

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

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [ed81dc6]
Page Load Metrics (802 ± 538 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint64167962412
domContentLoaded95515115
load5328708021121538
domInteractive95515116
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 33 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@seaona seaona marked this pull request as ready for review May 28, 2024 16:46
@seaona seaona requested a review from a team as a code owner May 28, 2024 16:46
@seaona seaona self-assigned this May 28, 2024
Copy link
Copy Markdown
Contributor

@chloeYue chloeYue left a comment

Choose a reason for hiding this comment

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

LGTM !

@seaona seaona merged commit 906e918 into develop May 28, 2024
@seaona seaona deleted the flaky-fix-vault-decryption branch May 28, 2024 17:53
@github-actions github-actions bot locked and limited conversation to collaborators May 28, 2024
@gauthierpetetin gauthierpetetin added the release-12.0.0 Issue or pull request that will be included in release 12.0.0 label Jun 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

release-12.0.0 Issue or pull request that will be included in release 12.0.0 team-extension-platform Extension Platform team

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants