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

Statemine integration test #1712

Merged
merged 27 commits into from
Mar 3, 2022
Merged

Statemine integration test #1712

merged 27 commits into from
Mar 3, 2022

Conversation

xlc
Copy link
Member

@xlc xlc commented Dec 19, 2021

Integration test between Karura & Statemine/Westmint.
Statemine does not support v0.9.13 so we can only use Westmint.

closes: #1763

RMRK transfer back to Statemine test based on: open-web3-stack/open-runtime-module-library#700

manual test:
image

xlc and others added 3 commits December 17, 2021 13:04
* origin/master: (53 commits)
  add new audit report
  Use ExitReason::Revert instead of ExitReason::Error (#1772)
  Claim Account to use Eip-712 (#1755)
  Benchmark evm (#1674)
  support evm create rpc and allow H160 default (#1771)
  Update template files license header. (#1770)
  Fix collect_fee (#1766)
  handle nonce for eth tx (#1707)
  updated to the ORML's test coverage file (#1760)
  rm runtime upgrade (#1757)
  Happy new year 2022. (#1761)
  Excluded some files from test coverage (#1759)
  XCM: add deposit error handler for multi-currency adapter. (#1756)
  update stable asset (#1758)
  Fix test coverage for acala (#1590)
  Fix collect_fee (#1754)
  Update HEADER-GPL3
  Update extrinsic-ordering-check-from-bin.yml (#1752)
  Update HEADER-GPL3
  bump version (#1751)
  ...
@codecov
Copy link

codecov bot commented Jan 13, 2022

Codecov Report

Merging #1712 (cbb8dbc) into master (868ea79) will increase coverage by 0.23%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1712      +/-   ##
==========================================
+ Coverage   86.07%   86.31%   +0.23%     
==========================================
  Files          89       91       +2     
  Lines       18923    19813     +890     
==========================================
+ Hits        16288    17101     +813     
- Misses       2635     2712      +77     
Impacted Files Coverage Δ
modules/currencies/src/tests.rs 100.00% <0.00%> (ø)
modules/currencies/src/lib.rs 79.89% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 868ea79...cbb8dbc. Read the comment docs.

@zqhxuyuan zqhxuyuan changed the title Westmint integration test Statemine integration test Feb 22, 2022
@zqhxuyuan zqhxuyuan marked this pull request as ready for review March 1, 2022 00:49

polkadot-primitives = { git = "https://github.com/paritytech/polkadot", branch = "release-v0.9.16" }
polkadot-runtime-parachains = { git = "https://github.com/paritytech/polkadot", branch = "release-v0.9.16" }
kusama-runtime = { git = "https://github.com/paritytech/polkadot", branch = "release-v0.9.16" }

xcm-emulator = { git = "https://github.com/shaunxw/xcm-simulator", rev = "a250ffc998bac4831c5692c591dee7bc13f3aead" }
#xcm-emulator = { git = "https://github.com/shaunxw/xcm-simulator", rev = "a250ffc998bac4831c5692c591dee7bc13f3aead" }
xcm-emulator = { git = "https://github.com/zqhxuyuan/xcm-simulator", rev = "65832f73b21f55b10dcb09460f8c8ad6ac8083f9" }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

deduplicate DMP: shaunxw/xcm-simulator#24

@zqhxuyuan zqhxuyuan requested a review from shaunxw March 3, 2022 14:11
pub ParachainMinFee: |location: MultiLocation| -> u128 {
#[allow(clippy::match_ref_pats)] // false positive
match (location.parents, location.first_interior()) {
(1, Some(Parachain(parachains::statemine::ID))) => 4_000_000_000,
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be managed by XcmInterface pallet so that we can dynamically updating it if needed

@xlc xlc merged commit 5a7481c into master Mar 3, 2022
@xlc xlc deleted the statemine branch March 3, 2022 21:33
syan095 pushed a commit that referenced this pull request Mar 8, 2022
…count

* origin/master: (21 commits)
  udpate predeploy and disable NFT precompile for karura & acala (#1942)
  remove old migrations (#1937)
  Optimized calculations t avoid error caused by division missing (#1935)
  Statemine integration test (#1712)
  switch to london (#1932)
  fix clap annotation (#1933)
  fix package name (#1928)
  type cleanup (#1931)
  bump version (#1930)
  V0.9.17 (#1924)
  update precompile set (#1929)
  bump version (#1926)
  use internal release for dev release
  fix evm-bench access_list (#1925)
  add missing precompiles and fixes (#1923)
  remove legacy trader location (#1920)
  cdp enhancement (#1886)
  add new call to execute completely fast match redeem (#1922)
  Change StorageDepositPerByte (#1919)
  add ACA to CollateralCurrencyIds (#1918)
  ...

# Conflicts:
#	Cargo.lock
#	orml
#	runtime/integration-tests/src/setup.rs
#	runtime/integration-tests/src/stable_asset.rs
#	runtime/mandala/Cargo.toml
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.

Integration test with Statemine
2 participants