Skip to content

Added wait for the overlay spinner before the click was performed#22622

Merged
HowardBraham merged 4 commits intodevelopfrom
e2e/22566-fix-flaky-test-change-language
Jan 23, 2024
Merged

Added wait for the overlay spinner before the click was performed#22622
HowardBraham merged 4 commits intodevelopfrom
e2e/22566-fix-flaky-test-change-language

Conversation

@hjetpoluru
Copy link
Copy Markdown
Contributor

Description

Tries to fix the "Settings - general tab, validate the change language functionality: User selects "Dansk" language and verify that navigating to a different page does not affect"

Related issues

Fixes: #22566

Screenshot of the cicrcleCI console log
image

@hjetpoluru hjetpoluru added team-extension-platform Extension Platform team INVALID-PR-TEMPLATE PR's body doesn't match template labels Jan 22, 2024
@hjetpoluru hjetpoluru requested a review from a team as a code owner January 22, 2024 18:32
@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.

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ac64a48) 68.12% compared to head (fd45e22) 68.12%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop   #22622   +/-   ##
========================================
  Coverage    68.12%   68.12%           
========================================
  Files         1087     1087           
  Lines        42653    42653           
  Branches     11349    11349           
========================================
  Hits         29056    29056           
  Misses       13597    13597           

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

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [b760c73]
Page Load Metrics (1215 ± 118 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint873241717536
domContentLoaded10213687436
load80016981215246118
domInteractive10213687436
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [03251a4]
Page Load Metrics (1366 ± 151 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint892691696330
domContentLoaded9182576029
load83518731366315151
domInteractive9182576029
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@hjetpoluru hjetpoluru self-assigned this Jan 22, 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.

Tested locally, works good.

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [fd45e22]
Page Load Metrics (1282 ± 144 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint914121808340
domContentLoaded10196576732
load84118741282301144
domInteractive9196576732
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@metamaskbot
Copy link
Copy Markdown
Collaborator

Builds ready [fd45e22]
Page Load Metrics (1282 ± 144 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint914121808340
domContentLoaded10196576732
load84118741282301144
domInteractive9196576732
Bundle size diffs
  • background: 0 Bytes (0.00%)
  • ui: 0 Bytes (0.00%)
  • common: 0 Bytes (0.00%)

@HowardBraham HowardBraham merged commit 3e7cf9a into develop Jan 23, 2024
@HowardBraham HowardBraham deleted the e2e/22566-fix-flaky-test-change-language branch January 23, 2024 17:29
@github-actions github-actions bot locked and limited conversation to collaborators Jan 23, 2024
@metamaskbot metamaskbot added the release-11.10.0 Issue or pull request that will be included in release 11.10.0 label Jan 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

INVALID-PR-TEMPLATE PR's body doesn't match template release-11.10.0 Issue or pull request that will be included in release 11.10.0 team-extension-platform Extension Platform team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix "Settings - general tab, validate the change language..." flaky tests

5 participants