Skip to content

feat: added Solana bridge e2e tests #33810

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

javiergarciavera
Copy link
Contributor

@javiergarciavera javiergarciavera commented Jun 20, 2025

Description

Open in GitHub Codespaces

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

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.

@javiergarciavera javiergarciavera requested a review from a team as a code owner June 20, 2025 15:22
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 metamaskbot added the team-qa QA team label Jun 20, 2025
@metamaskbot
Copy link
Collaborator

✨ Files requiring CODEOWNER review ✨

🔄 @MetaMask/swaps-engineers (26 files, +5624 -1)
  • 📁 test/
    • 📁 e2e/
      • 📁 tests/
        • 📁 bridge/
          • 📄 bridge-test-utils.ts +1 -1
        • 📁 solana/
          • 📁 mocks/
            • 📁 bridge/
              • 📄 asdasda
              • 📄 batchStatusSuccess.json +14 -0
              • 📄 bridgeEthGetTokens.json +133 -0
              • 📄 bridgeEthTopAssets.json +26 -0
              • 📄 bridgeEthToSolTransactionStatusComplete.json +51 -0
              • 📄 bridgeSolanaGetTokens.json +133 -0
              • 📄 bridgeSolToEthAccountsApi.json +1574 -0
              • 📄 bridgeTransactionStatusComplete.json +50 -0
              • 📄 bridgeTransactionStatusPending.json +11 -0
              • 📄 ethAccountPositiveBalance.json +125 -0
              • 📄 ethCall.json +5 -0
              • 📄 ethGetCode.json +5 -0
              • 📄 ethGetTransactionReceipt.json +102 -0
              • 📄 ethSendRawTransaction.json +5 -0
              • 📄 ethToSolTransaction.json +493 -0
              • 📄 getSuccessTransactionEthToSol.json +433 -0
              • 📄 quoteEthToSolBridge.json +446 -0
              • 📄 quoteSolToEthBridge.json +383 -0
              • 📄 solToEthTransaction.json +493 -0
              • 📄 spotPriceSolanaBridge.json +5 -0
              • 📄 spotPriceSolanaBridgeMarketData.json +86 -0
              • 📄 submitEthToSolTx.json +1 -0
              • 📄 tokenApiEth.json +52 -0
              • 📄 transactionsAccountApi.json +44 -0
              • 📄 transactionsAccountApiSolToEth.json +953 -0

@metamaskbot
Copy link
Collaborator

Builds ready [9dda5fa]
UI Startup Metrics (1206 ± 57 ms)
PlatformBuildTypePageMetricMean (ms)Min (ms)Max (ms)Std Dev (ms)P 75 (ms)P 95 (ms)
ChromeBrowserifyHomeuiStartup1206109514265712481289
load104494712185510821139
domContentLoaded103794011995610751133
domInteractive17134961735
firstPaint78194120439310651118
backgroundConnect84728718
firstReactRender21144752231
getState1563272027
initialActions001001
loadScripts79069693654827883
setupStore85183816
WebpackHomeuiStartup22421729276621223582666
load17291277221818918521999
domContentLoaded17231273221218918481994
domInteractive171165131457
firstPaint1696357980185299
backgroundConnect2210278262234
firstReactRender21091414101327397
getState144301301229
initialActions315134
loadScripts17201271220118818461992
setupStore237313491726
FirefoxBrowserifyHomeuiStartup13921259188710014241538
load1225110016018312601391
domContentLoaded1224110016018312601390
domInteractive1123652656126179
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect2414121162359
firstReactRender24223122527
getState105779924
initialActions001001
loadScripts1204108515748012381350
setupStore84767713
WebpackHomeuiStartup15941381228719416882105
load13771190201217014721787
domContentLoaded13761190201217014721787
domInteractive97334877690283
firstPaintNaNNaNNaNNaNNaNNaN
backgroundConnect23164582442
firstReactRender41306544447
getState125114131132
initialActions102111
loadScripts13571175198416914541763
setupStore95305821
Benchmark value 19 exceeds gate value 18 for chrome browserify home p95 backgroundConnect
Benchmark value 2243 exceeds gate value 2192 for chrome webpack home mean uiStartup
Benchmark value 1729 exceeds gate value 1711 for chrome webpack home mean load
Benchmark value 1724 exceeds gate value 1704 for chrome webpack home mean domContentLoaded
Benchmark value 1721 exceeds gate value 1699 for chrome webpack home mean loadScripts
Benchmark value 2666 exceeds gate value 2454 for chrome webpack home p95 uiStartup
Benchmark value 397 exceeds gate value 370 for chrome webpack home p95 firstReactRender
Benchmark value 1993 exceeds gate value 1970 for chrome webpack home p95 loadScripts
Benchmark value 113 exceeds gate value 110 for firefox browserify home mean domInteractive
Benchmark value 42 exceeds gate value 38 for firefox webpack home mean firstReactRender
Benchmark value 2105 exceeds gate value 1935 for firefox webpack home p95 uiStartup
Benchmark value 1787 exceeds gate value 1660 for firefox webpack home p95 load
Benchmark value 1787 exceeds gate value 1660 for firefox webpack home p95 domContentLoaded
Benchmark value 283 exceeds gate value 156 for firefox webpack home p95 domInteractive
Benchmark value 1763 exceeds gate value 1630 for firefox webpack home p95 loadScripts
Sum of mean exceeds: 118ms | Sum of p95 exceeds: 947ms
Sum of all benchmark exceeds: 1065ms

Bundle size diffs
  • background: 0 Bytes (0%)
  • ui: 0 Bytes (0%)
  • common: 0 Bytes (0%)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-qa QA team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants