Skip to content

feat(TAT-575): add fixture to import random generated accounts#17287

Merged
jmcruz1983 merged 1 commit into
mainfrom
feat-tat-575-e2e-testing-utilities
Jul 23, 2025
Merged

feat(TAT-575): add fixture to import random generated accounts#17287
jmcruz1983 merged 1 commit into
mainfrom
feat-tat-575-e2e-testing-utilities

Conversation

@jmcruz1983

@jmcruz1983 jmcruz1983 commented Jul 16, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor
  • add encrypt & decrypt vault utilities in fixture helpers
  • convert TEST_DAPP_LOCAL_URL into a function
  • replace SOLANA_TEST_DAPP_LOCAL_URL with function
  • add code formatting

Description

Changelog

CHANGELOG entry:

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.

@github-actions

github-actions Bot commented Jul 16, 2025

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.

@jmcruz1983 jmcruz1983 added team-qa QA team No QA Needed Apply this label when your PR does not need any QA effort. Run Smoke E2E labels Jul 16, 2025
@github-actions

github-actions Bot commented Jul 16, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: df13438fddb64ef780925004342de7956c5c5ff3
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/cfc53324-869a-41e4-8a8b-0fc56c5d6692

Note

  • You can rerun any failed steps by opening the Bitrise build, tapping Rebuild on the upper right then Rebuild unsuccessful Workflows
  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch 2 times, most recently from 0cf33e1 to 5ca849e Compare July 17, 2025 08:03
@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch from 5ca849e to 7e35ac1 Compare July 17, 2025 10:45
@github-actions

github-actions Bot commented Jul 17, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 7e35ac14c077f188bdcabce8e6e19b634df5a39d
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/dd99f5ab-7bcb-4c83-9fe4-331ed87440b6

Note

  • You can rerun any failed steps by opening the Bitrise build, tapping Rebuild on the upper right then Rebuild unsuccessful Workflows
  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch from 7e35ac1 to 04a8bdf Compare July 17, 2025 11:10
@github-actions

github-actions Bot commented Jul 17, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 04a8bdff6e7ffed85c6064e56522bebc1f76e127
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/76daad74-a9c6-4730-baef-d13d90676818

Note

  • You can rerun any failed steps by opening the Bitrise build, tapping Rebuild on the upper right then Rebuild unsuccessful Workflows
  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch 2 times, most recently from 3bac422 to 5eb0fbd Compare July 17, 2025 11:54
@codecov-commenter

codecov-commenter commented Jul 17, 2025

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.47%. Comparing base (c6a8f54) to head (5eb0fbd).
Report is 18 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main   #17287      +/-   ##
==========================================
+ Coverage   73.36%   73.47%   +0.10%     
==========================================
  Files        2802     2821      +19     
  Lines       61803    62246     +443     
  Branches     9867     9967     +100     
==========================================
+ Hits        45340    45733     +393     
- Misses      13587    13631      +44     
- Partials     2876     2882       +6

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch 5 times, most recently from 87e16c9 to 688bf6d Compare July 20, 2025 10:44
@github-actions

github-actions Bot commented Jul 20, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: 688bf6dc77be8999c2a259772d0eb4eddf3e5ec5
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/d61f8609-161e-4771-aea3-c516c5c35b56

Note

  • You can rerun any failed steps by opening the Bitrise build, tapping Rebuild on the upper right then Rebuild unsuccessful Workflows
  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch from 688bf6d to 4367bea Compare July 21, 2025 16:22
cmd-ob
cmd-ob previously approved these changes Jul 22, 2025
View reviewed changes
@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch from b3215b8 to d5c871c Compare July 22, 2025 13:11
@github-actions

github-actions Bot commented Jul 22, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

❌❌❌ pr_smoke_e2e_pipeline failed on Bitrise! ❌❌❌

Commit hash: d5c871cc71b86e12a53920a9672ab2682651876a
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/1ed07f58-599d-462c-ae5e-706b59e6ea38

Note

  • You can rerun any failed steps by opening the Bitrise build, tapping Rebuild on the upper right then Rebuild unsuccessful Workflows
  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

Tip

  • Check the documentation if you have any doubts on how to understand the failure on bitrise

@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch 2 times, most recently from f2e133c to 70bfa96 Compare July 22, 2025 15:45
@github-actions

github-actions Bot commented Jul 22, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 70bfa9654ed0dcc61ab19c378d3af676df364198
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/6988953e-542a-4300-8413-5f7971a39c9d

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@jmcruz1983
feat(TAT-575): add fixture to import random generated accounts
4794b70 
- add encrypt & decrypt vault utilities in fixture helpers
- convert TEST_DAPP_LOCAL_URL into a function
- replace SOLANA_TEST_DAPP_LOCAL_URL with function
- add code formatting
@jmcruz1983 jmcruz1983 force-pushed the feat-tat-575-e2e-testing-utilities branch from 70bfa96 to 4794b70 Compare July 23, 2025 07:52
@sonarqubecloud

sonarqubecloud Bot commented Jul 23, 2025

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@github-actions

github-actions Bot commented Jul 23, 2025 •
edited by metamaskbot
Loading

Copy link
Copy Markdown
Contributor

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 4794b70
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/7aeaae76-a8c9-441d-8aeb-e6d48c2cf0e8

Note

  • You can kick off another pr_smoke_e2e_pipeline on Bitrise by removing and re-applying the Run Smoke E2E label on the pull request

@jmcruz1983 jmcruz1983 added this pull request to the merge queue Jul 23, 2025
Merged via the queue into main with commit 4dee77e Jul 23, 2025
56 of 58 checks passed
@jmcruz1983 jmcruz1983 deleted the feat-tat-575-e2e-testing-utilities branch July 23, 2025 09:33
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 23, 2025
@metamaskbot metamaskbot added the release-7.53.0 Issue or pull request that will be included in release 7.53.0 label Jul 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@cmd-ob cmd-ob cmd-ob approved these changes

Assignees

No one assigned

Labels

No QA Needed Apply this label when your PR does not need any QA effort. release-7.53.0 Issue or pull request that will be included in release 7.53.0 team-qa QA team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jmcruz1983 @codecov-commenter @cmd-ob @metamaskbot