-
Notifications
You must be signed in to change notification settings - Fork 337
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
Test XCM transfer and pay fees with foreign asset #3047
base: master
Are you sure you want to change the base?
Conversation
WASM runtime size check:Compared to target branchMoonbase runtime: 2260 KB (no changes) ✅ Moonbeam runtime: 2232 KB (no changes) ✅ Moonriver runtime: 2224 KB (no changes) ✅ Compared to latest release (runtime-3300)Moonbase runtime: 2260 KB (+232 KB compared to latest release) Moonbeam runtime: 2232 KB (+236 KB compared to latest release) Moonriver runtime: 2224 KB (+232 KB compared to latest release) |
Coverage Report@@ Coverage Diff @@
## master gonza/test-xcm-transfer-and-pay-fees +/- ##
========================================================================
+ Coverage 74.81% 79.00% +4.19%
- Files 369 303 -66
- Lines 94137 88208 -5929
========================================================================
- Hits 70421 69685 -736
- Misses 23716 18523 -5193
|
test/suites/dev/moonbase/test-xcm-v4/test-xcm-evm-fee-with-origin-token.ts
Outdated
Show resolved
Hide resolved
}, | ||
}); | ||
|
||
async function registerAndFundAsset( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should move this function to helpers
folder. I would also suggest renaming it to registerAndFundOldAsset
.
Also, it would be nice to add a small comment explaining why the function was created.
test/suites/dev/moonbase/test-xcm-v4/test-xcm-evm-fee-with-origin-token.ts
Outdated
Show resolved
Hide resolved
}; | ||
|
||
/** | ||
* We are going to test the following scenario: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice docs btw :)
…gin-token.ts Co-authored-by: Agustín Rodriguez <[email protected]>
…gin-token.ts Co-authored-by: Agustín Rodriguez <[email protected]>
We test that an account in some parachain, can transfer some foreign assets to an account in Moonbeam, paying in another foreign asset. So that they don't need to buy GLMR for buying time for the execution.