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

[FLASK] Snaps e2e test stability improvements #18090

Merged
merged 3 commits into from
Mar 15, 2023
Merged

Conversation

bowensanders
Copy link
Contributor

Using waitForElement instead of arbitrary delays in many locations
Using pasteIntoField instead of fill in a few locations
other delay changes as needed

@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2023

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 [53d328b]
Page Load Metrics (1670 ± 56 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint99168124189
domContentLoaded14881866163611656
load15211890167011856
domInteractive14881866163611656
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@bowensanders bowensanders marked this pull request as ready for review March 9, 2023 21:54
@bowensanders bowensanders requested a review from a team as a code owner March 9, 2023 21:54
@codecov
Copy link

codecov bot commented Mar 9, 2023

Codecov Report

Merging #18090 (2eb81e3) into develop (423e085) will increase coverage by 0.06%.
The diff coverage is 55.83%.

@@             Coverage Diff             @@
##           develop   #18090      +/-   ##
===========================================
+ Coverage    63.86%   63.92%   +0.06%     
===========================================
  Files          908      908              
  Lines        35390    35397       +7     
  Branches      8966     8974       +8     
===========================================
+ Hits         22599    22625      +26     
+ Misses       12791    12772      -19     
Impacted Files Coverage Δ
app/scripts/controllers/preferences.js 77.16% <ø> (-2.73%) ⬇️
app/scripts/migrations/index.js 100.00% <ø> (ø)
shared/constants/metametrics.js 100.00% <ø> (ø)
shared/constants/network.ts 100.00% <ø> (ø)
...delete-network/confirm-delete-network.container.js 66.67% <0.00%> (ø)
ui/ducks/app/app.ts 62.28% <0.00%> (+1.60%) ⬆️
ui/ducks/metamask/metamask.js 76.19% <ø> (ø)
...rm-transaction/confirm-token-transaction-switch.js 5.26% <0.00%> (-3.07%) ⬇️
...tion-network-switch/confirmation-network-switch.js 11.11% <ø> (ø)
...pages/confirmation/templates/add-ethereum-chain.js 13.56% <0.00%> (-0.23%) ⬇️
... and 24 more

... and 3 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@bowensanders
Copy link
Contributor Author

Resolves issue #17643

Copy link
Member

@FrederikBolding FrederikBolding left a comment

Choose a reason for hiding this comment

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

left a few comments, most of them apply to all files

test/e2e/snaps/test-snap-cronjob.spec.js Outdated Show resolved Hide resolved
test/e2e/snaps/test-snap-bip-44.spec.js Outdated Show resolved Hide resolved
test/e2e/snaps/test-snap-bip-32.spec.js Show resolved Hide resolved
test/e2e/snaps/test-snap-rpc.spec.js Outdated Show resolved Hide resolved
@metamaskbot
Copy link
Collaborator

Builds ready [b0d600d]
Page Load Metrics (1693 ± 93 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint951821312211
domContentLoaded13832024167820096
load13832025169319493
domInteractive13832024167819996
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@metamaskbot
Copy link
Collaborator

Builds ready [2eb81e3]
Page Load Metrics (1659 ± 65 ms)
PlatformPageMetricMin (ms)Max (ms)Average (ms)StandardDeviation (ms)MarginOfError (ms)
ChromeHomefirstPaint99177132199
domContentLoaded12861970163714771
load13811970165913665
domInteractive12861970163714771
Bundle size diffs
  • background: 0 bytes
  • ui: 0 bytes
  • common: 0 bytes

@bowensanders bowensanders self-assigned this Mar 14, 2023
@bowensanders bowensanders linked an issue Mar 14, 2023 that may be closed by this pull request
Copy link
Contributor

@hmalik88 hmalik88 left a comment

Choose a reason for hiding this comment

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

LGTM

@bowensanders bowensanders merged commit 7a421e0 into develop Mar 15, 2023
@bowensanders bowensanders deleted the bowen/e2e-improve branch March 15, 2023 17:03
@github-actions github-actions bot locked and limited conversation to collaborators Mar 15, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FLASK] Make improvements to Snaps E2E tests
4 participants