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.

Sign up for GitHub

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

Jump to bottom

test: Redesign Signature Decoding Simulation #13026

Merged
merged 92 commits into from
Jan 16, 2025
Merged

test: Redesign Signature Decoding Simulation #13026

merged 92 commits into from
Jan 16, 2025
+300 −29

Conversation

digiwand
Copy link
Contributor

@digiwand digiwand commented Jan 16, 2025 •
edited
Loading

Description

Create Additional Tests for new Decoding Simulation feat

Related issues

Fixes: #13023
Relates to: #12994

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.

digiwand added 30 commits December 6, 2024 21:53
@digiwand
feat: Permit Simulation
87b746e 
todo:
- metrics
- precision tooltip
- tests
- styles:
  - right align row values
  - hide
@digiwand
refactor: PermitSimulation const name
60086e1
@digiwand
fix: label_change_type i18n
ae2a2cb
@digiwand
feat: add PermitSimulation metrics
5b5d094
@digiwand
feat: create ButtonPill component
dfc086f
@digiwand
feat: PermitSimulation token value BottomModal
d96f9aa
@digiwand
Merge branch 'main' into feat-permit-simulation
a665679
@digiwand
feat: Add Permit Simulation to TypedSignV3V4 redesign
518e51d
@digiwand
refactor: cleanup
f069e37
@digiwand
chore: update todo ValueDisplay shorten comment
fe15f12
@digiwand
refactor: update useGetTokenStandardAndDetails import
6777319
@digiwand
fix: PermitSimulation changeType
4bc90e0
@digiwand
test: PermitSimulation
b65adf3
@digiwand
fix: useTrackERC20WithoutDecimalInformation should run once
a79f2cd
@digiwand
fix: include typedSignV4ConfirmationState for PermitSimulation test
a5d96ec
@digiwand
test: create PermitSimulationValueDisplay test
05669d3 
todo:
- fix rerender issue
- fix removeEventListener detached console error
@digiwand
fix: TS lint signature util
bcccb06
@digiwand
Merge branch 'main' into feat-permit-simulation
22f754f
@digiwand
fix: readd dep to useTrackERC20WithoutDecimalInformation
567fc66
@digiwand
chore: update useTrackERC20WithoutDecimalInformation TS metricLocation
0c215c4
@digiwand
test: ValueDisplay test
1683123
@digiwand
fix: test allow style param InfoAddress
79a4003
@digiwand
test:fix: OrderDetails allow undefined style param
aa43eb9
@digiwand
test:fix: include InfoAddress undefined style param
8577591
@digiwand
chore: ButtonPill pressed -> isPressed
d53035f
@digiwand
build: mv ButtonPill files -> components-temp
2bdb950
@digiwand
test: create confirmations/utils/signature test
1ffe22e
@digiwand
test: add parseTypedDataMessage tests
779f36f
@digiwand
test: add shortenString tests for skipCharacterInEnd + update TS
5852d87
@digiwand
Merge branch 'main' into feat-permit-simulation
fce5367
@github-actions GitHub Actions
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-confirmations Push issues to confirmations team label Jan 16, 2025
@digiwand digiwand marked this pull request as ready for review January 16, 2025 10:51
@digiwand digiwand requested review from a team as code owners January 16, 2025 10:51
OGPoyraz
OGPoyraz previously approved these changes Jan 16, 2025
View reviewed changes
Base automatically changed from feat-decoding-simulation to main January 16, 2025 15:51
@digiwand digiwand dismissed OGPoyraz’s stale review January 16, 2025 15:51

The base branch was changed.

@github-actions github-actions bot requested a review from a team as a code owner January 16, 2025 15:51
@digiwand digiwand added the Run Smoke E2E Triggers smoke e2e on Bitrise label Jan 16, 2025
@digiwand digiwand requested a review from OGPoyraz January 16, 2025 16:01
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 16, 2025 •
edited by metamaskbot
Loading

https://bitrise.io/ Bitrise

✅✅✅ pr_smoke_e2e_pipeline passed on Bitrise! ✅✅✅

Commit hash: 854fa84
Build link: https://app.bitrise.io/app/be69d4368ee7e86d/pipelines/380bb752-64e7-4a2a-98a6-b5195a038d85

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

@sonarqubecloud SonarQubeCloud
Copy link

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

@digiwand digiwand added this pull request to the merge queue Jan 16, 2025
Merged via the queue into main with commit f7e30ca Jan 16, 2025
38 of 39 checks passed
@digiwand digiwand deleted the feat-decoding-simulation-tests branch January 16, 2025 16:52
@github-actions github-actions bot locked and limited conversation to collaborators Jan 16, 2025
@metamaskbot metamaskbot added the release-7.39.0 Issue or pull request that will be included in release 7.39.0 label Jan 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@matthewwalsh0 matthewwalsh0 matthewwalsh0 approved these changes

@OGPoyraz OGPoyraz OGPoyraz approved these changes

Assignees
No one assigned
Labels
release-7.39.0 Issue or pull request that will be included in release 7.39.0 Run Smoke E2E Triggers smoke e2e on Bitrise team-confirmations Push issues to confirmations team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add Redesign Signature Decoding Simulation tests
4 participants
@digiwand @OGPoyraz @matthewwalsh0 @metamaskbot