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

✅ ETH Compatibility Tests #425

Merged
merged 3 commits into from
Jan 16, 2025
Merged

✅ ETH Compatibility Tests #425

merged 3 commits into from
Jan 16, 2025

Conversation

JuaniRios
Copy link
Contributor

@JuaniRios JuaniRios commented Oct 24, 2024

What?

  • Add tests

How?

  • Test each new extrinsic
  • Test an ethereum project can be successfully funded
  • Test that the migration info is correctly setup with either the ETH or DOT account

@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from c3a0720 to 7961f2e Compare October 24, 2024 09:37
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from 1cc1e7b to 1da3b0a Compare October 24, 2024 09:38
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch 2 times, most recently from c819f56 to 1413c23 Compare October 24, 2024 09:40
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from 1da3b0a to 2fd7129 Compare October 24, 2024 09:41
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from 1413c23 to d269daa Compare October 24, 2024 09:41
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from 2fd7129 to 252f99a Compare October 24, 2024 10:59
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from d269daa to 88aabca Compare October 24, 2024 10:59
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from 252f99a to 85c8505 Compare October 24, 2024 11:00
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from 88aabca to f7cd35f Compare October 24, 2024 11:01
@JuaniRios JuaniRios changed the title eth compat. tests ✅ ETH Compatibility Tests Oct 24, 2024
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from f7cd35f to 8a90f38 Compare October 24, 2024 12:51
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch 10 times, most recently from 89e6c7e to e552785 Compare October 30, 2024 14:38
@JuaniRios JuaniRios marked this pull request as ready for review October 30, 2024 14:46
@JuaniRios JuaniRios requested review from lrazovic and vstam1 October 30, 2024 14:46
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from 6e95419 to 098e693 Compare November 28, 2024 11:52
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from 946f64d to fe71617 Compare November 28, 2024 11:53
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from 098e693 to e12717a Compare November 28, 2024 12:06
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from 06a8e9e to 57f33f9 Compare November 28, 2024 12:06
This was referenced Nov 29, 2024
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from e12717a to d4e8393 Compare December 20, 2024 13:48
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from 57f33f9 to 760014c Compare December 20, 2024 13:49
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from d4e8393 to 70882fb Compare January 10, 2025 11:49
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from 760014c to 0921bdb Compare January 10, 2025 11:49
@lrazovic lrazovic force-pushed the 10-24-eth_compat._tests branch from 0921bdb to 44bb4a8 Compare January 15, 2025 13:00
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from 44bb4a8 to 0921bdb Compare January 15, 2025 13:03
@JuaniRios JuaniRios force-pushed the 10-14-ethereum_compatibility branch from 70882fb to 64849ec Compare January 16, 2025 09:46
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch 2 times, most recently from 0fd182d to c1d16e0 Compare January 16, 2025 10:27
Copy link
Contributor Author

JuaniRios commented Jan 16, 2025

Merge activity

  • Jan 16, 7:09 AM EST: A user started a stack merge that includes this pull request via Graphite.
  • Jan 16, 7:11 AM EST: Graphite rebased this pull request as part of a merge.
  • Jan 16, 7:12 AM EST: A user merged this pull request with Graphite.

@JuaniRios JuaniRios changed the base branch from 10-14-ethereum_compatibility to graphite-base/425 January 16, 2025 12:09
JuaniRios added a commit that referenced this pull request Jan 16, 2025
## What?
- Add support for Ethereum based projects by letting users define the ethereum account that receives the tokens at the moment of participation.

## How?
- Participation info structs now have a receiving_account field which is an XCM `Junction`. This has to be either `AccountKey20` or `AccountId32`
- Issuers now define at project creation if they expect one or the other account `Junction`.
- Existing participation extrinsics now call the inner participation functions with AccountId32 == caller
- New participation extrinsics now accept a junction, and a signature. The signature will be checked against the account defined in the signature to prove ownership.

## Testing?
In the next PR: #425

## Anything Else?
- For now we only support `sr25519` signatures for Polkadot/Substrate-based accounts, and `secp256k1` for Ethereum.
- The Ethereum signatures need to follow the [`eth_signTypedData_v4`](https://docs.metamask.io/wallet/reference/json-rpc-methods/eth_signtypeddata_v4/) standard: https://eips.ethereum.org/EIPS/eip-712
@JuaniRios JuaniRios changed the base branch from graphite-base/425 to main January 16, 2025 12:09
@JuaniRios JuaniRios force-pushed the 10-24-eth_compat._tests branch from c1d16e0 to 2954cf9 Compare January 16, 2025 12:10
@JuaniRios JuaniRios merged commit 98046a2 into main Jan 16, 2025
1 check passed
@JuaniRios JuaniRios deleted the 10-24-eth_compat._tests branch January 16, 2025 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants